From 75f23e960026acbf5c93f8598cf4ab056ff4cbda Mon Sep 17 00:00:00 2001 From: Johnny Willemsen Date: Mon, 12 Nov 2012 19:27:25 +0000 Subject: Mon Nov 12 19:26:41 UTC 2012 Johnny Willemsen * ace/Malloc.cpp: * ace/Naming_Context.h: * ace/Naming_Context.cpp: * ace/Read_Buffer.cpp: * ace/Recursive_Thread_Mutex.cpp: * ace/SV_Semaphore_Simple.cpp: Coverity fixes --- ACE/ChangeLog | 10 ++++++++++ ACE/ace/Malloc.cpp | 4 ++++ ACE/ace/Naming_Context.cpp | 3 ++- ACE/ace/Naming_Context.h | 1 - ACE/ace/Read_Buffer.cpp | 8 ++++++-- ACE/ace/Recursive_Thread_Mutex.cpp | 4 ---- ACE/ace/SV_Semaphore_Simple.cpp | 3 ++- 7 files changed, 24 insertions(+), 9 deletions(-) diff --git a/ACE/ChangeLog b/ACE/ChangeLog index 26840c9238a..cfa41d8181e 100644 --- a/ACE/ChangeLog +++ b/ACE/ChangeLog @@ -1,3 +1,13 @@ +Mon Nov 12 19:26:41 UTC 2012 Johnny Willemsen + + * ace/Malloc.cpp: + * ace/Naming_Context.h: + * ace/Naming_Context.cpp: + * ace/Read_Buffer.cpp: + * ace/Recursive_Thread_Mutex.cpp: + * ace/SV_Semaphore_Simple.cpp: + Coverity fixes + Mon Nov 12 17:21:25 UTC 2012 Johnny Willemsen * ace/OS_NS_Thread.inl: diff --git a/ACE/ace/Malloc.cpp b/ACE/ace/Malloc.cpp index 4a7672b5041..a31ff854f7e 100644 --- a/ACE/ace/Malloc.cpp +++ b/ACE/ace/Malloc.cpp @@ -98,6 +98,10 @@ ACE_Control_Block::dump (void) const } ACE_Control_Block::ACE_Name_Node::ACE_Name_Node (void) + : name_ (0), + pointer_ (0), + next_ (0), + prev_ (0) { ACE_TRACE ("ACE_Control_Block::ACE_Name_Node::ACE_Name_Node"); } diff --git a/ACE/ace/Naming_Context.cpp b/ACE/ace/Naming_Context.cpp index 131abb65ebc..f45c355c59b 100644 --- a/ACE/ace/Naming_Context.cpp +++ b/ACE/ace/Naming_Context.cpp @@ -140,7 +140,8 @@ ACE_Naming_Context::close (void) ACE_Naming_Context::ACE_Naming_Context (void) : name_options_ (0), - name_space_ (0) + name_space_ (0), + netnameserver_host_ (0) { ACE_TRACE ("ACE_Naming_Context::ACE_Naming_Context"); diff --git a/ACE/ace/Naming_Context.h b/ACE/ace/Naming_Context.h index 9d85b65b74f..d260fae0245 100644 --- a/ACE/ace/Naming_Context.h +++ b/ACE/ace/Naming_Context.h @@ -269,7 +269,6 @@ private: /// 1 if we're on the same local machine as the name server, else 0. int local (void); - }; /** diff --git a/ACE/ace/Read_Buffer.cpp b/ACE/ace/Read_Buffer.cpp index b11702de11d..544b407f934 100644 --- a/ACE/ace/Read_Buffer.cpp +++ b/ACE/ace/Read_Buffer.cpp @@ -32,7 +32,9 @@ ACE_Read_Buffer::dump (void) const ACE_Read_Buffer::ACE_Read_Buffer (FILE *fp, bool close_on_delete, ACE_Allocator *alloc) - : stream_ (fp), + : size_ (0), + occurrences_ (0), + stream_ (fp), close_on_delete_ (close_on_delete), allocator_ (alloc) { @@ -45,7 +47,9 @@ ACE_Read_Buffer::ACE_Read_Buffer (FILE *fp, ACE_Read_Buffer::ACE_Read_Buffer (ACE_HANDLE handle, bool close_on_delete, ACE_Allocator *alloc) - : stream_ (ACE_OS::fdopen (handle, ACE_TEXT ("r"))), + : size_ (0), + occurrences_ (0), + stream_ (ACE_OS::fdopen (handle, ACE_TEXT ("r"))), close_on_delete_ (close_on_delete), allocator_ (alloc) { diff --git a/ACE/ace/Recursive_Thread_Mutex.cpp b/ACE/ace/Recursive_Thread_Mutex.cpp index 07603a9cc4e..1b43bbb6e61 100644 --- a/ACE/ace/Recursive_Thread_Mutex.cpp +++ b/ACE/ace/Recursive_Thread_Mutex.cpp @@ -103,10 +103,6 @@ ACE_Recursive_Thread_Mutex::get_nesting_level (void) #endif /* !ACE_HAS_WINCE */ } -ACE_Recursive_Thread_Mutex::ACE_Recursive_Thread_Mutex (const ACE_Recursive_Thread_Mutex &) -{ -} - void ACE_Recursive_Thread_Mutex::dump (void) const { diff --git a/ACE/ace/SV_Semaphore_Simple.cpp b/ACE/ace/SV_Semaphore_Simple.cpp index dbc8f139580..bdf4531f4e1 100644 --- a/ACE/ace/SV_Semaphore_Simple.cpp +++ b/ACE/ace/SV_Semaphore_Simple.cpp @@ -211,7 +211,8 @@ ACE_SV_Semaphore_Simple::~ACE_SV_Semaphore_Simple (void) this->close (); } -ACE_SV_Semaphore_Simple::ACE_SV_Semaphore_Simple (void) +ACE_SV_Semaphore_Simple::ACE_SV_Semaphore_Simple (void) : + sem_number_ (0) { ACE_TRACE ("ACE_SV_Semaphore_Simple::ACE_SV_Semaphore_Simple"); this->init (); -- cgit v1.2.1