diff options
author | Baptiste Daroussin <bapt@FreeBSD.org> | 2015-05-09 22:43:44 +0000 |
---|---|---|
committer | Baptiste Daroussin <bapt@FreeBSD.org> | 2015-05-09 22:43:44 +0000 |
commit | 020fd2ca7fd6eb436c750af2f59d5d60c9b9d221 (patch) | |
tree | cb5ea60c3edd598d5f4e1450dc93d2630d5216ab /pw | |
parent | 47edccabad63e44c2abad61ce4d5c1fd47b94c74 (diff) | |
download | pw-darwin-020fd2ca7fd6eb436c750af2f59d5d60c9b9d221.tar.gz pw-darwin-020fd2ca7fd6eb436c750af2f59d5d60c9b9d221.tar.zst pw-darwin-020fd2ca7fd6eb436c750af2f59d5d60c9b9d221.zip |
Use snprintf(3) instead of sprintf(3)
Remove useless "else"
Diffstat (limited to 'pw')
-rw-r--r-- | pw/pw_user.c | 15 |
1 files changed, 7 insertions, 8 deletions
diff --git a/pw/pw_user.c b/pw/pw_user.c index bdb3cd6..d7aed71 100644 --- a/pw/pw_user.c +++ b/pw/pw_user.c @@ -1018,17 +1018,16 @@ static char * pw_homepolicy(struct userconf * cnf, struct cargs * args, char const * user) { struct carg *arg = getarg(args, 'd'); + static char home[128]; if (arg) - return arg->val; - else { - static char home[128]; + return (arg->val); - if (cnf->home == NULL || *cnf->home == '\0') - errx(EX_CONFIG, "no base home directory set"); - sprintf(home, "%s/%s", cnf->home, user); - return home; - } + if (cnf->home == NULL || *cnf->home == '\0') + errx(EX_CONFIG, "no base home directory set"); + snprintf(home, sizeof(home), "%s/%s", cnf->home, user); + + return (home); } static char * |