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: Thu, 21 May 2020 20:34:37 -0700 From: Eric Biggers <ebiggers@...nel.org> To: linux-ext4@...r.kernel.org Cc: Jaegeuk Kim <jaegeuk@...nel.org>, linux-fsdevel@...r.kernel.org, Al Viro <viro@...iv.linux.org.uk>, stable@...r.kernel.org Subject: Re: [PATCH] ext4: fix race between ext4_sync_parent() and rename() On Wed, May 06, 2020 at 11:31:40AM -0700, Eric Biggers wrote: > /* > * The directory inode may have gone through rmdir by now. But > * the inode itself and its blocks are still allocated (we hold > - * a reference to the inode so it didn't go through > - * ext4_evict_inode()) and so we are safe to flush metadata > - * blocks and the inode. > + * a reference to the inode via its dentry), so it didn't go > + * through ext4_evict_inode()) and so we are safe to flush > + * metadata blocks and the inode. > */ > ret = sync_mapping_buffers(inode->i_mapping); Just realized that in this comment, the closing parenthesis I added after "dentry" shouldn't be there. Ted, feel free to fix this if you're so inclined. - Eric
Powered by blists - more mailing lists