d/freebsd-skq
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix bug introduced in rev 1.23:

Browse Source 
pw_equal does not check crypted password field, so one cannot change
crypted password keeping other fields intact.

Approved by:	des
MCF after:	3 days
This commit is contained in:
Dmitry Morozovsky 2006-09-25 15:06:24 +00:00
parent d038d0bd66
commit 50789a2f3a
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
usr.bin/chpass/chpass.c
View File

@ -217,7 +217,12 @@ main(int argc, char *argv[])
pw_fini(); pw_fini();
if (pw == NULL) if (pw == NULL)
err(1, "edit()"); err(1, "edit()");
if (pw_equal(old_pw, pw)) /*
* pw_equal does not check for crypted passwords, so we
* should do it explicitly
*/
if (pw_equal(old_pw, pw) &&
strcmp(old_pw->pw_passwd, pw->pw_passwd) == 0)
errx(0, "user information unchanged"); errx(0, "user information unchanged");
} }