From 28e57c18c33347da6dbae6f259448bc405c776e8 Mon Sep 17 00:00:00 2001 From: Johnny Willemsen Date: Sun, 9 Dec 2012 14:38:45 +0000 Subject: Sun Dec 9 14:38:27 UTC 2012 Johnny Willemsen * ace/POSIX_Proactor.cpp: * ace/Parse_Node.cpp: * ace/SV_Message_Queue.inl: * ace/SV_Message_Queue.cpp: Fixed coverity errors --- ACE/ChangeLog | 8 ++++++++ ACE/ace/POSIX_Proactor.cpp | 6 +++++- ACE/ace/Parse_Node.cpp | 1 + ACE/ace/SV_Message_Queue.cpp | 3 ++- ACE/ace/SV_Message_Queue.inl | 2 +- 5 files changed, 17 insertions(+), 3 deletions(-) (limited to 'ACE') 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 + + * 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 * 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; } -- cgit v1.2.1