diff options
| author |   Baptiste Daroussin <bapt@FreeBSD.org> | 2014-11-12 22:27:53 +0000 |
|---|---|---|
| committer | Baptiste Daroussin <bapt@FreeBSD.org> | 2014-11-12 22:27:53 +0000 |
| commit | bcd9ef6b1872f72a7932f3a2554348ec81e7e42d (patch) | |
| tree | c9fb97c7dc6e9f7ab2ae2901274bca2100194ecf /pw | |
| parent | cf28a17818058981489f2d7b9146b87ffd2b5049 (diff) | |
| download | pw-darwin-bcd9ef6b1872f72a7932f3a2554348ec81e7e42d.tar.gz pw-darwin-bcd9ef6b1872f72a7932f3a2554348ec81e7e42d.tar.zst pw-darwin-bcd9ef6b1872f72a7932f3a2554348ec81e7e42d.zip | |
Fix wrong message when using pw -V with a non existent directory
Add a regression test about it
PR: 194971
Submitted by: Freddy DISSAUX <bugzilla@dsx.bsdsx.fr>
Diffstat (limited to 'pw')
| -rw-r--r-- | pw/pw.c | 8 | ||||
| -rw-r--r-- | pw/tests/Makefile | 2 | ||||
| -rwxr-xr-x | pw/tests/pw_etcdir.sh | 18 |
3 files changed, 27 insertions, 1 deletions
@@ -98,6 +98,7 @@ main(int argc, char *argv[]) int which = -1; char *config = NULL; struct userconf *cnf; + struct stat st; static const char *opts[W_NUM][M_NUM] = { @@ -143,6 +144,13 @@ main(int argc, char *argv[]) if (argv[1][1] == 'V') { optarg = &argv[1][2]; if (*optarg == '\0') { + if (stat(argv[2], &st) != 0) + errx(EX_OSFILE, \ + "no such directory `%s'", + argv[2]); + if (!S_ISDIR(st.st_mode)) + errx(EX_OSFILE, "`%s' not a " + "directory", argv[2]); optarg = argv[2]; ++argv; --argc; diff --git a/pw/tests/Makefile b/pw/tests/Makefile index fabf624..0391d1f 100644 --- a/pw/tests/Makefile +++ b/pw/tests/Makefile @@ -5,7 +5,7 @@ TESTSRC= ${.CURDIR}/../../../contrib/netbsd-tests/usr.sbin/useradd TESTSDIR= ${TESTSBASE}/usr.sbin/pw -ATF_TESTS_SH= pw_delete pw_modify +ATF_TESTS_SH= pw_delete pw_modify pw_etcdir TEST_METADATA.pw_delete+= required_user="root" TEST_METADATA.pw_modify+= required_user="root" diff --git a/pw/tests/pw_etcdir.sh b/pw/tests/pw_etcdir.sh new file mode 100755 index 0000000..edb97fe --- /dev/null +++ b/pw/tests/pw_etcdir.sh @@ -0,0 +1,18 @@ +# $FreeBSD$ + +# When provide -V dir, dir must exists +atf_test_case etcdir_must_exists +etcdir_must_exists_head() { + atf_set "descr" "When provide -V dir, dir must exists" +} + +etcdir_must_exists_body() { + local fakedir="/this_directory_does_not_exists" + atf_check -e inline:"pw: no such directory \`$fakedir'\n" \ + -s exit:72 -x pw -V ${fakedir} usershow root +} + +atf_init_test_cases() { + atf_add_test_case etcdir_must_exists +} + |