[Pkg-shadow-commits] r3392 - in upstream/trunk: . src

[Nicolas FRANÇOIS]

Author: nekral-guest
Date: 2011-07-14 13:29:48 +0000 (Thu, 14 Jul 2011)
New Revision: 3392

Modified:
   upstream/trunk/ChangeLog
   upstream/trunk/src/usermod.c
Log:
	* src/usermod.c (new_pwent): Document that pw_locate will not fail
	because getpwnam returned successfully.

Modified: upstream/trunk/ChangeLog
===================================================================
--- upstream/trunk/ChangeLog	2011-07-14 13:29:43 UTC (rev 3391)
+++ upstream/trunk/ChangeLog	2011-07-14 13:29:48 UTC (rev 3392)
@@ -24,6 +24,8 @@
 	* man/usermod.8.xml: Document behavior of an empty EXPIRE_DATE.
 	* man/usermod.8.xml: Document that the mail spool might have to be
 	renamed (as for the homedir)
+	* src/usermod.c (new_pwent): Document that pw_locate will not fail
+	because getpwnam returned successfully.
 
 2011-07-08  Nicolas François  <nicolas.francois at centraliens.net>
 

Modified: upstream/trunk/src/usermod.c
===================================================================
--- upstream/trunk/src/usermod.c	2011-07-14 13:29:43 UTC (rev 3391)
+++ upstream/trunk/src/usermod.c	2011-07-14 13:29:48 UTC (rev 3392)
@@ -402,6 +402,10 @@
 {
 	if (lflg) {
 		if (pw_locate (user_newname) != NULL) {
+			/* This should never happen.
+			 * It was already checked that the user doesn't
+			 * exist on the system.
+			 */
 			fprintf (stderr,
 			         _("%s: user '%s' already exists in %s\n"),
 			         Prog, user_newname, pw_dbname ());