[Pkg-shadow-commits] r2477 - in upstream/trunk: . libmisc

[nekral-guest at alioth.debian.org]

Author: nekral-guest
Date: 2008-12-23 22:42:22 +0000 (Tue, 23 Dec 2008)
New Revision: 2477

Modified:
   upstream/trunk/ChangeLog
   upstream/trunk/libmisc/chkname.c
Log:
	* libmisc/chkname.c: Use a bool when possible instead of integers.
	* libmisc/chkname.c: Add brackets and parenthesis.


Modified: upstream/trunk/ChangeLog
===================================================================
--- upstream/trunk/ChangeLog	2008-12-23 22:36:44 UTC (rev 2476)
+++ upstream/trunk/ChangeLog	2008-12-23 22:42:22 UTC (rev 2477)
@@ -1,5 +1,10 @@
 2008-12-23  Nicolas François  <nicolas.francois at centraliens.net>
 
+	* libmisc/chkname.c: Use a bool when possible instead of integers.
+	* libmisc/chkname.c: Add brackets and parenthesis.
+
+2008-12-23  Nicolas François  <nicolas.francois at centraliens.net>
+
 	* man/groupadd.8.xml, configure.in, man/config.xml.in: Use the
 	real group name length limit in the documentation.
 

Modified: upstream/trunk/libmisc/chkname.c
===================================================================
--- upstream/trunk/libmisc/chkname.c	2008-12-23 22:36:44 UTC (rev 2476)
+++ upstream/trunk/libmisc/chkname.c	2008-12-23 22:42:22 UTC (rev 2477)
@@ -100,8 +100,10 @@
 	 * Arbitrary limit for group names.
 	 * HP-UX 10 limits to 16 characters
 	 */
-	if (GROUP_NAME_MAX_LENGTH && strlen (name) > GROUP_NAME_MAX_LENGTH)
+	if (   (GROUP_NAME_MAX_LENGTH > 0)
+	    && (strlen (name) > GROUP_NAME_MAX_LENGTH)) {
 		return false;
+	}
 
 	return is_valid_name (name);
 }