diff options
author | David Nugent <davidn@FreeBSD.org> | 2000-06-18 02:18:15 +0000 |
---|---|---|
committer | David Nugent <davidn@FreeBSD.org> | 2000-06-18 02:18:15 +0000 |
commit | f20cc02ebf048cdff1e4d22c657afa20d725797a (patch) | |
tree | 6b815ab9f4053b5776d003e75ee074f966a3dd89 /pw | |
parent | 06942a0cdf3606d24a458c573469b65ba918c210 (diff) | |
download | pw-darwin-f20cc02ebf048cdff1e4d22c657afa20d725797a.tar.gz pw-darwin-f20cc02ebf048cdff1e4d22c657afa20d725797a.tar.zst pw-darwin-f20cc02ebf048cdff1e4d22c657afa20d725797a.zip |
Fix a long-standing bug in the rename case previously masked by another bug
just fixed in .db file updating.
Diffstat (limited to 'pw')
-rw-r--r-- | pw/pw_user.c | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/pw/pw_user.c b/pw/pw_user.c index 395b69a..1a93990 100644 --- a/pw/pw_user.c +++ b/pw/pw_user.c @@ -681,8 +681,16 @@ pw_user(struct userconf * cnf, int mode, struct cargs * args) if (mode == M_ADD || getarg(args, 'G') != NULL) editgroups(pwd->pw_name, cnf->groups); - /* pwd may have been invalidated */ - if ((pwd = GETPWNAM(a_name->val)) == NULL) + /* go get a current version of pwd */ + pwd = GETPWNAM(a_name->val); + if (pwd == NULL) { + /* This will fail when we rename, so special case that */ + if (mode == M_UPDATE && (arg = getarg(args, 'l')) != NULL) { + a_name->val = arg->val; /* update new name */ + pwd = GETPWNAM(a_name->val); /* refetch renamed rec */ + } + } + if (pwd == NULL) /* can't go on without this */ errx(EX_NOUSER, "user '%s' disappeared during update", a_name->val); grp = GETGRGID(pwd->pw_gid); |