Date: Thu, 11 Jul 2002 22:33:34 -0700 (PDT) From: "Crist J. Clark" <cjc@FreeBSD.org> To: cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org Subject: cvs commit: src/usr.bin/chpass chpass.c Message-ID: <200207120533.g6C5XYrA017954@freefall.freebsd.org>
next in thread | raw e-mail | index | archive | help
cjc 2002/07/11 22:33:34 PDT
Modified files: (Branch: RELENG_4)
usr.bin/chpass chpass.c
Log:
In preventing races in making changes to the password file when the
locking mechanism was relaxed, it was assumed the '-a' option was only
being used to add new entries. The result was '-a' option could not be
used to overwrite an existing entry, but this is a legitimate use. Now
when doing an overwrite, check if the user exists and load the old
entry for later verification. If the user does not exist, there is no
reason to check later.
Note that I this is only fixed in 4-STABLE. When des overhauled the
passwd related commands and functions in 5-CURRENT (thanx!), the logic
of the file was changed and it doesn't have _this_ bug... but it may
have some others with respect to the '-a' option.
PR: bin/39919
Revision Changes Path
1.16.2.3 +7 -2 src/usr.bin/chpass/chpass.c
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200207120533.g6C5XYrA017954>