Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 16 Apr 2001 22:49:07 -0700
From:      Dima Dorfman <dima@unixfreak.org>
To:        current@freebsd.org
Cc:        kris@freebsd.org
Subject:   Cosmetic bug in pw_util.c (with patch)
Message-ID:  <20010417054907.CA7C83E09@bazooka.unixfreak.org>

next in thread | raw e-mail | index | archive | help
Revision 1.18 of pw_util.c (fix for security advisory "chpass" 00:58)
introduces a cosmetic bug which manifests itself as such:

	> passwd
	Changing local password for dima.
	Old password:
	passwd: (null): Permission denied
               ^^^^^^^^^
	passwd: /etc/master.passwd: unchanged

Only machines *not* using NIS will display this, since passwd(1) acts
differently when changing an NIS password.  Trivial patch to fix this
is attached below.

Thanks,

					Dima Dorfman
					dima@unixfreak.org


Index: pw_util.c
===================================================================
RCS file: /st/src/FreeBSD/src/usr.sbin/vipw/pw_util.c,v
retrieving revision 1.18
diff -u -r1.18 pw_util.c
--- pw_util.c	2000/07/12 00:49:40	1.18
+++ pw_util.c	2001/04/17 05:41:38
@@ -249,8 +249,12 @@
 #ifdef YP
 	extern int _use_yp;
 #endif /* YP */
-	if (err)
-		warn("%s", name);
+	if (err) {
+		if (name != NULL)
+			warn("%s", name);
+		else
+			warn(NULL);
+	}
 #ifdef YP
 	if (_use_yp)
 		warnx("NIS information unchanged");

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-current" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20010417054907.CA7C83E09>