[Selinux-commits] SF.net SVN: selinux: [2719] trunk/checkpolicy

[<mil...@us...>]

Revision: 2719
          http://selinux.svn.sourceforge.net/selinux/?rev=2719&view=rev
Author:   millertc
Date:     2008-01-08 08:12:09 -0800 (Tue, 08 Jan 2008)

Log Message:
-----------
Subject: quiet checkpolicy warnings

Fix shadowed variable in dispol/dismod
Use bison instead of yacc to avoid an unused label warning.

Signed-off-by: Todd C. Miller <tm...@tr...>
Acked-by:  Stephen Smalley <sd...@ty...>

Modified Paths:
--------------
    trunk/checkpolicy/Makefile
    trunk/checkpolicy/test/dismod.c
    trunk/checkpolicy/test/dispol.c

Modified: trunk/checkpolicy/Makefile
===================================================================
--- trunk/checkpolicy/Makefile	2008-01-03 15:24:01 UTC (rev 2718)
+++ trunk/checkpolicy/Makefile	2008-01-08 16:12:09 UTC (rev 2719)
@@ -8,6 +8,8 @@
 INCLUDEDIR ?= $(PREFIX)/include
 TARGETS = checkpolicy checkmodule
 
+YACC = bison -y
+
 CFLAGS ?= -g -Wall -O2 -pipe -fno-strict-aliasing
 
 override CFLAGS += -I. -I${INCLUDEDIR}

Modified: trunk/checkpolicy/test/dismod.c
===================================================================
--- trunk/checkpolicy/test/dismod.c	2008-01-03 15:24:01 UTC (rev 2718)
+++ trunk/checkpolicy/test/dismod.c	2008-01-08 16:12:09 UTC (rev 2719)
@@ -666,13 +666,13 @@
 	return 0;
 }
 
-int display_handle_unknown(policydb_t * policydb, FILE * out_fp)
+int display_handle_unknown(policydb_t * p, FILE * out_fp)
 {
-	if (policydb->handle_unknown == ALLOW_UNKNOWN)
+	if (p->handle_unknown == ALLOW_UNKNOWN)
 		fprintf(out_fp, "Allow unknown classes and perms\n");
-	else if (policydb->handle_unknown == DENY_UNKNOWN)
+	else if (p->handle_unknown == DENY_UNKNOWN)
 		fprintf(out_fp, "Deny unknown classes and perms\n");
-	else if (policydb->handle_unknown == REJECT_UNKNOWN)
+	else if (p->handle_unknown == REJECT_UNKNOWN)
 		fprintf(out_fp, "Reject unknown classes and perms\n");
 	return 0;
 }

Modified: trunk/checkpolicy/test/dispol.c
===================================================================
--- trunk/checkpolicy/test/dispol.c	2008-01-03 15:24:01 UTC (rev 2718)
+++ trunk/checkpolicy/test/dispol.c	2008-01-08 16:12:09 UTC (rev 2719)
@@ -274,13 +274,13 @@
 	return 1;
 }
 
-int display_handle_unknown(policydb_t * policydb, FILE * out_fp)
+int display_handle_unknown(policydb_t * p, FILE * out_fp)
 {
-	if (policydb->handle_unknown == ALLOW_UNKNOWN)
+	if (p->handle_unknown == ALLOW_UNKNOWN)
 		fprintf(out_fp, "Allow unknown classes and permisions\n");
-	else if (policydb->handle_unknown == DENY_UNKNOWN)
+	else if (p->handle_unknown == DENY_UNKNOWN)
 		fprintf(out_fp, "Deny unknown classes and permisions\n");
-	else if (policydb->handle_unknown == REJECT_UNKNOWN)
+	else if (p->handle_unknown == REJECT_UNKNOWN)
 		fprintf(out_fp, "Reject unknown classes and permisions\n");
 	return 0;
 }


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