diff options
| author | Johnny Willemsen <jwillemsen@remedy.nl> | 2012-12-09 14:38:45 +0000 |
|---|---|---|
| committer | Johnny Willemsen <jwillemsen@remedy.nl> | 2012-12-09 14:38:45 +0000 |
| commit | 28e57c18c33347da6dbae6f259448bc405c776e8 (patch) | |
| tree | d3f4f9cea997caf7c642d9072bae7ee83a9e6252 /ACE | |
| parent | 8de4a68f954846cf6a9bc3d47c6a3e5b40810042 (diff) | |
| download | ATCD-28e57c18c33347da6dbae6f259448bc405c776e8.tar.gz | |
Sun Dec 9 14:38:27 UTC 2012 Johnny Willemsen <jwillemsen@remedy.nl>
* ace/POSIX_Proactor.cpp:
* ace/Parse_Node.cpp:
* ace/SV_Message_Queue.inl:
* ace/SV_Message_Queue.cpp:
Fixed coverity errors
Diffstat (limited to 'ACE')
| -rw-r--r-- | ACE/ChangeLog | 8 | ||||
| -rw-r--r-- | ACE/ace/POSIX_Proactor.cpp | 6 | ||||
| -rw-r--r-- | ACE/ace/Parse_Node.cpp | 1 | ||||
| -rw-r--r-- | ACE/ace/SV_Message_Queue.cpp | 3 | ||||
| -rw-r--r-- | ACE/ace/SV_Message_Queue.inl | 2 |
5 files changed, 17 insertions, 3 deletions
diff --git a/ACE/ChangeLog b/ACE/ChangeLog index 7dbfdc41f41..7074ba24b67 100644 --- a/ACE/ChangeLog +++ b/ACE/ChangeLog @@ -1,3 +1,11 @@ +Sun Dec 9 14:38:27 UTC 2012 Johnny Willemsen <jwillemsen@remedy.nl> + + * ace/POSIX_Proactor.cpp: + * ace/Parse_Node.cpp: + * ace/SV_Message_Queue.inl: + * ace/SV_Message_Queue.cpp: + Fixed coverity errors + Fri Dec 7 20:14:27 UTC 2012 Johnny Willemsen <jwillemsen@remedy.nl> * include/makeinclude/platform_g++_common.GNU: diff --git a/ACE/ace/POSIX_Proactor.cpp b/ACE/ace/POSIX_Proactor.cpp index 5f9c4bc4c28..eec70234c90 100644 --- a/ACE/ace/POSIX_Proactor.cpp +++ b/ACE/ace/POSIX_Proactor.cpp @@ -630,7 +630,11 @@ ACE_AIOCB_Notify_Pipe_Manager::ACE_AIOCB_Notify_Pipe_Manager (ACE_POSIX_AIOCB_Pr read_stream_ (posix_aiocb_proactor) { // Open the pipe. - this->pipe_.open (); + if (this->pipe_.open () == -1) + ACE_ERROR ((LM_ERROR, + ACE_TEXT("%N:%l:%p\n"), + ACE_TEXT("ACE_AIOCB_Notify_Pipe_Manager::ACE_AIOCB_Notify_Pipe_Manager:") + ACE_TEXT("Open of pipe failed"))); // Set write side in NONBLOCK mode ACE::set_flags (this->pipe_.write_handle (), ACE_NONBLOCK); diff --git a/ACE/ace/Parse_Node.cpp b/ACE/ace/Parse_Node.cpp index 73b4d20ada1..3e8cbdc7268 100644 --- a/ACE/ace/Parse_Node.cpp +++ b/ACE/ace/Parse_Node.cpp @@ -426,6 +426,7 @@ ACE_Location_Node::dump (void) const ACE_Location_Node::ACE_Location_Node (void) : pathname_ (0), + must_delete_ (0), dll_ (), symbol_ (0) { diff --git a/ACE/ace/SV_Message_Queue.cpp b/ACE/ace/SV_Message_Queue.cpp index 332464d52df..6118fce8c40 100644 --- a/ACE/ace/SV_Message_Queue.cpp +++ b/ACE/ace/SV_Message_Queue.cpp @@ -21,7 +21,8 @@ ACE_SV_Message_Queue::dump (void) const #endif /* ACE_HAS_DUMP */ } -ACE_SV_Message_Queue::ACE_SV_Message_Queue (void) +ACE_SV_Message_Queue::ACE_SV_Message_Queue (void) : + internal_id_ (-1) { ACE_TRACE ("ACE_SV_Message_Queue::ACE_SV_Message_Queue"); } diff --git a/ACE/ace/SV_Message_Queue.inl b/ACE/ace/SV_Message_Queue.inl index b84f6379901..a8cd134776c 100644 --- a/ACE/ace/SV_Message_Queue.inl +++ b/ACE/ace/SV_Message_Queue.inl @@ -38,7 +38,7 @@ ACE_INLINE int ACE_SV_Message_Queue::remove (void) { ACE_TRACE ("ACE_SV_Message_Queue::remove"); - int result = this->control (IPC_RMID); + int const result = this->control (IPC_RMID); this->internal_id_ = -1; return result; } |