On 15.12.2016 20:19, Eric Biggers wrote: > From: Eric Biggers <ebiggers@xxxxxxxxxx> > > Now that fscrypt_has_permitted_context() compares the fscrypt_context > rather than the fscrypt_info when needed, it is no longer necessary to > delay fscrypt_has_permitted_context() from ->lookup() to ->open() for > regular files, as introduced in commit ff978b09f973 ("ext4 crypto: move > context consistency check to ext4_file_open()"). Therefore the check in > ->open(), along with the dget_parent() hack, can be removed. > > It's also no longer necessary to check the file type before calling > fscrypt_has_permitted_context(). > > This patch makes these changes for both ext4 and f2fs. > > Signed-off-by: Eric Biggers <ebiggers@xxxxxxxxxx> > --- > fs/ext4/file.c | 12 ------------ > fs/ext4/namei.c | 10 ++-------- > fs/f2fs/file.c | 15 +++++---------- > fs/f2fs/namei.c | 7 ++----- > 4 files changed, 9 insertions(+), 35 deletions(-) Can please also take care of UBIFS? :-) Thanks, //richard -- To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html