[Pkg-shadow-commits] r3222 - in upstream/trunk: . lib

[Nicolas FRANÇOIS]

Author: nekral-guest
Date: 2010-08-20 20:35:52 +0000 (Fri, 20 Aug 2010)
New Revision: 3222

Modified:
   upstream/trunk/ChangeLog
   upstream/trunk/lib/groupio.c
Log:
	* lib/groupio.c: Avoid implicit conversion from signed to unsigned.


Modified: upstream/trunk/ChangeLog
===================================================================
--- upstream/trunk/ChangeLog	2010-08-20 20:34:44 UTC (rev 3221)
+++ upstream/trunk/ChangeLog	2010-08-20 20:35:52 UTC (rev 3222)
@@ -1,5 +1,9 @@
 2010-08-20  Nicolas François  <nicolas.francois at centraliens.net>
 
+	* lib/groupio.c: Avoid implicit conversion from signed to unsigned.
+
+2010-08-20  Nicolas François  <nicolas.francois at centraliens.net>
+
 	* lib/commonio.c: Make sure there are no NULL pointer dereference.
 
 2010-08-20  Nicolas François  <nicolas.francois at centraliens.net>

Modified: upstream/trunk/lib/groupio.c
===================================================================
--- upstream/trunk/lib/groupio.c	2010-08-20 20:34:44 UTC (rev 3221)
+++ upstream/trunk/lib/groupio.c	2010-08-20 20:35:52 UTC (rev 3222)
@@ -382,16 +382,19 @@
 		struct commonio_entry *new;
 		struct group *new_gptr;
 		unsigned int members = 0;
-		int i;
+		unsigned int i;
 
 		/* Check if this group must be split */
-		if (!gr->changed)
+		if (!gr->changed) {
 			continue;
-		if (NULL == gptr)
+		}
+		if (NULL == gptr) {
 			continue;
+		}
 		for (members = 0; NULL != gptr->gr_mem[members]; members++);
-		if (members <= max_members)
+		if (members <= max_members) {
 			continue;
+		}
 
 		new = (struct commonio_entry *) malloc (sizeof *new);
 		if (NULL == new) {