[PATCH] exfat: fix passing zero to ERR_PTR() in exfat_mkdir()

Yang Wen posted 1 patch 1 week, 5 days ago
fs/exfat/namei.c | 7 ++++---
1 file changed, 4 insertions(+), 3 deletions(-)
[PATCH] exfat: fix passing zero to ERR_PTR() in exfat_mkdir()
Posted by Yang Wen 1 week, 5 days ago
Detected by Smatch.

namei.c:890 exfat_mkdir() warn:
	passing zero to 'ERR_PTR'

Signed-off-by: Yang Wen <anmuxixixi@gmail.com>
---
 fs/exfat/namei.c | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/fs/exfat/namei.c b/fs/exfat/namei.c
index 670116ae9ec8..2bfdece79b17 100644
--- a/fs/exfat/namei.c
+++ b/fs/exfat/namei.c
@@ -873,9 +873,10 @@ static struct dentry *exfat_mkdir(struct mnt_idmap *idmap, struct inode *dir,
 
 	i_pos = exfat_make_i_pos(&info);
 	inode = exfat_build_inode(sb, &info, i_pos);
-	err = PTR_ERR_OR_ZERO(inode);
-	if (err)
+	if (IS_ERR(inode)) {
+		err = PTR_ERR(inode);
 		goto unlock;
+	}
 
 	inode_inc_iversion(inode);
 	EXFAT_I(inode)->i_crtime = simple_inode_init_ts(inode);
@@ -886,7 +887,7 @@ static struct dentry *exfat_mkdir(struct mnt_idmap *idmap, struct inode *dir,
 
 unlock:
 	mutex_unlock(&EXFAT_SB(sb)->s_lock);
-	return ERR_PTR(err);
+	return err ? ERR_PTR(err) : NULL;
 }
 
 static int exfat_check_dir_empty(struct super_block *sb,
-- 
2.43.0
Re: [PATCH] exfat: fix passing zero to ERR_PTR() in exfat_mkdir()
Posted by Namjae Jeon 1 week, 4 days ago
On Mon, Mar 23, 2026 at 12:32 AM Yang Wen <anmuxixixi@gmail.com> wrote:
>
> Detected by Smatch.
>
> namei.c:890 exfat_mkdir() warn:
>         passing zero to 'ERR_PTR'
>
> Signed-off-by: Yang Wen <anmuxixixi@gmail.com>
Applied it to #dev.
Thanks!