Re: [PATCH] simplify ntfs_access()

2012-11-20 Thread Miod Vallat
> Hi Tech, > > In ntfs_access(), 'mask' is being built up in > three different places. From what I can see > 'mode' doesn't change within this function, > so can't we simply set 'mask' once before > checking owner, groups and everyone? > Comments? Look again. The three instances compute different

[PATCH] simplify ntfs_access()

2012-11-20 Thread Michael W. Bombardieri
Hi Tech, In ntfs_access(), 'mask' is being built up in three different places. From what I can see 'mode' doesn't change within this function, so can't we simply set 'mask' once before checking owner, groups and everyone? Comments? - Michael Index: ntfs_vnops.c =