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 linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-ID: <20231122205926.GH38156@ZenIV> Date: Wed, 22 Nov 2023 20:59:26 +0000 From: Al Viro <viro@...iv.linux.org.uk> To: Gabriel Krisman Bertazi <krisman@...e.de> Cc: brauner@...nel.org, tytso@....edu, ebiggers@...nel.org, jaegeuk@...nel.org, linux-fsdevel@...r.kernel.org, linux-ext4@...r.kernel.org, linux-f2fs-devel@...ts.sourceforge.net, Gabriel Krisman Bertazi <krisman@...labora.com> Subject: Re: [PATCH v6 3/9] fs: Expose name under lookup to d_revalidate hooks On Wed, Aug 16, 2023 at 01:07:57AM -0400, Gabriel Krisman Bertazi wrote: > From: Gabriel Krisman Bertazi <krisman@...labora.com> > > Negative dentries support on case-insensitive ext4/f2fs will require > access to the name under lookup to ensure it matches the dentry. This > adds the information on d_revalidate and updates its implementation. There's actually one hell of a stronger reason for that particular change; uses of ->d_name in ->d_revalidate() instances are often racy. So IMO this is the right way to go, regardless of c-i stuff, except that it ought to be followed by making individual ->d_revalidate() instances use the damn argument, now that they have it in stable form. Said followups don't need to be in the same series, obviously.
Powered by blists - more mailing lists