BUG#: 8696
authorsahana.prabhakar <sahana.prabhakar>
Wed, 27 Jan 2010 06:36:30 +0000 (06:36 +0000)
committersahana.prabhakar <sahana.prabhakar>
Wed, 27 Jan 2010 06:36:30 +0000 (06:36 +0000)
TITLE: Leak in CIMConfigCommand.cpp.
DESCRIPTION: The ConfigFileHandler pointer is not deleted in CIMConfigCommand.cpp.

src/Clients/cimconfig/CIMConfigCommand.cpp
src/Clients/cimconfig/CIMConfigCommand.h

index b78907c087ab969322707afde6c31d9e729072b5..6d57f163183c4236a77f14f0c36e9619712e8669 100644 (file)
@@ -995,8 +995,8 @@ Uint32 CIMConfigCommand::execute(
             //
             // Open default config files and load current config properties
             //
-            _configFileHandler =
-                new ConfigFileHandler(currentFile, plannedFile, true);
+            _configFileHandler.reset(
+                new ConfigFileHandler(currentFile, plannedFile, true));;
         }
         catch (const NoSuchFile&)
         {
index 71809e54ca7bea9dab798d44f0432f1cfd99e787..e718bf472e760c5bddf7fae2f85a7d1d1712c41c 100644 (file)
@@ -142,7 +142,7 @@ private:
     /**
         Configuration File handler
     */
-    ConfigFileHandler* _configFileHandler;
+    AutoPtr<ConfigFileHandler> _configFileHandler;
 
     /**
     The CIM Client reference