From 4205d2d71c2eaf3380216ab31b2e0619f8001539 Mon Sep 17 00:00:00 2001 From: David Nugent Date: Sun, 18 Jun 2000 02:18:15 +0000 Subject: [PATCH] Fix a long-standing bug in the rename case previously masked by another bug just fixed in .db file updating. --- pw/pw_user.c | 12 ++++++++++-- 1 file 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); -- 2.47.1