diff options
author | Baptiste Daroussin <bapt@FreeBSD.org> | 2017-01-22 20:03:18 +0000 |
---|---|---|
committer | Baptiste Daroussin <bapt@FreeBSD.org> | 2017-01-22 20:03:18 +0000 |
commit | e18f3da5e35c62cbf864d1717e6ae25bb6014dba (patch) | |
tree | e5e7fc139b4d4232f3befcd932c4026a9324f619 /pw/pw_user.c | |
parent | f856c3ca12b9891190c15f332513b70a0a2044b5 (diff) | |
download | pw-darwin-e18f3da5e35c62cbf864d1717e6ae25bb6014dba.tar.gz pw-darwin-e18f3da5e35c62cbf864d1717e6ae25bb6014dba.tar.zst pw-darwin-e18f3da5e35c62cbf864d1717e6ae25bb6014dba.zip |
Really restore the old behaviour for pw usermod -m
It again reinstall missing skel files without overwriting changed one
Add a regression test about it
Reported by: ae
MFC after: 3 days
Diffstat (limited to 'pw/pw_user.c')
-rw-r--r-- | pw/pw_user.c | 12 |
1 files changed, 3 insertions, 9 deletions
diff --git a/pw/pw_user.c b/pw/pw_user.c index 6e991a8..8cffac9 100644 --- a/pw/pw_user.c +++ b/pw/pw_user.c @@ -1493,7 +1493,7 @@ pw_user_mod(int argc, char **argv, char *arg1) intmax_t id = -1; int ch, fd = -1; size_t i, j; - bool quiet, createhome, pretty, dryrun, nis, edited, docreatehome; + bool quiet, createhome, pretty, dryrun, nis, edited; bool precrypted; mode_t homemode = 0; time_t expire_days, password_days, now; @@ -1503,7 +1503,7 @@ pw_user_mod(int argc, char **argv, char *arg1) passwd = NULL; class = nispasswd = NULL; quiet = createhome = pretty = dryrun = nis = precrypted = false; - edited = docreatehome = false; + edited = false; if (arg1 != NULL) { if (arg1[strspn(arg1, "0123456789")] == '\0') @@ -1697,10 +1697,6 @@ pw_user_mod(int argc, char **argv, char *arg1) edited = true; } - if (createhome && fstatat(conf.rootfd, pwd->pw_dir, &st, 0) == -1) { - docreatehome = true; - } - if (homedir && strcmp(pwd->pw_dir, homedir) != 0) { pwd->pw_dir = homedir; edited = true; @@ -1708,8 +1704,6 @@ pw_user_mod(int argc, char **argv, char *arg1) if (!createhome) warnx("WARNING: home `%s' does not exist", pwd->pw_dir); - else - docreatehome = true; } else if (!S_ISDIR(st.st_mode)) { warnx("WARNING: home `%s' is not a directory", pwd->pw_dir); @@ -1801,7 +1795,7 @@ pw_user_mod(int argc, char **argv, char *arg1) * that this also `works' for editing users if -m is used, but * existing files will *not* be overwritten. */ - if (PWALTDIR() != PWF_ALT && docreatehome && pwd->pw_dir && + if (PWALTDIR() != PWF_ALT && createhome && pwd->pw_dir && *pwd->pw_dir == '/' && pwd->pw_dir[1]) { if (!skel) skel = cnf->dotdir; |