Re: CVS commit: src/lib/libperfuse

[David Holland <dholland-sourcechanges%netbsd.org@localhost>]

On Fri, Sep 05, 2014 at 01:57:19PM +0000, Emmanuel Dreyfus wrote:
 > >  > Modified Files:
 > >  >      src/lib/libperfuse: ops.c perfuse.c
 > >  > 
 > >  > Log Message:
 > >  > Improve POSIX compliance of FUSE filesystems through PERUSE
 > >  > - access denied is EPERM and not EACCES
 > > 
 > > wait, what?
 > 
 > EACCES is when you lack permission for the parent directories. 
 > This errno is enforced by the LOOKUP method. If you have access
 > to the directory but not to the object itself, you get EPERM,
 > and this is enforced by the SETATTR method, which will not be
 > called if you did not succeed LOOKUP first.
 > [...]

If you're sure that all the cases match ffs (which we believe to be
correct on this stuff, modulo any kauth glitches) then I have no
objection. It's just that most fs-level permission failures are
supposed to be EACCES so I was alarmed.

 > Please Cc: me for the reply because I am not subscribed to this 
 > list. And your reply is needed even if you agree because I filled
 > a pullup ticket (I did not see you reply), which is now stalled 
 > because of your objection: I now need your approval :-)

Ticket number?

-- 
David A. Holland
dholland%netbsd.org@localhost