On Sun, Apr 28, 2002 at 07:02:21PM -0800, Ethan Benson wrote:
> On Mon, Apr 29, 2002 at 12:35:05PM +1000, Timothy Shimmin wrote:
> > Hi Ethan,
> >
> > Yeah your behaviour with the file gaining execute permission in the
> > user ACE definitely looks wrong _but_ I can NOT repeat it locally.
> >
> > tes@sagan /mnt/xfs0/testdir/test1> setfacl -dm u:tes:r-- .
> > tes@sagan /mnt/xfs0/testdir/test1> getfacl .
> > # file: .
> > # owner: tes
> > # group: tes
> > user::rwx
> > group::rwx
> > other::r-x
> > default:user::rwx
> > default:user:tes:r--
> > default:group::rwx
> > default:mask::rwx
> > default:other::r-x
> >
> > tes@sagan /mnt/xfs0/testdir/test1> touch foo
> > tes@sagan /mnt/xfs0/testdir/test1> getfacl foo
> > # file: foo
> > # owner: tes
> > # group: tes
> > user::rw-
> > user:tes:r--
> > group::rwx #effective:rw-
> > mask::rw-
> > other::r--
> >
> > You could try adding printk's in
> > linux/fs/xfs/xfs_acl.c/xfs_acl_inherit() & xfs_acl_filter_mode() and
> > see what is going wrong.
> >
> > OOI, what is the output from running "check 051" in
> > the cmd/xfstests directory (i.e. the acl regression test) ?
> > (You need to look at cmd/xfstests/README about setting this stuff up;
> > one needs to setup some variables to point to xfs filesystems etc...)
>
> i assume you used current CVS XFS?
Yep.
> im on 2.4.18 split patches, so perhaps this is already fixed,
Perhaps.
(But I've never seen this as a bug before - so there were
no intentional fixes AFAIK.)
--Tim
|