[Selinux-commits] SF.net SVN: selinux: [2887] trunk/libsepol/src/expand.c

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

Revision: 2887
          http://selinux.svn.sourceforge.net/selinux/?rev=2887&view=rev
Author:   ssmalley
Date:     2008-05-27 13:06:24 -0700 (Tue, 27 May 2008)

Log Message:
-----------
Remove nonsensical tests of !(&...\.node) - they are always false.

Modified Paths:
--------------
    trunk/libsepol/src/expand.c

Modified: trunk/libsepol/src/expand.c
===================================================================
--- trunk/libsepol/src/expand.c	2008-05-27 20:03:37 UTC (rev 2886)
+++ trunk/libsepol/src/expand.c	2008-05-27 20:06:24 UTC (rev 2887)
@@ -544,9 +544,6 @@
 	role_datum_t *role = (role_datum_t *) datum;
 	expand_state_t *state = (expand_state_t *) data;
 
-	if (!(&role->dominates.node)) 
-		return 0;
-
 	if (map_ebitmap(&role->dominates, &mapped_roles, state->rolemap))
 		return -1;
 
@@ -619,11 +616,6 @@
 		}
 	}
 
-	if (!(&new_role->dominates.node)) {
-		ebitmap_init(&new_role->dominates);
-	}
-
-
 	/* The dominates bitmap is going to be wrong for the moment, 
  	 * we'll come back later and remap them, after we are sure all 
  	 * the roles have been added */
@@ -642,10 +634,6 @@
 		return -1;
 	}
 
-	if (!(&new_role->types.types.node)) {
-		ebitmap_init(&new_role->types.types);
-	}
-
 	if (ebitmap_union(&new_role->types.types, &tmp_union_types)) {
 		ERR(state->handle, "Out of memory!");
 		ebitmap_destroy(&tmp_union_types);
@@ -820,10 +808,6 @@
 		return -1;
 	}
 
-	if (!(&new_user->roles.roles.node)) {
-		ebitmap_init(&new_user->roles.roles);
-	}
-
 	if (ebitmap_union(&new_user->roles.roles, &tmp_union)) {
 		ERR(state->handle, "Out of memory!");
 		ebitmap_destroy(&tmp_union);


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