From patchwork Thu Jan 30 20:11:06 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Stephen Smalley X-Patchwork-Id: 11358607 Return-Path: Received: from mail.kernel.org (pdx-korg-mail-1.web.codeaurora.org [172.30.200.123]) by pdx-korg-patchwork-2.web.codeaurora.org (Postfix) with ESMTP id 17C3313B4 for ; Thu, 30 Jan 2020 20:10:45 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id CFE812082E for ; Thu, 30 Jan 2020 20:10:44 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=tycho.nsa.gov header.i=@tycho.nsa.gov header.b="oPJxw88P" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727142AbgA3UKo (ORCPT ); Thu, 30 Jan 2020 15:10:44 -0500 Received: from UPDC19PA19.eemsg.mail.mil ([214.24.27.194]:35413 "EHLO UPDC19PA19.eemsg.mail.mil" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726514AbgA3UKo (ORCPT ); Thu, 30 Jan 2020 15:10:44 -0500 X-EEMSG-check-017: 52221384|UPDC19PA19_ESA_OUT01.csd.disa.mil X-IronPort-AV: E=Sophos;i="5.70,382,1574121600"; d="scan'208";a="52221384" Received: from emsm-gh1-uea11.ncsc.mil ([214.29.60.3]) by UPDC19PA19.eemsg.mail.mil with ESMTP/TLS/DHE-RSA-AES256-SHA256; 30 Jan 2020 20:10:40 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tycho.nsa.gov; i=@tycho.nsa.gov; q=dns/txt; s=tycho.nsa.gov; t=1580415040; x=1611951040; h=from:to:cc:subject:date:message-id:mime-version: content-transfer-encoding; bh=2ndt9O6PqTCwkTMR6yoSUdV2zcQgP+K41YH2om+DCnQ=; b=oPJxw88PB82derSzrBcj51EXFPbTr/DoYa8gg6C6jzxtOnGWdLk4jvXf zMAYIejt+lLbgafIR8RJR+IQylhtxv04YhVy1mVWXWRkKGUJeYNLlCBuT mvsKjiwQfF8b+iGG73y/OYJcvGUEMrZK+size6KjFsMxDOF5tLoViEZ4o 2H2JYD30aR33nN7eYYrJeS2uHEL9vU4iJu7uFLJHnoz4sSyyiQGcL84wR E3xfPk4TP4eyTMIxNPkj5vmCirWwGacLMfS28OCsFWZZbcs13eEtRxzau Xvca/j4GEBebyWzsslqQoy8I5njuqGcdsU8b7IaVenfjVjLX/csI2AWqm g==; X-IronPort-AV: E=Sophos;i="5.70,382,1574121600"; d="scan'208";a="38484124" IronPort-PHdr: 9a23:hGo5VRT5gu6KKExYqh/7Ckcyatpsv+yvbD5Q0YIujvd0So/mwa67ZBaGt8tkgFKBZ4jH8fUM07OQ7/m8HzdQqs/b4ThCKMUKC0Zez51O3kQJO42sMQXDNvnkbig3ToxpdWRO2DWFC3VTA9v0fFbIo3e/vnY4ExT7MhdpdKyuQtaBx8u42Pqv9JLNfg5GmCSyYa9oLBWxsA7dqtQajZFtJ6osxRbFuHVFdudZyW91O16emwv36sOs8JJ+6ShdtO8t+sBaXanmY6g0SKFTASg7PWwy+MDlrwTIQxGV5nsbXGUWkx5IDBbA4RrnQJr/sTb0u/Rk1iWCMsL4Ub47WTK576d2UxDokzsINyQ48G7MlMN9ir9QrQ+7qBx+x47UZ5yVNOZ7c6jAc94WWXZNU8BMXCNPGIO8a5YEAfQHM+hWsoLxo0ICoBW6CAWpAu7k1z1GiWLs3aAi0+ovDxzI0gIjEdwJv3vao874OLsRXO2v0KXF1ynPY+9Q1Dzg6IbIaBchofSUULxtbcre11EvFwPbgVWWtIfrIjaV2f4Js2if8eVhVf+khmk8qw5suTii3dssi4nViYIVzVDI7yN5wJ0vKtGiR057ZsCkHYJWuiqHOYV2RcYiTHtpuCY80rAGtp+7fDQKyJQ63BHTceCIc4+N4h/lSe2fIi94iWp4dL+wiBu+60itxvDmWsWq31tGsDBJnsTKu3sQzRLc8NKHReF4/kq53DaP0B3c5f9cLEAvkKrbN4YhwrktlpoPqUjDHjH5mEHxjKKOakUr4PKo6+X6YrX9vJOcNol0ig/5MqQ1hMOwHf40MhQOXmeH+eS806fv/UvlT7VRkvI6iKzZsJHeJcgDp665BRFa0po75hqiADqr384UkHkaIF5fZh6KgJblN0vTLPzgCPewmVWskDNlx/DcOb3hB43ALmPen7f6Zrt9905dxxYzzdBY/Z5UDKoBL+jpVk/+s9zYEAc1MxaozOb/FNV9yoQeVHqVAq+YLKzSsUWH5vgyLOaUeIAaoDb9JOIk5/7ok3A5gkQQcrK13ZsWbnC4AuppI1+fYXXyntcNC3sFvg07TObykl2NTSZTZ2quX6I7/jw7D4emDYDYRoCrmrOOwDm0E4ZMZm9YF1+MEW3nd5meV/gQbyKSJ9drkiYYWri5V48hyRauuRfhy7V5KurU+ysYtY/s1dVu+e3Tmg899SZuA8SezW6NVWd0kX0MRzMs26B/u0N9wE+Z0adkm/xYCcBT5/RRXwciL5Hc0uh6BsvqWg3bY9eJTE+qQtChATE2VNI+3cUCbFp6G9Wn3Vj/2H+yCqIRv6SCGZhx96XbxXW3LMF4mFjc06x0tEUrWstCMyWdg6d78wXCT9rSn16xi7ehdaNa2jXEsmiE0zzd7wljTAdsXPCdDjgkbUzMoIG8vRLP X-IPAS-Result: A2AvAADvNzNe/wHyM5BlHAEBAQEBBwEBEQEEBAEBgWcHAQELAYF8gRhUASASKow3YIZbBosmj06BewkBAQEBAQEBAQEbEAwBAYRAglQ0CQ4CEAEBAQQBAQEBAQUDAQFshTcMgjspgzMBRoFRgmc/AYJWJQ+vCYkFgT6BOAGHQYR4eYEHgRE2gxuCSxkCgU2GBASXfZdggkOCTIR3hUWJJgwbg0CXQpdElCo5gVgrCAIYCCEPgycJRxgNkhCECIJJhCAjAzCOUgEB Received: from tarius.tycho.ncsc.mil (HELO tarius.infosec.tycho.ncsc.mil) ([144.51.242.1]) by emsm-gh1-uea11.NCSC.MIL with ESMTP; 30 Jan 2020 20:10:27 +0000 Received: from moss-pluto.infosec.tycho.ncsc.mil (moss-pluto [192.168.25.131]) by tarius.infosec.tycho.ncsc.mil (8.14.7/8.14.4) with ESMTP id 00UK9bao216072; Thu, 30 Jan 2020 15:09:40 -0500 From: Stephen Smalley To: paul@paul-moore.com Cc: selinux@vger.kernel.org, omosnace@redhat.com, richard_c_haines@btinternet.com, Stephen Smalley Subject: [PATCH v2 1/2] testsuite: provide support for testing labeled NFS Date: Thu, 30 Jan 2020 15:11:06 -0500 Message-Id: <20200130201107.4152-1-sds@tycho.nsa.gov> X-Mailer: git-send-email 2.24.1 MIME-Version: 1.0 Sender: selinux-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: selinux@vger.kernel.org Provide instructions in the README.md file, the required kernel config options in defconfig, and a nfs.sh script for running the testsuite within a labeled NFS mount. This depends on the previous change to enable running over labeled NFS without failures. This completes the first part of https://github.com/SELinuxProject/selinux-testsuite/issues/32. What remains unfinished is adding tests that context mounts are properly honored, with and without security_label in exports, for NFS, and default labeling of NFS when neither security_label nor context mounts are used (i.e. genfscon default of nfs_t). Signed-off-by: Stephen Smalley --- v2 moves nfs.sh under tools/. As before, these two patches depend on the previous one ("testsuite:enable running over labeled NFS") in order to allow the testsuite to pass on NFS mounts. README.md | 41 +++++++++++++++++++++++++++++++++++++++++ defconfig | 10 ++++++++++ tools/nfs.sh | 13 +++++++++++++ 3 files changed, 64 insertions(+) create mode 100755 tools/nfs.sh diff --git a/README.md b/README.md index 4352796edb2d..e02ae9ac6d6f 100644 --- a/README.md +++ b/README.md @@ -118,6 +118,47 @@ the tests: tests/infiniband_pkey/ibpkey_test.conf tests/infiniband_endport/ibendport_test.conf +#### NFS + +It is possible to run most of the tests within a labeled NFS mount in +order to exercise the NFS security labeling functionality. Certain +tests have been excluded from such testing due to differences between +NFS and local filesystems; these tests will be automatically skipped. + +You will need to install an additional package, the package below +is for Fedora/RHEL but other Linux distributions should have a similar +package: + +* nfs-utils _(for `nfsd', `exportfs', and other NFS-related programs)_ + +On a modern Fedora system you can install this dependency with the +following command: + + # dnf install nfs-utils + +If your distribution does not use systemd as its init system, you will +need to customize the nfs.sh script found in the tools directory to +correctly start and stop the nfs server. You may also choose to not +start/stop the nfs-server as part of the script by removing those lines +if you are already using NFS for other reasons. + +Before running the tests in a labeled NFS mount, first ensure that you +can run them successfully on a local filesystem following the standard +instructions further below. Any failures that occur on a local +filesystem should also typically be expected when running over NFS. + +To run the tests within a labeled NFS mount, you can run the +nfs.sh script while in the selinux-testsuite directory: + + # cd selinux-testsuite + # ./tools/nfs.sh + +The script will start the nfs-server, export the mount containing the +testsuite directory with the security_label option to localhost, mount +it via NFSv4.2 on /mnt/selinux-testsuite, switch to that directory, +and run the testsuite there. After completion, it will unmount and +unexport the mount and then stop the nfs-server. + ## Running the Tests Create a shell with the `unconfined_r` or `sysadm_r` role and the Linux diff --git a/defconfig b/defconfig index 7cb6a2ca7f71..8419e40b79dc 100644 --- a/defconfig +++ b/defconfig @@ -94,3 +94,13 @@ CONFIG_TRACEPOINTS=y CONFIG_BLK_DEV_LOOP=m CONFIG_BLK_DEV_LOOP_MIN_COUNT=0 CONFIG_QFMT_V2=y + +# Test labeled NFS. +# This is not required for SELinux operation itself. +CONFIG_NFS_FS=m +CONFIG_NFS_V4=m +CONFIG_NFS_V4_2=y +CONFIG_NFS_V4_SECURITY_LABEL=y +CONFIG_NFSD=m +CONFIG_NFSD_V4=y +CONFIG_NFSD_V4_SECURITY_LABEL=y diff --git a/tools/nfs.sh b/tools/nfs.sh new file mode 100755 index 000000000000..31c66c377cae --- /dev/null +++ b/tools/nfs.sh @@ -0,0 +1,13 @@ +#!/bin/sh -e +MOUNT=`stat --print %m .` +TESTDIR=`pwd` +systemctl start nfs-server +exportfs -orw,no_root_squash,security_label localhost:$MOUNT +mkdir -p /mnt/selinux-testsuite +mount -t nfs -o vers=4.2 localhost:$TESTDIR /mnt/selinux-testsuite +pushd /mnt/selinux-testsuite +make test +popd +umount /mnt/selinux-testsuite +exportfs -u localhost:$MOUNT +systemctl stop nfs-server