lkml.org 
[lkml]   [2008]   [May]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [patch 06/14] hfsplus: remove hfsplus_permission()
From
Date
> > And so I strongly suggest this patch as well.  Apparently this feature
> > is not even used by anybody, otherwise somebody would have noticed
> > when Trond's fix broke execute permission checking on regular files...
>
> And I'm very strongly against this.
> If you wanted to rip out the resource inode logic, it would be more than
> this.
> At worst this code does nothing right now, so why this sudden rush to
> remove this without even an attempt of fixing it?

Because

a) it's nontrivial to fix (even understanding the problem is
nontrivial, see Documentation/filesystems/directory-locking)

b) the feature is obviously unused, so the easiest fix is simply to
remove it for the time being.

We generally don't leave code around which is known to be deadlockable
by unprivileged users. Of course if you have a better fix, then
that's fine.

Miklos


\
 
 \ /
  Last update: 2008-05-23 20:05    [W:0.064 / U:0.548 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site