lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Wed, 7 Jul 2021 21:37:05 +0200 From: Jan Kara <jack@...e.cz> To: Gabriel Krisman Bertazi <krisman@...labora.com> Cc: amir73il@...il.com, djwong@...nel.org, tytso@....edu, david@...morbit.com, jack@...e.com, dhowells@...hat.com, khazhy@...gle.com, linux-fsdevel@...r.kernel.org, linux-ext4@...r.kernel.org, kernel@...labora.com Subject: Re: [PATCH v3 05/15] inotify: Don't force FS_IN_IGNORED On Tue 29-06-21 15:10:25, Gabriel Krisman Bertazi wrote: > According to Amir: > > "FS_IN_IGNORED is completely internal to inotify and there is no need > to set it in i_fsnotify_mask at all, so if we remove the bit from the > output of inotify_arg_to_mask() no functionality will change and we will > be able to overload the event bit for FS_ERROR." > > This is done in preparation to overload FS_ERROR with the notification > mechanism in fanotify. > > Suggested-by: Amir Goldstein <amir73il@...il.com> > Signed-off-by: Gabriel Krisman Bertazi <krisman@...labora.com> > Reviewed-by: Amir Goldstein <amir73il@...il.com> Looks good. Feel free to add: Reviewed-by: Jan Kara <jack@...e.cz> Honza > --- > fs/notify/inotify/inotify_user.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/fs/notify/inotify/inotify_user.c b/fs/notify/inotify/inotify_user.c > index 98f61b31745a..4d17be6dd58d 100644 > --- a/fs/notify/inotify/inotify_user.c > +++ b/fs/notify/inotify/inotify_user.c > @@ -89,10 +89,10 @@ static inline __u32 inotify_arg_to_mask(struct inode *inode, u32 arg) > __u32 mask; > > /* > - * Everything should accept their own ignored and should receive events > - * when the inode is unmounted. All directories care about children. > + * Everything should receive events when the inode is unmounted. > + * All directories care about children. > */ > - mask = (FS_IN_IGNORED | FS_UNMOUNT); > + mask = (FS_UNMOUNT); > if (S_ISDIR(inode->i_mode)) > mask |= FS_EVENT_ON_CHILD; > > -- > 2.32.0 > -- Jan Kara <jack@...e.com> SUSE Labs, CR
Powered by blists - more mailing lists