Changeset 8327 for trunk/src/kernel32/handlenames.cpp

Timestamp:

Apr 29, 2002, 7:05:30 PM (23 years ago)

Author:

sandervl

Message:

PH: memory leak fixes; extra checks for NULL pointers; CreateProcess changes for debug option; GetModuleFileName doesn't count 0 terminator

File:

• trunk/src/kernel32/handlenames.cpp (modified) (3 diffs)

trunk/src/kernel32/handlenames.cpp

@@ -1 @@
-/* $Id: handlenames.cpp,v 1.4 2002-04-08 11:22:47 phaller Exp $ */
+/* $Id: handlenames.cpp,v 1.5 2002-04-29 17:05:29 sandervl Exp $ */
 
 /*
@@ -117 +117 @@
 HandleNames::HandleNames(void)
 {
+  // Note: as handleNameMgr is a static object, the
+  // destructor will never be called, this memory is leaked.
   pSymbolicLinks = new CLinearList();
 }
@@ -333 +335 @@
     rc = FALSE;
   else
+  {
     // 2 - remove the link
     pSymbolicLinks->removeObject(pHandleName);
+    
+    if (NULL != pHandleName->pszSymbolicLink )
+      free( pHandleName->pszSymbolicLink );
+    
+    if (NULL != pHandleName->pszTarget )
+      free( pHandleName->pszTarget );
+    
+    free( pHandleName );
+  }
   
   mtxHandleNameMgr.leave();