diff options
| author | Andreas Schneider <asn@samba.org> | 2018-12-03 11:05:46 +0100 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 2018-12-05 05:22:43 +0100 |
| commit | a92f0ccce606be12e851a4100fbb44b069c5fe87 (patch) | |
| tree | 1c3862e0a8920e559c8e78b742bbc2a06c8e0543 /source3 | |
| parent | 99695528f7453023446956d5f8f0656574e243af (diff) | |
| download | samba-a92f0ccce606be12e851a4100fbb44b069c5fe87.tar.gz | |
s3:tests: Add test for checking that root is not allowed as home dir
BUG: https://bugzilla.samba.org/show_bug.cgi?id=13699
Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Wed Dec 5 05:22:43 CET 2018 on sn-devel-144
Diffstat (limited to 'source3')
| -rwxr-xr-x | source3/script/tests/test_homes.sh | 37 |
1 files changed, 37 insertions, 0 deletions
diff --git a/source3/script/tests/test_homes.sh b/source3/script/tests/test_homes.sh index 06de0a0c301..90e84550dbc 100755 --- a/source3/script/tests/test_homes.sh +++ b/source3/script/tests/test_homes.sh @@ -88,6 +88,39 @@ EOF return 0 } +test_slashuser_home() +{ + tmpfile=$PREFIX/smbclient_homes_slashuser_commands + cat > $tmpfile <<EOF +ls +quit +EOF + + USERNAME=slashuser + + cmd='CLI_FORCE_INTERACTIVE=yes $SMBCLIENT "$@" -U$USERNAME%$PASSWORD //$SERVER/$USERNAME $CONFIGURATION < $tmpfile 2>&1' + eval echo "$cmd" + out=$(eval $cmd) + ret=$? + rm -f $tmpfile + + if [ $ret -ne 1 ] ; then + echo "$out" + echo "The server should reject connecting ret=$ret" + return 1 + fi + + echo "$out" | grep 'NT_STATUS_BAD_NETWORK_NAME' + ret=$? + if [ $ret -ne 0 ] ; then + echo "$out" + echo 'failed - should get: NT_STATUS_BAD_NETWORK_NAME.' + return 1 + fi + + return 0 +} + testit "test gooduser home" \ test_gooduser_home || \ failed=`expr $failed + 1` @@ -96,4 +129,8 @@ testit "test eviluser home reject" \ test_eviluser_home || \ failed=`expr $failed + 1` +testit "test slashuser home reject" \ + test_slashuser_home || \ + failed=`expr $failed + 1` + testok $0 $failed |