[Selinux-commits] SF.net SVN: selinux: [2780] trunk/libsemanage/src/direct_api.c

[<ssm...@us...>]

Revision: 2780
          http://selinux.svn.sourceforge.net/selinux/?rev=2780&view=rev
Author:   ssmalley
Date:     2008-02-04 07:41:21 -0800 (Mon, 04 Feb 2008)

Log Message:
-----------
Author: Stephen Smalley
Email: sd...@ty...
Subject: libsemanage: free base immediately after expand
Date: Fri, 01 Feb 2008 09:11:06 -0500

Drop the base module immediately after expanding, so that the memory can
be reused for the remainder of the transaction.

Signed-off-by:  Stephen Smalley <sd...@ty...>
Acked-By: Joshua Brindle <me...@ma...>
Acked-By: Todd C. Miller <tm...@tr...>

Modified Paths:
--------------
    trunk/libsemanage/src/direct_api.c

Modified: trunk/libsemanage/src/direct_api.c
===================================================================
--- trunk/libsemanage/src/direct_api.c	2008-02-04 15:26:35 UTC (rev 2779)
+++ trunk/libsemanage/src/direct_api.c	2008-02-04 15:41:21 UTC (rev 2780)
@@ -642,6 +642,9 @@
 		retval = semanage_expand_sandbox(sh, base, &out);
 		if (retval < 0)
 			goto cleanup;
+	
+		sepol_module_package_free(base);
+		base = NULL;
 
 		dbase_policydb_attach((dbase_policydb_t *) pusers_base->dbase,
 				      out);
@@ -718,7 +721,6 @@
 	dbase_policydb_detach((dbase_policydb_t *) pbools->dbase);
 
 	free(mod_filenames);
-	sepol_module_package_free(base);
 	sepol_policydb_free(out);
 	semanage_release_trans_lock(sh);
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.