generic: clear of suid/sgid on direct write

Check that suid/sgid bits are cleared on direct write. XFS triggered
WARN_ON_ONCE in this case. Patchset from Jan Kara fixed the warning:

http://oss.sgi.com/archives/xfs/2014-12/msg00071.html

This test is inspired by a test case from Eric Sandeen, and follows
the test steps in generic/193. This test requires direct I/O, it's
not added to generic/193 but to a new test, so that generic/193
still runs on filesystems don't have direct I/O support.

Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Eryu Guan <eguan@redhat.com>
This commit is contained in:
Eryu Guan
2016-05-26 13:59:48 +08:00
parent 86e7366e84
commit 6f54d8136b
3 changed files with 113 additions and 0 deletions
+98
View File
@@ -0,0 +1,98 @@
#! /bin/bash
# FS QA Test 355
#
# Test clear of suid/sgid on direct write.
#
#-----------------------------------------------------------------------
# Copyright (c) 2016 Red Hat Inc., All Rights Reserved.
#
# This program is free software; you can redistribute it and/or
# modify it under the terms of the GNU General Public License as
# published by the Free Software Foundation.
#
# This program is distributed in the hope that it would be useful,
# but WITHOUT ANY WARRANTY; without even the implied warranty of
# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
# GNU General Public License for more details.
#
# You should have received a copy of the GNU General Public License
# along with this program; if not, write the Free Software Foundation,
# Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
#-----------------------------------------------------------------------
#
seq=`basename $0`
seqres=$RESULT_DIR/$seq
echo "QA output created by $seq"
here=`pwd`
tmp=/tmp/$$
status=1 # failure is the default!
trap "_cleanup; exit \$status" 0 1 2 3 15
_cleanup()
{
cd /
rm -f $tmp.*
}
# get standard environment, filters and checks
. ./common/rc
. ./common/filter
rm -f $seqres.full
# real QA test starts here
_supported_fs generic
_supported_os Linux
_require_test
_require_user
_require_odirect
testfile=$TEST_DIR/$seq.test
rm -f $testfile
do_io()
{
su $qa_user -c "$XFS_IO_PROG -d -c 'pwrite 0 4k' $testfile" \
>>$seqres.full
}
echo "Check that suid/sgid bits are cleared after direct write"
# create testfile and set base ownership & permission
echo "this is a test" >> $testfile
chmod 644 $testfile
chown $qa_user:$qa_user $testfile
echo "== with no exec perm"
chmod ug+s $testfile
echo -n "before: "; stat -c '%A' $testfile
do_io
echo -n "after: "; stat -c '%A' $testfile
echo "== with user exec perm"
chmod ug+s $testfile
chmod u+x $testfile
echo -n "before: "; stat -c '%A' $testfile
do_io
echo -n "after: "; stat -c '%A' $testfile
echo "== with group exec perm"
chmod ug+s $testfile
chmod g+x $testfile
chmod u-x $testfile
echo -n "before: "; stat -c '%A' $testfile
do_io
echo -n "after: "; stat -c '%A' $testfile
echo "== with user+group exec perm"
chmod ug+s $testfile
chmod u+x $testfile
echo -n "before: "; stat -c '%A' $testfile
do_io
echo -n "after: "; stat -c '%A' $testfile
# success, all done
status=0
exit
+14
View File
@@ -0,0 +1,14 @@
QA output created by 355
Check that suid/sgid bits are cleared after direct write
== with no exec perm
before: -rwSr-Sr--
after: -rw-r-Sr--
== with user exec perm
before: -rwsr-Sr--
after: -rwxr-Sr--
== with group exec perm
before: -rwSr-sr--
after: -rw-r-xr--
== with user+group exec perm
before: -rwsr-sr--
after: -rwxr-xr--
+1
View File
@@ -357,3 +357,4 @@
352 auto clone
353 auto quick clone
354 auto
355 auto quick