Received: with ECARTIS (v1.0.0; list linux-xfs); Wed, 23 Oct 2002 07:05:43 -0700 (PDT) Received: from ledzep.americas.sgi.com (eaganfw1.sgi.com [198.149.7.1]) by oss.sgi.com (8.12.5/8.12.5) with SMTP id g9NE5duR027353 for ; Wed, 23 Oct 2002 07:05:40 -0700 Received: from daisy-e185.americas.sgi.com (daisy-e185.americas.sgi.com [128.162.185.214]) by ledzep.americas.sgi.com (SGI-8.9.3/americas-smart-nospam1.1) with ESMTP id JAA23405 for ; Wed, 23 Oct 2002 09:05:41 -0500 (CDT) Received: from stout.americas.sgi.com (stout.americas.sgi.com [128.162.187.5]) by daisy-e185.americas.sgi.com (SGI-8.9.3/SGI-server-1.8) with ESMTP id JAA04770 for ; Wed, 23 Oct 2002 09:05:41 -0500 (CDT) From: Eric Sandeen Received: by stout.americas.sgi.com (8.11.6/SGI-client-1.7) id g9NE1Cq10406; Wed, 23 Oct 2002 09:01:12 -0500 Message-Id: <200210231401.g9NE1Cq10406@stout.americas.sgi.com> Date: Wed, 23 Oct 2002 09:01:12 -0500 Subject: TAKE - Fix root exec access checks on files with acls X-archive-position: 1200 X-ecartis-version: Ecartis v1.0.0 Sender: linux-xfs-bounce@oss.sgi.com Errors-to: linux-xfs-bounce@oss.sgi.com X-original-sender: sandeen@sgi.com Precedence: bulk X-list: linux-xfs Fix root exec access checks on files with acls The standard VFS access checks look for at least one +x bit set before allowing root (CAP_DAC_OVERRIDE) exec access to the file. This does the analogous thing for files with acls, looking for at least 1 effective +x ace on the file before granting root/CAP_DAC_OVERRIDE exec access. There has been some discussion on the acl-devel list that -any- +x on -any- ace (even only the mask) should allow exec access for root, but I think this method (checking for effective +x) makes more sense. Easy enough to change if the consensus shifts. This should close internal bug 870306, although I forgot to tell ptools that. :) Date: Wed Oct 23 07:00:12 PDT 2002 Workarea: stout.americas.sgi.com:/localhome/src/sandeen/2.4.x-xfs/workarea-alwaysclean The following file(s) were checked into: bonnie.engr.sgi.com:/isms/slinx/2.4.x-xfs Modid: 2.4.x-xfs:slinx:130837a linux/fs/xfs/xfs_acl.c - 1.37 - When checking for CAP_DAC_OVERRIDE exec access on files with acls, look for an effective exec permission in the acls before granting access.