PERFORCE change 113344 for review

Todd Miller millert at FreeBSD.org
Mon Jan 22 16:04:03 UTC 2007


http://perforce.freebsd.org/chv.cgi?CH=113344

Change 113344 by millert at millert_macbook on 2007/01/22 16:02:51

	sebsd_label_destroy() may be passed a NULL pointer if one
	of the per-policy label init routines fails.  In this case
	the framework has to clean up an allocations that did succeed
	but it is not smart enough to only clean up the ones up
	until the point of failure.

Affected files ...

.. //depot/projects/trustedbsd/sedarwin8/policies/sedarwin/sedarwin/sebsd.c#69 edit

Differences ...

==== //depot/projects/trustedbsd/sedarwin8/policies/sedarwin/sedarwin/sebsd.c#69 (text+ko) ====

@@ -550,7 +550,8 @@
 sebsd_label_destroy(struct label *label)
 {
 
-	mac_zfree(sebsd_label_zone, SLOT(label));
+	if (SLOT(label)  != NULL)
+		mac_zfree(sebsd_label_zone, SLOT(label));
 	SLOT(label) = NULL;
 }
 


More information about the trustedbsd-cvs mailing list