[ros-diffs] [cwittich] 32533: fix some more memory leaks

cwittich at svn.reactos.org cwittich at svn.reactos.org
Fri Feb 29 14:26:29 CET 2008


Author: cwittich
Date: Fri Feb 29 16:26:28 2008
New Revision: 32533

URL: http://svn.reactos.org/svn/reactos?rev=32533&view=rev
Log:
fix some more memory leaks

Modified:
    trunk/reactos/tools/rbuild/cdfile.cpp
    trunk/reactos/tools/rbuild/compilationunit.cpp
    trunk/reactos/tools/rbuild/include.cpp
    trunk/reactos/tools/rbuild/installfile.cpp
    trunk/reactos/tools/rbuild/module.cpp
    trunk/reactos/tools/rbuild/rbuild.h

Modified: trunk/reactos/tools/rbuild/cdfile.cpp
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/rbuild/cdfile.cpp?rev=32533&r1=32532&r2=32533&view=diff
==============================================================================
--- trunk/reactos/tools/rbuild/cdfile.cpp (original)
+++ trunk/reactos/tools/rbuild/cdfile.cpp Fri Feb 29 16:26:28 2008
@@ -34,6 +34,14 @@
 		return path;
 }
 
+CDFile::~CDFile ()
+{
+	if ( source )
+		delete target;
+	if ( source )
+		delete target;
+}
+
 CDFile::CDFile ( const Project& project,
                  const XMLElement& cdfileNode,
                  const string& path )

Modified: trunk/reactos/tools/rbuild/compilationunit.cpp
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/rbuild/compilationunit.cpp?rev=32533&r1=32532&r2=32533&view=diff
==============================================================================
--- trunk/reactos/tools/rbuild/compilationunit.cpp (original)
+++ trunk/reactos/tools/rbuild/compilationunit.cpp Fri Feb 29 16:26:28 2008
@@ -58,6 +58,9 @@
 	size_t i;
 	for ( i = 0; i < files.size (); i++ )
 		delete files[i];
+
+	if ( default_name )
+		delete default_name;
 }
 
 void

Modified: trunk/reactos/tools/rbuild/include.cpp
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/rbuild/include.cpp?rev=32533&r1=32532&r2=32533&view=diff
==============================================================================
--- trunk/reactos/tools/rbuild/include.cpp (original)
+++ trunk/reactos/tools/rbuild/include.cpp Fri Feb 29 16:26:28 2008
@@ -53,6 +53,8 @@
 
 Include::~Include()
 {
+	if ( directory )
+		delete directory;
 }
 
 void

Modified: trunk/reactos/tools/rbuild/installfile.cpp
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/rbuild/installfile.cpp?rev=32533&r1=32532&r2=32533&view=diff
==============================================================================
--- trunk/reactos/tools/rbuild/installfile.cpp (original)
+++ trunk/reactos/tools/rbuild/installfile.cpp Fri Feb 29 16:26:28 2008
@@ -21,6 +21,15 @@
 #include "rbuild.h"
 
 using std::string;
+
+InstallFile::~InstallFile()
+{
+	if ( source )
+		delete source;
+
+	if ( target )
+		delete source;
+}
 
 InstallFile::InstallFile ( const Project& project,
                            const XMLElement& installfileNode,

Modified: trunk/reactos/tools/rbuild/module.cpp
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/rbuild/module.cpp?rev=32533&r1=32532&r2=32533&view=diff
==============================================================================
--- trunk/reactos/tools/rbuild/module.cpp (original)
+++ trunk/reactos/tools/rbuild/module.cpp Fri Feb 29 16:26:28 2008
@@ -490,6 +490,12 @@
 		delete install;
 	if ( output )
 		delete output;
+	if ( metadata )
+		delete metadata;
+	if ( bootstrap )
+		delete bootstrap;
+	if ( importLibrary )
+		delete importLibrary;
 }
 
 void
@@ -1590,6 +1596,13 @@
 }
 
 
+ImportLibrary::~ImportLibrary ()
+{
+	if ( source )
+		delete source;
+}
+
+
 ImportLibrary::ImportLibrary ( const Project& project,
                                const XMLElement& node,
                                const Module& module )

Modified: trunk/reactos/tools/rbuild/rbuild.h
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/tools/rbuild/rbuild.h?rev=32533&r1=32532&r2=32533&view=diff
==============================================================================
--- trunk/reactos/tools/rbuild/rbuild.h (original)
+++ trunk/reactos/tools/rbuild/rbuild.h Fri Feb 29 16:26:28 2008
@@ -569,6 +569,7 @@
 	ImportLibrary ( const Project& project,
 	                const XMLElement& node,
 	                const Module& module );
+	~ImportLibrary ();
 };
 
 
@@ -816,6 +817,7 @@
 	CDFile ( const Project& project,
 	         const XMLElement& bootstrapNode,
 	         const std::string& path );
+	~CDFile ();
 private:
 	static std::string ReplaceVariable ( const std::string& name,
 	                                     const std::string& value,
@@ -832,6 +834,7 @@
 	InstallFile ( const Project& project,
 	              const XMLElement& bootstrapNode,
 	              const std::string& path );
+	~InstallFile ();
 };
 
 




More information about the Ros-diffs mailing list