Eric, Jaegeuk,
On Mon, Apr 24, 2017 at 7:00 PM, Eric Biggers <ebigge...@gmail.com> wrote:
> From: Eric Biggers <ebigg...@google.com>
>
> Switch f2fs directory searches to use the fscrypt_match_name() helper
> function. There should be no functional change.
> -#ifdef CONFIG_F2FS_FS_ENCRYPTION
> - if (unlikely(!name->name)) {
> - if (fname->usr_fname->name[0] == '_') {
> - if (de_name.len > 32 &&
> - !memcmp(de_name.name + ((de_name.len
> - 17) & ~15),
> - fname->crypto_buf.name + 8,
> 16))
> - goto found;
> - goto not_match;
> - }
> - name->name = fname->crypto_buf.name;
> - name->len = fname->crypto_buf.len;
> - }
Sorry if this is a stupid question, but why do you have to compare hashes _and_
the last few bytes of the bigname?
A lookup via bigname gives you two 32bits hash values, and there I'd assume that
this is sufficient for a collisions free lookup. Especially since an
resumed readdir()
with a 64bits cookie has to work too on your filesystem.
--
Thanks,
//richard
------------------------------------------------------------------------------
Check out the vibrant tech community on one of the world's most
engaging tech sites, Slashdot.org! http://sdm.link/slashdot
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
