[systemd-devel] [PATCH 1/2] test: add basic seccomp tests
Ronny Chevalier
chevalier.ronny at gmail.com
Thu Feb 13 00:58:53 PST 2014
2014-02-13 3:34 GMT+01:00 Dave Reisner <d at falconindy.com>:
> On Thu, Jan 23, 2014 at 01:34:57AM +0100, Ronny Chevalier wrote:
>> ---
>> test/TEST-04-SECCOMP/Makefile | 1 +
>> test/TEST-04-SECCOMP/test-seccomp.sh | 11 ++++
>> test/TEST-04-SECCOMP/test.sh | 79 +++++++++++++++++++++++++++++
>> test/TEST-04-SECCOMP/will-fail.service | 6 +++
>> test/TEST-04-SECCOMP/will-not-fail.service | 6 +++
>> test/TEST-04-SECCOMP/will-not-fail2.service | 6 +++
>> 6 files changed, 109 insertions(+)
>> create mode 120000 test/TEST-04-SECCOMP/Makefile
>> create mode 100755 test/TEST-04-SECCOMP/test-seccomp.sh
>> create mode 100755 test/TEST-04-SECCOMP/test.sh
>> create mode 100644 test/TEST-04-SECCOMP/will-fail.service
>> create mode 100644 test/TEST-04-SECCOMP/will-not-fail.service
>> create mode 100644 test/TEST-04-SECCOMP/will-not-fail2.service
>>
>> diff --git a/test/TEST-04-SECCOMP/Makefile b/test/TEST-04-SECCOMP/Makefile
>> new file mode 120000
>> index 0000000..e9f93b1
>> --- /dev/null
>> +++ b/test/TEST-04-SECCOMP/Makefile
>> @@ -0,0 +1 @@
>> +../TEST-01-BASIC/Makefile
>> \ No newline at end of file
>> diff --git a/test/TEST-04-SECCOMP/test-seccomp.sh b/test/TEST-04-SECCOMP/test-seccomp.sh
>> new file mode 100755
>> index 0000000..fef334e
>> --- /dev/null
>> +++ b/test/TEST-04-SECCOMP/test-seccomp.sh
>> @@ -0,0 +1,11 @@
>> +#!/bin/bash -x
>> +
>> +systemctl start will-fail.service
>> +systemctl start will-not-fail.service
>> +systemctl start will-not-fail2.service
>> +systemctl is-failed will-fail.service | grep failed || exit 1
>> +systemctl is-failed will-not-fail.service | grep failed && exit 1
>> +systemctl is-failed will-not-fail2.service | grep failed && exit 1
>
> This is weird. You should be able to rely on the exit code rather than
> parsing the output, but it seems this was broken in e3e0314b.
>
Yes, this is why I did this.
>> +
>> +touch /testok
>> +exit 0
>> diff --git a/test/TEST-04-SECCOMP/test.sh b/test/TEST-04-SECCOMP/test.sh
>> new file mode 100755
>> index 0000000..c29192e
>> --- /dev/null
>> +++ b/test/TEST-04-SECCOMP/test.sh
>> @@ -0,0 +1,79 @@
>> +#!/bin/bash
>> +# -*- mode: shell-script; indent-tabs-mode: nil; sh-basic-offset: 4; -*-
>> +# ex: ts=8 sw=4 sts=4 et filetype=sh
>> +TEST_DESCRIPTION="seccomp tests"
>> +
>> +. $TEST_BASE_DIR/test-functions
>> +
>> +check_result_qemu() {
>> + ret=1
>> + mkdir -p $TESTDIR/root
>> + mount ${LOOPDEV}p1 $TESTDIR/root
>> + [[ -e $TESTDIR/root/testok ]] && ret=0
>> + [[ -f $TESTDIR/root/failed ]] && cp -a $TESTDIR/root/failed $TESTDIR
>> + cp -a $TESTDIR/root/var/log/journal $TESTDIR
>> + umount $TESTDIR/root
>> + [[ -f $TESTDIR/failed ]] && cat $TESTDIR/failed
>> + ls -l $TESTDIR/journal/*/*.journal
>> + test -s $TESTDIR/failed && ret=$(($ret+1))
>> + return $ret
>> +}
>> +
>> +test_run() {
>> + if run_qemu; then
>> + check_result_qemu || return 1
>> + else
>> + dwarn "can't run QEMU, skipping"
>> + fi
>> + if check_nspawn; then
>> + run_nspawn
>> + check_result_nspawn || return 1
>> + else
>> + dwarn "can't run systemd-nspawn, skipping"
>> + fi
>> + return 0
>> +}
>> +
>> +test_setup() {
>> + create_empty_image
>> + mkdir -p $TESTDIR/root
>> + mount ${LOOPDEV}p1 $TESTDIR/root
>> +
>> + # Create what will eventually be our root filesystem onto an overlay
>> + (
>> + LOG_LEVEL=5
>> + eval $(udevadm info --export --query=env --name=${LOOPDEV}p2)
>> +
>> + setup_basic_environment
>> +
>> + # setup the testsuite service
>> + cat >$initdir/etc/systemd/system/testsuite.service <<EOF
>> +[Unit]
>> +Description=Testsuite service
>> +After=multi-user.target
>> +
>> +[Service]
>> +ExecStart=/test-seccomp.sh
>> +Type=oneshot
>> +EOF
>> +
>> + # copy the units used by this test
>> + cp {will-fail,will-not-fail,will-not-fail2}.service \
>> + $initdir/etc/systemd/system
>> + cp test-seccomp.sh $initdir/
>> +
>> + setup_testsuite
>> + )
>> + setup_nspawn_root
>> +
>> + ddebug "umount $TESTDIR/root"
>> + umount $TESTDIR/root
>> +}
>> +
>> +test_cleanup() {
>> + umount $TESTDIR/root 2>/dev/null
>> + [[ $LOOPDEV ]] && losetup -d $LOOPDEV
>> + return 0
>> +}
>> +
>> +do_test "$@"
>> diff --git a/test/TEST-04-SECCOMP/will-fail.service b/test/TEST-04-SECCOMP/will-fail.service
>> new file mode 100644
>> index 0000000..18e034e
>> --- /dev/null
>> +++ b/test/TEST-04-SECCOMP/will-fail.service
>> @@ -0,0 +1,6 @@
>> +[Unit]
>> +Description=Will fail
>> +
>> +[Service]
>> +ExecStart=/bin/echo "This should not be seen"
>> +SystemCallFilter=ioperm
>> diff --git a/test/TEST-04-SECCOMP/will-not-fail.service b/test/TEST-04-SECCOMP/will-not-fail.service
>> new file mode 100644
>> index 0000000..c56797f
>> --- /dev/null
>> +++ b/test/TEST-04-SECCOMP/will-not-fail.service
>> @@ -0,0 +1,6 @@
>> +[Unit]
>> +Description=Will not fail
>> +
>> +[Service]
>> +ExecStart=/bin/echo "Foo bar"
>> +SystemCallFilter=~ioctl
>> diff --git a/test/TEST-04-SECCOMP/will-not-fail2.service b/test/TEST-04-SECCOMP/will-not-fail2.service
>> new file mode 100644
>> index 0000000..2df05e3
>> --- /dev/null
>> +++ b/test/TEST-04-SECCOMP/will-not-fail2.service
>> @@ -0,0 +1,6 @@
>> +[Unit]
>> +Description=Reset SystemCallFilter
>> +
>> +[Service]
>> +ExecStart=/bin/echo "Foo bar"
>> +SystemCallFilter=
>> --
>> 1.8.5.3
>>
>> _______________________________________________
>> systemd-devel mailing list
>> systemd-devel at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/systemd-devel
More information about the systemd-devel
mailing list