All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] configfs: fix refcount warning in configfs_get_config_item()
@ 2026-07-29  8:55 Joshua Crofts
  2026-07-29  9:00 ` Joshua Crofts
  2026-07-29 10:06 ` Breno Leitao
  0 siblings, 2 replies; 7+ messages in thread
From: Joshua Crofts @ 2026-07-29  8:55 UTC (permalink / raw)
  To: Andreas Hindborg, Breno Leitao; +Cc: linux-kernel, syzbot+35790eb7861f8fc57382

syzbot reported a "refcount_t: addition on 0; use-after-free" warning
in configfs_get_config_item().

This occurs when configfs_get_config_item() races with a concurrent
teardown (e.g. rmdir). When the target config_item's refcount drops to
0, configfs_get_config_item() calls config_item_get(), which
unconditionally increments the refcount via kref_get(), triggering the
refcount warning.

Fix this by using config_item_get_unless_zero(), which safely returns
NULL if the refcount is already 0.

Reported-by: syzbot+35790eb7861f8fc57382@syzkaller.appspotmail.com
Closes: https://syzkaller.appspot.com/bug?extid=35790eb7861f8fc57382
Signed-off-by: Joshua Crofts <joshua.crofts1@gmail.com>
---
 fs/configfs/configfs_internal.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/configfs/configfs_internal.h b/fs/configfs/configfs_internal.h
index acdeea8e2..5448af3a8 100644
--- a/fs/configfs/configfs_internal.h
+++ b/fs/configfs/configfs_internal.h
@@ -124,7 +124,7 @@ static inline struct config_item *configfs_get_config_item(struct dentry *dentry
 	spin_lock(&dentry->d_lock);
 	if (!d_unhashed(dentry)) {
 		struct configfs_dirent * sd = dentry->d_fsdata;
-		item = config_item_get(sd->s_element);
+		item = config_item_get_unless_zero(sd->s_element);
 	}
 	spin_unlock(&dentry->d_lock);
 
-- 
2.47.3


^ permalink raw reply related	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2026-07-29 15:06 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-07-29  8:55 [PATCH] configfs: fix refcount warning in configfs_get_config_item() Joshua Crofts
2026-07-29  9:00 ` Joshua Crofts
2026-07-29  9:51   ` Jeffin Philip
2026-07-29 13:50     ` Joshua Crofts
2026-07-29 10:06 ` Breno Leitao
2026-07-29 13:31   ` Joshua Crofts
2026-07-29 15:05     ` Jeffin Philip

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.