From: Gabriel Krisman Bertazi <krisman@suse.de>
To: viro@zeniv.linux.org.uk, brauner@kernel.org, tytso@mit.edu,
ebiggers@kernel.org, jaegeuk@kernel.org
Cc: linux-fsdevel@vger.kernel.org, linux-ext4@vger.kernel.org,
linux-f2fs-devel@lists.sourceforge.net,
Gabriel Krisman Bertazi <krisman@suse.de>
Subject: [PATCH v5 01/10] fs: Expose helper to check if a directory needs casefolding
Date: Fri, 11 Aug 2023 20:41:37 -0400 [thread overview]
Message-ID: <20230812004146.30980-2-krisman@suse.de> (raw)
In-Reply-To: <20230812004146.30980-1-krisman@suse.de>
In preparation to use it in ecryptfs, move needs_casefolding into a
public header and give it a namespaced name.
Signed-off-by: Gabriel Krisman Bertazi <krisman@suse.de>
---
fs/libfs.c | 14 ++------------
include/linux/fs.h | 21 +++++++++++++++++++++
2 files changed, 23 insertions(+), 12 deletions(-)
diff --git a/fs/libfs.c b/fs/libfs.c
index 5b851315eeed..8d0b64cfd5da 100644
--- a/fs/libfs.c
+++ b/fs/libfs.c
@@ -1381,16 +1381,6 @@ bool is_empty_dir_inode(struct inode *inode)
}
#if IS_ENABLED(CONFIG_UNICODE)
-/*
- * Determine if the name of a dentry should be casefolded.
- *
- * Return: if names will need casefolding
- */
-static bool needs_casefold(const struct inode *dir)
-{
- return IS_CASEFOLDED(dir) && dir->i_sb->s_encoding;
-}
-
/**
* generic_ci_d_compare - generic d_compare implementation for casefolding filesystems
* @dentry: dentry whose name we are checking against
@@ -1411,7 +1401,7 @@ static int generic_ci_d_compare(const struct dentry *dentry, unsigned int len,
char strbuf[DNAME_INLINE_LEN];
int ret;
- if (!dir || !needs_casefold(dir))
+ if (!dir || !dir_is_casefolded(dir))
goto fallback;
/*
* If the dentry name is stored in-line, then it may be concurrently
@@ -1453,7 +1443,7 @@ static int generic_ci_d_hash(const struct dentry *dentry, struct qstr *str)
const struct unicode_map *um = sb->s_encoding;
int ret = 0;
- if (!dir || !needs_casefold(dir))
+ if (!dir || !dir_is_casefolded(dir))
return 0;
ret = utf8_casefold_hash(um, dentry, str);
diff --git a/include/linux/fs.h b/include/linux/fs.h
index 6867512907d6..e3b631c6d24a 100644
--- a/include/linux/fs.h
+++ b/include/linux/fs.h
@@ -3213,6 +3213,27 @@ static inline bool dir_relax_shared(struct inode *inode)
return !IS_DEADDIR(inode);
}
+/**
+ * dir_is_casefolded - Safely determine if filenames inside of a
+ * directory should be casefolded.
+ * @dir: The directory inode to be checked
+ *
+ * Filesystems should usually rely on this instead of checking the
+ * S_CASEFOLD flag directly when handling inodes, to avoid surprises
+ * with corrupted volumes. Checking i_sb->s_encoding ensures the
+ * filesystem knows how to deal with case-insensitiveness.
+ *
+ * Return: if names will need casefolding
+ */
+static inline bool dir_is_casefolded(const struct inode *dir)
+{
+#if IS_ENABLED(CONFIG_UNICODE)
+ return IS_CASEFOLDED(dir) && dir->i_sb->s_encoding;
+#else
+ return false;
+#endif
+}
+
extern bool path_noexec(const struct path *path);
extern void inode_nohighmem(struct inode *inode);
--
2.41.0
WARNING: multiple messages have this Message-ID (diff)
From: Gabriel Krisman Bertazi <krisman@suse.de>
To: viro@zeniv.linux.org.uk, brauner@kernel.org, tytso@mit.edu,
ebiggers@kernel.org, jaegeuk@kernel.org
Cc: linux-fsdevel@vger.kernel.org,
Gabriel Krisman Bertazi <krisman@suse.de>,
linux-ext4@vger.kernel.org,
linux-f2fs-devel@lists.sourceforge.net
Subject: [f2fs-dev] [PATCH v5 01/10] fs: Expose helper to check if a directory needs casefolding
Date: Fri, 11 Aug 2023 20:41:37 -0400 [thread overview]
Message-ID: <20230812004146.30980-2-krisman@suse.de> (raw)
In-Reply-To: <20230812004146.30980-1-krisman@suse.de>
In preparation to use it in ecryptfs, move needs_casefolding into a
public header and give it a namespaced name.
Signed-off-by: Gabriel Krisman Bertazi <krisman@suse.de>
---
fs/libfs.c | 14 ++------------
include/linux/fs.h | 21 +++++++++++++++++++++
2 files changed, 23 insertions(+), 12 deletions(-)
diff --git a/fs/libfs.c b/fs/libfs.c
index 5b851315eeed..8d0b64cfd5da 100644
--- a/fs/libfs.c
+++ b/fs/libfs.c
@@ -1381,16 +1381,6 @@ bool is_empty_dir_inode(struct inode *inode)
}
#if IS_ENABLED(CONFIG_UNICODE)
-/*
- * Determine if the name of a dentry should be casefolded.
- *
- * Return: if names will need casefolding
- */
-static bool needs_casefold(const struct inode *dir)
-{
- return IS_CASEFOLDED(dir) && dir->i_sb->s_encoding;
-}
-
/**
* generic_ci_d_compare - generic d_compare implementation for casefolding filesystems
* @dentry: dentry whose name we are checking against
@@ -1411,7 +1401,7 @@ static int generic_ci_d_compare(const struct dentry *dentry, unsigned int len,
char strbuf[DNAME_INLINE_LEN];
int ret;
- if (!dir || !needs_casefold(dir))
+ if (!dir || !dir_is_casefolded(dir))
goto fallback;
/*
* If the dentry name is stored in-line, then it may be concurrently
@@ -1453,7 +1443,7 @@ static int generic_ci_d_hash(const struct dentry *dentry, struct qstr *str)
const struct unicode_map *um = sb->s_encoding;
int ret = 0;
- if (!dir || !needs_casefold(dir))
+ if (!dir || !dir_is_casefolded(dir))
return 0;
ret = utf8_casefold_hash(um, dentry, str);
diff --git a/include/linux/fs.h b/include/linux/fs.h
index 6867512907d6..e3b631c6d24a 100644
--- a/include/linux/fs.h
+++ b/include/linux/fs.h
@@ -3213,6 +3213,27 @@ static inline bool dir_relax_shared(struct inode *inode)
return !IS_DEADDIR(inode);
}
+/**
+ * dir_is_casefolded - Safely determine if filenames inside of a
+ * directory should be casefolded.
+ * @dir: The directory inode to be checked
+ *
+ * Filesystems should usually rely on this instead of checking the
+ * S_CASEFOLD flag directly when handling inodes, to avoid surprises
+ * with corrupted volumes. Checking i_sb->s_encoding ensures the
+ * filesystem knows how to deal with case-insensitiveness.
+ *
+ * Return: if names will need casefolding
+ */
+static inline bool dir_is_casefolded(const struct inode *dir)
+{
+#if IS_ENABLED(CONFIG_UNICODE)
+ return IS_CASEFOLDED(dir) && dir->i_sb->s_encoding;
+#else
+ return false;
+#endif
+}
+
extern bool path_noexec(const struct path *path);
extern void inode_nohighmem(struct inode *inode);
--
2.41.0
_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
next prev parent reply other threads:[~2023-08-12 0:41 UTC|newest]
Thread overview: 65+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-08-12 0:41 [PATCH v5 00/10] Support negative dentries on case-insensitive ext4 and f2fs Gabriel Krisman Bertazi
2023-08-12 0:41 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 0:41 ` Gabriel Krisman Bertazi [this message]
2023-08-12 0:41 ` [f2fs-dev] [PATCH v5 01/10] fs: Expose helper to check if a directory needs casefolding Gabriel Krisman Bertazi
2023-08-12 1:59 ` Eric Biggers
2023-08-12 1:59 ` [f2fs-dev] " Eric Biggers
2023-08-12 23:06 ` Theodore Ts'o
2023-08-12 23:06 ` [f2fs-dev] " Theodore Ts'o
2023-08-13 0:12 ` Eric Biggers
2023-08-13 0:12 ` [f2fs-dev] " Eric Biggers
2023-08-13 3:08 ` Matthew Wilcox
2023-08-13 3:08 ` [f2fs-dev] " Matthew Wilcox
2023-08-13 4:30 ` Eric Biggers
2023-08-13 4:30 ` [f2fs-dev] " Eric Biggers
2023-08-14 11:38 ` Theodore Ts'o
2023-08-14 11:38 ` [f2fs-dev] " Theodore Ts'o
2023-08-14 17:22 ` Eric Biggers
2023-08-14 17:22 ` [f2fs-dev] " Eric Biggers
2023-08-15 3:59 ` Theodore Ts'o
2023-08-15 3:59 ` [f2fs-dev] " Theodore Ts'o
2023-08-14 15:02 ` Gabriel Krisman Bertazi
2023-08-14 15:02 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-14 19:14 ` Eric Biggers
2023-08-14 19:14 ` [f2fs-dev] " Eric Biggers
2023-08-14 19:26 ` Gabriel Krisman Bertazi
2023-08-14 19:26 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 0:41 ` [PATCH v5 02/10] ecryptfs: Reject casefold directory inodes Gabriel Krisman Bertazi
2023-08-12 0:41 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 0:41 ` [PATCH v5 03/10] 9p: Split ->weak_revalidate from ->revalidate Gabriel Krisman Bertazi
2023-08-12 0:41 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 0:41 ` [PATCH v5 04/10] fs: Expose name under lookup to d_revalidate hooks Gabriel Krisman Bertazi
2023-08-12 0:41 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 2:15 ` Eric Biggers
2023-08-12 2:15 ` [f2fs-dev] " Eric Biggers
2023-08-17 7:00 ` kernel test robot
2023-08-17 7:00 ` [f2fs-dev] " kernel test robot
2023-08-17 9:12 ` kernel test robot
2023-08-17 9:12 ` [f2fs-dev] " kernel test robot
2023-08-12 0:41 ` [PATCH v5 05/10] fs: Add DCACHE_CASEFOLDED_NAME flag Gabriel Krisman Bertazi
2023-08-12 0:41 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 2:17 ` Eric Biggers
2023-08-12 2:17 ` [f2fs-dev] " Eric Biggers
2023-08-14 15:03 ` Gabriel Krisman Bertazi
2023-08-14 15:03 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 9:33 ` kernel test robot
2023-08-12 0:41 ` [PATCH v5 06/10] libfs: Validate negative dentries in case-insensitive directories Gabriel Krisman Bertazi
2023-08-12 0:41 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 2:41 ` Eric Biggers
2023-08-12 2:41 ` [f2fs-dev] " Eric Biggers
2023-08-14 14:50 ` Gabriel Krisman Bertazi
2023-08-14 14:50 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-14 18:42 ` Eric Biggers
2023-08-14 18:42 ` [f2fs-dev] " Eric Biggers
2023-08-14 19:21 ` Gabriel Krisman Bertazi
2023-08-14 19:21 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-14 19:26 ` Eric Biggers
2023-08-14 19:26 ` [f2fs-dev] " Eric Biggers
2023-08-12 0:41 ` [PATCH v5 07/10] libfs: Chain encryption checks after case-insensitive revalidation Gabriel Krisman Bertazi
2023-08-12 0:41 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 0:41 ` [PATCH v5 08/10] libfs: Merge encrypted_ci_dentry_ops and ci_dentry_ops Gabriel Krisman Bertazi
2023-08-12 0:41 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 0:41 ` [PATCH v5 09/10] ext4: Enable negative dentries on case-insensitive lookup Gabriel Krisman Bertazi
2023-08-12 0:41 ` [f2fs-dev] " Gabriel Krisman Bertazi
2023-08-12 0:41 ` [PATCH v5 10/10] f2fs: " Gabriel Krisman Bertazi
2023-08-12 0:41 ` [f2fs-dev] " Gabriel Krisman Bertazi
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20230812004146.30980-2-krisman@suse.de \
--to=krisman@suse.de \
--cc=brauner@kernel.org \
--cc=ebiggers@kernel.org \
--cc=jaegeuk@kernel.org \
--cc=linux-ext4@vger.kernel.org \
--cc=linux-f2fs-devel@lists.sourceforge.net \
--cc=linux-fsdevel@vger.kernel.org \
--cc=tytso@mit.edu \
--cc=viro@zeniv.linux.org.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.