Commit f874fa1c authored by Eric Biggers's avatar Eric Biggers Committed by Jaegeuk Kim
Browse files

f2fs: split f2fs_d_compare() from f2fs_match_name() 



Sharing f2fs_ci_compare() between comparing cached dentries
(f2fs_d_compare()) and comparing on-disk dentries (f2fs_match_name())
doesn't work as well as intended, as these actions fundamentally differ
in several ways (e.g. whether the task may sleep, whether the directory
is stable, whether the casefolded name was precomputed, whether the
dentry will need to be decrypted once we allow casefold+encrypt, etc.)

Just make f2fs_d_compare() implement what it needs directly, and rework
f2fs_ci_compare() to be specialized for f2fs_match_name().

Signed-off-by: default avatarEric Biggers <ebiggers@google.com>
Reviewed-by: default avatarChao Yu <yuchao0@huawei.com>
Signed-off-by: default avatarJaegeuk Kim <jaegeuk@kernel.org>
parent ff5f85c8

fs/f2fs/dir.c

+35 −35
Original line number Diff line number Diff line
@@ -107,36 +107,28 @@ static struct f2fs_dir_entry *find_in_block(struct inode *dir, 
/*

 * Test whether a case-insensitive directory entry matches the filename

 * being searched for.

 *

 * Returns: 0 if the directory entry matches, more than 0 if it

 * doesn't match or less than zero on error.

 */

int f2fs_ci_compare(const struct inode *parent, const struct qstr *name,

static bool f2fs_match_ci_name(const struct inode *dir, const struct qstr *name,

			       const struct qstr *entry, bool quick)

{

	const struct f2fs_sb_info *sbi = F2FS_SB(parent->i_sb);

	const struct f2fs_sb_info *sbi = F2FS_SB(dir->i_sb);

	const struct unicode_map *um = sbi->s_encoding;

	int ret;

	int res;



	if (quick)

		ret = utf8_strncasecmp_folded(um, name, entry);

		res = utf8_strncasecmp_folded(um, name, entry);

	else

		ret = utf8_strncasecmp(um, name, entry);



	if (ret < 0) {

		/* Handle invalid character sequence as either an error

		 * or as an opaque byte sequence.

		res = utf8_strncasecmp(um, name, entry);

	if (res < 0) {

		/*

		 * In strict mode, ignore invalid names.  In non-strict mode,

		 * fall back to treating them as opaque byte sequences.

		 */

		if (f2fs_has_strict_mode(sbi))

			return -EINVAL;



		if (name->len != entry->len)

			return 1;



		return !!memcmp(name->name, entry->name, name->len);

		if (f2fs_has_strict_mode(sbi) || name->len != entry->len)

			return false;

		return !memcmp(name->name, entry->name, name->len);

	}



	return ret;

	return res == 0;

}



static void f2fs_fname_setup_ci_filename(struct inode *dir,
@@ -188,9 +180,9 @@ static inline bool f2fs_match_name(struct f2fs_dentry_ptr *d, 
		if (cf_str->name) {

			struct qstr cf = {.name = cf_str->name,

					  .len = cf_str->len};

			return !f2fs_ci_compare(parent, &cf, &entry, true);

			return f2fs_match_ci_name(parent, &cf, &entry, true);

		}

		return !f2fs_ci_compare(parent, fname->usr_fname, &entry,

		return f2fs_match_ci_name(parent, fname->usr_fname, &entry,

					  false);

	}

#endif
@@ -1080,17 +1072,25 @@ const struct file_operations f2fs_dir_operations = { 
static int f2fs_d_compare(const struct dentry *dentry, unsigned int len,

			  const char *str, const struct qstr *name)

{

	struct qstr qstr = {.name = str, .len = len };

	const struct dentry *parent = READ_ONCE(dentry->d_parent);

	const struct inode *inode = READ_ONCE(parent->d_inode);

	const struct inode *dir = READ_ONCE(parent->d_inode);

	const struct f2fs_sb_info *sbi = F2FS_SB(dentry->d_sb);

	struct qstr entry = QSTR_INIT(str, len);

	int res;



	if (!inode || !IS_CASEFOLDED(inode)) {

		if (len != name->len)

			return -1;

		return memcmp(str, name->name, len);

	}

	if (!dir || !IS_CASEFOLDED(dir))

		goto fallback;



	res = utf8_strncasecmp(sbi->s_encoding, name, &entry);

	if (res >= 0)

		return res;



	return f2fs_ci_compare(inode, name, &qstr, false);

	if (f2fs_has_strict_mode(sbi))

		return -EINVAL;

fallback:

	if (len != name->len)

		return 1;

	return !!memcmp(str, name->name, len);

}



static int f2fs_d_hash(const struct dentry *dentry, struct qstr *str)

fs/f2fs/f2fs.h

+0 −5
Original line number Diff line number Diff line
@@ -3142,11 +3142,6 @@ int f2fs_update_extension_list(struct f2fs_sb_info *sbi, const char *name, 
							bool hot, bool set);

struct dentry *f2fs_get_parent(struct dentry *child);



extern int f2fs_ci_compare(const struct inode *parent,

			   const struct qstr *name,

			   const struct qstr *entry,

			   bool quick);



/*

 * dir.c

 */

CRA Git | Maintained and supported by SUSTech CRA and CCSE