summaryrefslogtreecommitdiffstats
path: root/pw
diff options
context:
space:
mode:
authorBaptiste Daroussin <bapt@FreeBSD.org>2015-08-01 12:18:48 +0000
committerBaptiste Daroussin <bapt@FreeBSD.org>2015-08-01 12:18:48 +0000
commit5bb254a38fa9e233bcdb24956e1f8ccf2b19182b (patch)
treec8cd68e3a1ebde2d6a577a92162935d0a778c39b /pw
parent99c28d09467b167abaa8b517679f480b4e21d40b (diff)
downloadpw-darwin-5bb254a38fa9e233bcdb24956e1f8ccf2b19182b.tar.gz
pw-darwin-5bb254a38fa9e233bcdb24956e1f8ccf2b19182b.tar.zst
pw-darwin-5bb254a38fa9e233bcdb24956e1f8ccf2b19182b.zip
Partial revert of r286152
More work needed on the cli validation
Diffstat (limited to 'pw')
-rw-r--r--pw/pw.c10
-rw-r--r--pw/pw_conf.c2
-rw-r--r--pw/pw_user.c8
-rw-r--r--pw/pwupd.h2
4 files changed, 6 insertions, 16 deletions
diff --git a/pw/pw.c b/pw/pw.c
index bca6715..88c83db 100644
--- a/pw/pw.c
+++ b/pw/pw.c
@@ -262,11 +262,6 @@ main(int argc, char *argv[])
case 'c':
conf.gecos = pw_checkname(optarg, 1);
break;
- case 'e':
- conf.expire_days = strtonum(optarg, 0, INT_MAX, &errstr);
- if (errstr)
- errx(EX_USAGE, "Invalid expired days: %s", optarg);
- break;
case 'g':
if (which == 0) { /* for user* */
addarg(&arglist, 'g', optarg);
@@ -326,11 +321,6 @@ main(int argc, char *argv[])
case 'o':
conf.checkduplicate = false;
break;
- case 'p':
- conf.password_days = strtonum(optarg, 0, INT_MAX, &errstr);
- if (errstr)
- errx(EX_USAGE, "Invalid password days: %s", optarg);
- break;
case 'q':
conf.quiet = true;
break;
diff --git a/pw/pw_conf.c b/pw/pw_conf.c
index b723c31..8ba8c07 100644
--- a/pw/pw_conf.c
+++ b/pw/pw_conf.c
@@ -367,6 +367,8 @@ read_userconfig(char const * file)
warnx("Invalid expire days:"
" '%s'; ignoring", q);
}
+ if ((q = unquote(q)) != NULL && isdigit(*q))
+ config.expire_days = atoi(q);
break;
case _UC_PASSWORD:
if ((q = unquote(q)) != NULL) {
diff --git a/pw/pw_user.c b/pw/pw_user.c
index 6e07f1f..eca8235 100644
--- a/pw/pw_user.c
+++ b/pw/pw_user.c
@@ -418,14 +418,14 @@ pw_user(int mode, char *name, long id, struct cargs * args)
errx(EX_OSFILE, "root home `%s' is not a directory", cnf->home);
}
- if (conf.expire_days > 0)
- cnf->expire_days = conf.expire_days;
+ if ((arg = getarg(args, 'e')) != NULL)
+ cnf->expire_days = atoi(arg->val);
if ((arg = getarg(args, 'y')) != NULL)
cnf->nispasswd = arg->val;
- if (conf.password_days > 0)
- cnf->password_days = conf.password_days;
+ if ((arg = getarg(args, 'p')) != NULL && arg->val)
+ cnf->password_days = atoi(arg->val);
if ((arg = getarg(args, 'g')) != NULL) {
if (!*(p = arg->val)) /* Handle empty group list specially */
diff --git a/pw/pwupd.h b/pw/pwupd.h
index 9685bea..054c5a5 100644
--- a/pw/pwupd.h
+++ b/pw/pwupd.h
@@ -86,8 +86,6 @@ struct pwconf {
char *newname;
char *config;
char *gecos;
- int expire_days;
- int password_days;
int fd;
int rootfd;
int which;