[Selinux-commits] SF.net SVN: selinux: [2868] trunk/policycoreutils/semanage/seobject.py

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

Revision: 2868
          http://selinux.svn.sourceforge.net/selinux/?rev=2868&view=rev
Author:   ssmalley
Date:     2008-04-18 07:12:40 -0700 (Fri, 18 Apr 2008)

Log Message:
-----------
Author: Daniel J Walsh
Email: dw...@re...
Subject: semanage man page updated for booleans
Date: Tue, 08 Apr 2008 09:59:02 -0400

[sds:  some fixes to the entry, including mention of the 
--on/--off/-1/-0 options and removal of the -T option as that
doesn't apply here.]

[sds:  also add checking of return status to seobject.py for
set_active call]

Modified Paths:
--------------
    trunk/policycoreutils/semanage/seobject.py

Modified: trunk/policycoreutils/semanage/seobject.py
===================================================================
--- trunk/policycoreutils/semanage/seobject.py	2008-04-18 14:00:16 UTC (rev 2867)
+++ trunk/policycoreutils/semanage/seobject.py	2008-04-18 14:12:40 UTC (rev 2868)
@@ -1377,7 +1377,9 @@
 		if rc < 0:
 			raise ValueError(_("Could not start semanage transaction"))
 
-                rc = semanage_bool_set_active(self.sh, k, b)
+		rc = semanage_bool_set_active(self.sh, k, b)
+		if rc < 0:
+			raise ValueError(_("Could not set active value of boolean %s") % name)
 		rc = semanage_bool_modify_local(self.sh, k, b)
 		if rc < 0:
 			raise ValueError(_("Could not modify boolean %s") % name)


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