Correctly detect the case where a password entry was changed while we were
preparing to edit it. PR: bin/50563
This commit is contained in:
parent
a4280091c3
commit
e7d9d9217c
@ -475,8 +475,7 @@ pw_copy(int ffd, int tfd, struct passwd *pw, struct passwd *old_pw)
|
||||
*q = '\0';
|
||||
fpw = pw_scan(r, PWSCAN_MASTER);
|
||||
*q = t;
|
||||
if ((old_pw && !pw_equal(fpw, old_pw)) ||
|
||||
(!old_pw && strcmp(fpw->pw_name, pw->pw_name))) {
|
||||
if (strcmp(fpw->pw_name, pw->pw_name) != 0) {
|
||||
/* nope */
|
||||
free(fpw);
|
||||
if (write(tfd, p, q - p + 1) != q - p + 1)
|
||||
@ -484,6 +483,12 @@ pw_copy(int ffd, int tfd, struct passwd *pw, struct passwd *old_pw)
|
||||
++q;
|
||||
continue;
|
||||
}
|
||||
if (old_pw && !pw_equal(fpw, old_pw)) {
|
||||
warnx("entry inconsistent");
|
||||
free(fpw);
|
||||
errno = EINVAL; /* hack */
|
||||
goto err;
|
||||
}
|
||||
free(fpw);
|
||||
|
||||
/* it is, replace it */
|
||||
|
Loading…
Reference in New Issue
Block a user