Re: [PATCH 1/1] fs/inode.c: spelling in inode_owner_or_capable doc

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Apr 21, 2017 at 01:37:58PM +0200, Aurelien Aptel wrote:
>   *
> - * Return true if current either has CAP_FOWNER in a namespace with the
> + * Return true if current task has either CAP_FOWNER in a namespace with the
>   * inode owner uid mapped, or owns the file.

This is wrong.  "if the current task has either FOO or owns the file".
It's also unnecessarily complex.  Try this:

 * Return true if the current task owns the file or has CAP_FOWNER in a
 * namespace with the inode owner uid mapped

("either" in this context would imply exclusive-or, which isn't the intended
meaning).



[Index of Archives]     [Linux Ext4 Filesystem]     [Union Filesystem]     [Filesystem Testing]     [Ceph Users]     [Ecryptfs]     [AutoFS]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux Cachefs]     [Reiser Filesystem]     [Linux RAID]     [Samba]     [Device Mapper]     [CEPH Development]
  Powered by Linux