diff options
author | Ian Lepore <ian@FreeBSD.org> | 2018-07-27 15:17:24 +0000 |
---|---|---|
committer | Ian Lepore <ian@FreeBSD.org> | 2018-07-27 15:17:24 +0000 |
commit | bcfb085a8b157ca9c5e5943fd81ed045eb0e7ccc (patch) | |
tree | 2b2c70062aacb1fb34953f1e486f50a8cd9c8daa /pw/pw_vpw.c | |
parent | 59e370118429dc4d56d94ff61ef1dc26dd1d774e (diff) | |
download | pw-darwin-bcfb085a8b157ca9c5e5943fd81ed045eb0e7ccc.tar.gz pw-darwin-bcfb085a8b157ca9c5e5943fd81ed045eb0e7ccc.tar.zst pw-darwin-bcfb085a8b157ca9c5e5943fd81ed045eb0e7ccc.zip |
Remove some code that's no longer needed because it's now part of pw_scan(3).
It was also leading to segfaults; pw can be NULL when control reaches these
lines now, because of the way my previous change restructured the loops.
Reported by: lwhsu@
Diffstat (limited to 'pw/pw_vpw.c')
-rw-r--r-- | pw/pw_vpw.c | 12 |
1 files changed, 0 insertions, 12 deletions
diff --git a/pw/pw_vpw.c b/pw/pw_vpw.c index 4517a74..192d588 100644 --- a/pw/pw_vpw.c +++ b/pw/pw_vpw.c @@ -113,18 +113,6 @@ vnextpwent(char const *nam, uid_t uid, int doclose) } free(line); - /* - * If we read the non-master passwd, some fields may not have been - * populated. Clean them up so that the output looks the same as that - * generated using getpwnam() which also inits them to these values. - */ - if (!(pw->pw_fields & _PWF_CLASS)) - pw->pw_class = ""; - if (!(pw->pw_fields & _PWF_CHANGE)) - pw->pw_change = 0; - if (!(pw->pw_fields & _PWF_EXPIRE)) - pw->pw_expire = 0; - return (pw); } |