diff options
author | Eric Biggers <ebiggers@google.com> | 2017-04-24 13:00:11 -0400 |
---|---|---|
committer | Theodore Ts'o <tytso@mit.edu> | 2017-05-04 11:44:38 -0400 |
commit | 067d1023b6beeded688500d417e40e16981437e0 (patch) | |
tree | 6fe281855041f786d0e276d1054d0646635a3f20 | |
parent | 17159420a6c18bb3515ff85598b5ccf1a572763d (diff) |
ext4: switch to using fscrypt_match_name()
Switch ext4 directory searches to use the fscrypt_match_name() helper
function. There should be no functional change.
Signed-off-by: Eric Biggers <ebiggers@google.com>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
-rw-r--r-- | fs/ext4/namei.c | 22 |
1 files changed, 5 insertions, 17 deletions
diff --git a/fs/ext4/namei.c b/fs/ext4/namei.c index 2d866e4a9e7a..a87e49c31248 100644 --- a/fs/ext4/namei.c +++ b/fs/ext4/namei.c | |||
@@ -1245,29 +1245,17 @@ static void dx_insert_block(struct dx_frame *frame, u32 hash, ext4_lblk_t block) | |||
1245 | static inline int ext4_match(struct ext4_filename *fname, | 1245 | static inline int ext4_match(struct ext4_filename *fname, |
1246 | struct ext4_dir_entry_2 *de) | 1246 | struct ext4_dir_entry_2 *de) |
1247 | { | 1247 | { |
1248 | const void *name = fname_name(fname); | 1248 | struct fscrypt_name f; |
1249 | u32 len = fname_len(fname); | ||
1250 | 1249 | ||
1251 | if (!de->inode) | 1250 | if (!de->inode) |
1252 | return 0; | 1251 | return 0; |
1253 | 1252 | ||
1253 | f.usr_fname = fname->usr_fname; | ||
1254 | f.disk_name = fname->disk_name; | ||
1254 | #ifdef CONFIG_EXT4_FS_ENCRYPTION | 1255 | #ifdef CONFIG_EXT4_FS_ENCRYPTION |
1255 | if (unlikely(!name)) { | 1256 | f.crypto_buf = fname->crypto_buf; |
1256 | if (fname->usr_fname->name[0] == '_') { | ||
1257 | int ret; | ||
1258 | if (de->name_len <= 32) | ||
1259 | return 0; | ||
1260 | ret = memcmp(de->name + ((de->name_len - 17) & ~15), | ||
1261 | fname->crypto_buf.name + 8, 16); | ||
1262 | return (ret == 0) ? 1 : 0; | ||
1263 | } | ||
1264 | name = fname->crypto_buf.name; | ||
1265 | len = fname->crypto_buf.len; | ||
1266 | } | ||
1267 | #endif | 1257 | #endif |
1268 | if (de->name_len != len) | 1258 | return fscrypt_match_name(&f, de->name, de->name_len); |
1269 | return 0; | ||
1270 | return (memcmp(de->name, name, len) == 0) ? 1 : 0; | ||
1271 | } | 1259 | } |
1272 | 1260 | ||
1273 | /* | 1261 | /* |