diff options
| author | wpaul <wpaul@FreeBSD.org> | 1997-03-29 20:29:12 +0000 |
|---|---|---|
| committer | wpaul <wpaul@FreeBSD.org> | 1997-03-29 20:29:12 +0000 |
| commit | b2e5051d34b86bf4192d8670cc9e7c47bfdfae33 (patch) | |
| tree | 6becd82bdcf498d22c3509fe2ce65da9b7fbc5ab /usr.bin/chpass | |
| parent | 3d2a63277b84042b3b3e1caab98539fa6815ca04 (diff) | |
| download | FreeBSD-src-b2e5051d34b86bf4192d8670cc9e7c47bfdfae33.zip FreeBSD-src-b2e5051d34b86bf4192d8670cc9e7c47bfdfae33.tar.gz | |
Fix for PR #3141: check for NULL before strdup()ing pw->pw_class.
(I'm not sure why this happens, though I suspect it may be because
the server is configured with only passwd maps instead of both passwd
and master.passwd maps. This is allowed, but I think in this case
pw_class is left NULL, hence the problem.)
Also applied similar patch to chpass/pw_yp.c just for paranoia's sake.
Diffstat (limited to 'usr.bin/chpass')
| -rw-r--r-- | usr.bin/chpass/pw_yp.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/usr.bin/chpass/pw_yp.c b/usr.bin/chpass/pw_yp.c index 514bf7c..0080e85 100644 --- a/usr.bin/chpass/pw_yp.c +++ b/usr.bin/chpass/pw_yp.c @@ -35,7 +35,7 @@ * Center for Telecommunications Research * Columbia University, New York City * - * $Id$ + * $Id: pw_yp.c,v 1.9 1997/02/22 19:54:26 peter Exp $ */ #ifdef YP @@ -441,7 +441,8 @@ void yp_submit(pw) master_yppasswd.newpw.pw_gecos = strdup(pw->pw_gecos); master_yppasswd.newpw.pw_dir = strdup(pw->pw_dir); master_yppasswd.newpw.pw_shell = strdup(pw->pw_shell); - master_yppasswd.newpw.pw_class = strdup(pw->pw_class); + master_yppasswd.newpw.pw_class = pw->pw_class != NULL ? + strdup(pw->pw_class) : ""; master_yppasswd.oldpass = ""; /* not really needed */ master_yppasswd.domain = yp_domain; } else { |
