[PATCH] afs: Fix updating of i_mode due to 3rd party change

Al Viro viro at zeniv.linux.org.uk
Wed Feb 10 14:09:10 EST 2021


On Wed, Feb 10, 2021 at 07:01:31PM +0000, David Howells wrote:
> Al Viro <viro at zeniv.linux.org.uk> wrote:
> 
> > On Wed, Feb 10, 2021 at 08:59:52AM +0000, David Howells wrote:
> > > Fix afs_apply_status() to mask off the irrelevant bits from status->mode
> > > when OR'ing them into i_mode.  This can happen when a 3rd party chmod
> > > occurs.
> > > 
> > > Also fix afs_inode_init_from_status() to mask off the mode bits when
> > > initialising i_mode.
> > 
> > afd_inode_init_from_status() would be better off with
> > 	status->mode &= S_IALLUGO;
> > before the switch, seeing that normally it has all-zeroes in type bits anyway.
> 
> That might cause the check in afs_apply_status() to malfunction.  Any such
> masking off would need to be done earlier than that.
> 
> Also, it's been suggested that I should also mask off the SUID, SGID and VTX
> bits.

No sticky bit on AFS, you mean?



More information about the linux-afs mailing list