From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from relay.sgi.com (relay2.corp.sgi.com [137.38.102.29]) by oss.sgi.com (Postfix) with ESMTP id C0A817F3F for ; Thu, 11 Jul 2013 13:28:40 -0500 (CDT) Received: from cuda.sgi.com (cuda3.sgi.com [192.48.176.15]) by relay2.corp.sgi.com (Postfix) with ESMTP id A543B30407B for ; Thu, 11 Jul 2013 11:28:40 -0700 (PDT) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by cuda.sgi.com with ESMTP id CnCksP4P9J7Hh1fy for ; Thu, 11 Jul 2013 11:28:36 -0700 (PDT) Date: Thu, 11 Jul 2013 15:28:30 -0300 From: Carlos Maiolino Subject: Re: [PATCH V2] xfstests: generic/313, test sgid inheritance on subdirs Message-ID: <20130711182829.GC10711@andromeda.usersys.redhat.com> References: <51A68175.9020202@redhat.com> <51A7B03E.2080909@sandeen.net> <20130612192320.GA12955@andromeda.usersys.redhat.com> <20130708215151.GK20932@sgi.com> <20130711175315.GB10711@andromeda.usersys.redhat.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20130711175315.GB10711@andromeda.usersys.redhat.com> List-Id: XFS Filesystem from SGI List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: xfs-bounces@oss.sgi.com Sender: xfs-bounces@oss.sgi.com To: Ben Myers Cc: Eric Sandeen , Eric Sandeen , xfs-oss > > > > generic/313 - output mismatch (see /root/xfstests/results/generic/313.out.bad) > > --- tests/generic/313.out 2013-07-08 16:27:41.787710646 -0500 > > +++ /root/xfstests/results/generic/313.out.bad 2013-07-08 16:47:46.052683735 -0500 > > @@ -1,3 +1,3 @@ > > QA output created by 313 > > -drwxr-sr-x. TEST_DIR/313-dir/subdir > > +drwxr-sr-x TEST_DIR/313-dir/subdir > > drwxrwsr-x+ TEST_DIR/313-dir/subdir2 > > ... > > (Run 'diff -u tests/generic/313.out /root/xfstests/results/generic/313.out.bad' to see the entire diff) > > > > Looks like there could be a problem with ls? Have you seen that? > > > > Thanks, > > Ben > > > Actually I don't think this is a problem, but the way newer `ls` versions are > displaying the object permissions (the 'dot' at the end of the permissions, > indicating there are no extra attributes). > > I'm going to take a look on what's going on, but I still believe it's just a > matter of add the 'dot' to the xfstests correct output. > > I had this same problem when testing my patch and needed to fix it locally, but > missed to warn you guys, my apologies > Just a matter of information: >>From coreutils: commit b3677e5e383103bf1764b2c8a9329b1c17934b24 Author: Jim Meyering Date: Wed Apr 2 22:26:45 2008 +0200 ls: use '.' (not +) as SELinux-only alt. access flag in ls -l output So, this test is selinux dependent, it will provide different outputs whether the system has selinux enabled or not. Since the test itself creates their own directories, checking if the selinux is enabled or not and checking the proper output depending on selinux activity should avoid false positives on this test. I.e. if the selinux is enabled, the `ls -l` output will print the 'dot' at the end of the permissions, otherwise, nothing will be printed and Eric's test will pass without problem. I think this is something worth to mention on xfstests README or some other documentation, mainly because any kind of test like this, but done with the TEST_DEV might be even worst since we don't recreate the filesystem while using TEST_DEV, so, objects there can be created with selinux attrs or not (if created when selinux is disabled) and have the attributes added later. I'm probably being too paranoid here talking about the TEST_DEV, but, I thought it was worth to mention. Cheers. -- Carlos _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs