All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yang Yingliang <yangyingliang@huawei.com>
To: <linux-kernel@vger.kernel.org>, <qemu-devel@nongnu.org>,
	<linux-f2fs-devel@lists.sourceforge.net>,
	<linux-erofs@lists.ozlabs.org>, <ocfs2-devel@oss.oracle.com>,
	<linux-mtd@lists.infradead.org>, <amd-gfx@lists.freedesktop.org>
Cc: <gregkh@linuxfoundation.org>, <rafael@kernel.org>,
	<somlo@cmu.edu>, <mst@redhat.com>, <jaegeuk@kernel.org>,
	<chao@kernel.org>, <hsiangkao@linux.alibaba.com>,
	<huangjianan@oppo.com>, <mark@fasheh.com>, <jlbec@evilplan.org>,
	<joseph.qi@linux.alibaba.com>, <akpm@linux-foundation.org>,
	<alexander.deucher@amd.com>, <luben.tuikov@amd.com>,
	<richard@nod.at>, <liushixin2@huawei.com>
Subject: [PATCH 11/11] ubifs: Fix memory leak in ubifs_sysfs_init()
Date: Fri, 21 Oct 2022 10:21:02 +0800	[thread overview]
Message-ID: <20221021022102.2231464-12-yangyingliang@huawei.com> (raw)
In-Reply-To: <20221021022102.2231464-1-yangyingliang@huawei.com>

From: Liu Shixin <liushixin2@huawei.com>

When insmod ubifs.ko, a kmemleak reported as below:

 unreferenced object 0xffff88817fb1a780 (size 8):
   comm "insmod", pid 25265, jiffies 4295239702 (age 100.130s)
   hex dump (first 8 bytes):
     75 62 69 66 73 00 ff ff                          ubifs...
   backtrace:
     [<ffffffff81b3fc4c>] slab_post_alloc_hook+0x9c/0x3c0
     [<ffffffff81b44bf3>] __kmalloc_track_caller+0x183/0x410
     [<ffffffff8198d3da>] kstrdup+0x3a/0x80
     [<ffffffff8198d486>] kstrdup_const+0x66/0x80
     [<ffffffff83989325>] kvasprintf_const+0x155/0x190
     [<ffffffff83bf55bb>] kobject_set_name_vargs+0x5b/0x150
     [<ffffffff83bf576b>] kobject_set_name+0xbb/0xf0
     [<ffffffff8100204c>] do_one_initcall+0x14c/0x5a0
     [<ffffffff8157e380>] do_init_module+0x1f0/0x660
     [<ffffffff815857be>] load_module+0x6d7e/0x7590
     [<ffffffff8158644f>] __do_sys_finit_module+0x19f/0x230
     [<ffffffff815866b3>] __x64_sys_finit_module+0x73/0xb0
     [<ffffffff88c98e85>] do_syscall_64+0x35/0x80
     [<ffffffff88e00087>] entry_SYSCALL_64_after_hwframe+0x63/0xcd

When kset_register() failed, we should call kset_put to cleanup it.

Fixes: 2e3cbf425804 ("ubifs: Export filesystem error counters")
Signed-off-by: Liu Shixin <liushixin2@huawei.com>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
---
 fs/ubifs/sysfs.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/fs/ubifs/sysfs.c b/fs/ubifs/sysfs.c
index 06ad8fa1fcfb..54270ad36321 100644
--- a/fs/ubifs/sysfs.c
+++ b/fs/ubifs/sysfs.c
@@ -144,6 +144,8 @@ int __init ubifs_sysfs_init(void)
 	kobject_set_name(&ubifs_kset.kobj, "ubifs");
 	ubifs_kset.kobj.parent = fs_kobj;
 	ret = kset_register(&ubifs_kset);
+	if (ret)
+		kset_put(&ubifs_kset);
 
 	return ret;
 }
-- 
2.25.1


WARNING: multiple messages have this Message-ID (diff)
From: Yang Yingliang via Ocfs2-devel <ocfs2-devel@oss.oracle.com>
To: <linux-kernel@vger.kernel.org>, <qemu-devel@nongnu.org>,
	<linux-f2fs-devel@lists.sourceforge.net>,
	<linux-erofs@lists.ozlabs.org>, <ocfs2-devel@oss.oracle.com>,
	<linux-mtd@lists.infradead.org>, <amd-gfx@lists.freedesktop.org>
Cc: alexander.deucher@amd.com, richard@nod.at, mst@redhat.com,
	gregkh@linuxfoundation.org, somlo@cmu.edu, chao@kernel.org,
	huangjianan@oppo.com, liushixin2@huawei.com,
	luben.tuikov@amd.com, hsiangkao@linux.alibaba.com,
	rafael@kernel.org, jaegeuk@kernel.org
Subject: [Ocfs2-devel] [PATCH 11/11] ubifs: Fix memory leak in ubifs_sysfs_init()
Date: Fri, 21 Oct 2022 10:21:02 +0800	[thread overview]
Message-ID: <20221021022102.2231464-12-yangyingliang@huawei.com> (raw)
In-Reply-To: <20221021022102.2231464-1-yangyingliang@huawei.com>

From: Liu Shixin <liushixin2@huawei.com>

When insmod ubifs.ko, a kmemleak reported as below:

 unreferenced object 0xffff88817fb1a780 (size 8):
   comm "insmod", pid 25265, jiffies 4295239702 (age 100.130s)
   hex dump (first 8 bytes):
     75 62 69 66 73 00 ff ff                          ubifs...
   backtrace:
     [<ffffffff81b3fc4c>] slab_post_alloc_hook+0x9c/0x3c0
     [<ffffffff81b44bf3>] __kmalloc_track_caller+0x183/0x410
     [<ffffffff8198d3da>] kstrdup+0x3a/0x80
     [<ffffffff8198d486>] kstrdup_const+0x66/0x80
     [<ffffffff83989325>] kvasprintf_const+0x155/0x190
     [<ffffffff83bf55bb>] kobject_set_name_vargs+0x5b/0x150
     [<ffffffff83bf576b>] kobject_set_name+0xbb/0xf0
     [<ffffffff8100204c>] do_one_initcall+0x14c/0x5a0
     [<ffffffff8157e380>] do_init_module+0x1f0/0x660
     [<ffffffff815857be>] load_module+0x6d7e/0x7590
     [<ffffffff8158644f>] __do_sys_finit_module+0x19f/0x230
     [<ffffffff815866b3>] __x64_sys_finit_module+0x73/0xb0
     [<ffffffff88c98e85>] do_syscall_64+0x35/0x80
     [<ffffffff88e00087>] entry_SYSCALL_64_after_hwframe+0x63/0xcd

When kset_register() failed, we should call kset_put to cleanup it.

Fixes: 2e3cbf425804 ("ubifs: Export filesystem error counters")
Signed-off-by: Liu Shixin <liushixin2@huawei.com>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
---
 fs/ubifs/sysfs.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/fs/ubifs/sysfs.c b/fs/ubifs/sysfs.c
index 06ad8fa1fcfb..54270ad36321 100644
--- a/fs/ubifs/sysfs.c
+++ b/fs/ubifs/sysfs.c
@@ -144,6 +144,8 @@ int __init ubifs_sysfs_init(void)
 	kobject_set_name(&ubifs_kset.kobj, "ubifs");
 	ubifs_kset.kobj.parent = fs_kobj;
 	ret = kset_register(&ubifs_kset);
+	if (ret)
+		kset_put(&ubifs_kset);
 
 	return ret;
 }
-- 
2.25.1


_______________________________________________
Ocfs2-devel mailing list
Ocfs2-devel@oss.oracle.com
https://oss.oracle.com/mailman/listinfo/ocfs2-devel

WARNING: multiple messages have this Message-ID (diff)
From: Yang Yingliang via Linux-f2fs-devel <linux-f2fs-devel@lists.sourceforge.net>
To: <linux-kernel@vger.kernel.org>, <qemu-devel@nongnu.org>,
	<linux-f2fs-devel@lists.sourceforge.net>,
	<linux-erofs@lists.ozlabs.org>, <ocfs2-devel@oss.oracle.com>,
	<linux-mtd@lists.infradead.org>, <amd-gfx@lists.freedesktop.org>
Cc: alexander.deucher@amd.com, richard@nod.at, mst@redhat.com,
	gregkh@linuxfoundation.org, somlo@cmu.edu, liushixin2@huawei.com,
	joseph.qi@linux.alibaba.com, luben.tuikov@amd.com,
	jlbec@evilplan.org, hsiangkao@linux.alibaba.com,
	rafael@kernel.org, jaegeuk@kernel.org, akpm@linux-foundation.org,
	mark@fasheh.com
Subject: [f2fs-dev] [PATCH 11/11] ubifs: Fix memory leak in ubifs_sysfs_init()
Date: Fri, 21 Oct 2022 10:21:02 +0800	[thread overview]
Message-ID: <20221021022102.2231464-12-yangyingliang@huawei.com> (raw)
In-Reply-To: <20221021022102.2231464-1-yangyingliang@huawei.com>

From: Liu Shixin <liushixin2@huawei.com>

When insmod ubifs.ko, a kmemleak reported as below:

 unreferenced object 0xffff88817fb1a780 (size 8):
   comm "insmod", pid 25265, jiffies 4295239702 (age 100.130s)
   hex dump (first 8 bytes):
     75 62 69 66 73 00 ff ff                          ubifs...
   backtrace:
     [<ffffffff81b3fc4c>] slab_post_alloc_hook+0x9c/0x3c0
     [<ffffffff81b44bf3>] __kmalloc_track_caller+0x183/0x410
     [<ffffffff8198d3da>] kstrdup+0x3a/0x80
     [<ffffffff8198d486>] kstrdup_const+0x66/0x80
     [<ffffffff83989325>] kvasprintf_const+0x155/0x190
     [<ffffffff83bf55bb>] kobject_set_name_vargs+0x5b/0x150
     [<ffffffff83bf576b>] kobject_set_name+0xbb/0xf0
     [<ffffffff8100204c>] do_one_initcall+0x14c/0x5a0
     [<ffffffff8157e380>] do_init_module+0x1f0/0x660
     [<ffffffff815857be>] load_module+0x6d7e/0x7590
     [<ffffffff8158644f>] __do_sys_finit_module+0x19f/0x230
     [<ffffffff815866b3>] __x64_sys_finit_module+0x73/0xb0
     [<ffffffff88c98e85>] do_syscall_64+0x35/0x80
     [<ffffffff88e00087>] entry_SYSCALL_64_after_hwframe+0x63/0xcd

When kset_register() failed, we should call kset_put to cleanup it.

Fixes: 2e3cbf425804 ("ubifs: Export filesystem error counters")
Signed-off-by: Liu Shixin <liushixin2@huawei.com>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
---
 fs/ubifs/sysfs.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/fs/ubifs/sysfs.c b/fs/ubifs/sysfs.c
index 06ad8fa1fcfb..54270ad36321 100644
--- a/fs/ubifs/sysfs.c
+++ b/fs/ubifs/sysfs.c
@@ -144,6 +144,8 @@ int __init ubifs_sysfs_init(void)
 	kobject_set_name(&ubifs_kset.kobj, "ubifs");
 	ubifs_kset.kobj.parent = fs_kobj;
 	ret = kset_register(&ubifs_kset);
+	if (ret)
+		kset_put(&ubifs_kset);
 
 	return ret;
 }
-- 
2.25.1



_______________________________________________
Linux-f2fs-devel mailing list
Linux-f2fs-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel

WARNING: multiple messages have this Message-ID (diff)
From: Yang Yingliang <yangyingliang@huawei.com>
To: <linux-kernel@vger.kernel.org>, <qemu-devel@nongnu.org>,
	<linux-f2fs-devel@lists.sourceforge.net>,
	<linux-erofs@lists.ozlabs.org>, <ocfs2-devel@oss.oracle.com>,
	<linux-mtd@lists.infradead.org>, <amd-gfx@lists.freedesktop.org>
Cc: <gregkh@linuxfoundation.org>, <rafael@kernel.org>,
	<somlo@cmu.edu>, <mst@redhat.com>, <jaegeuk@kernel.org>,
	<chao@kernel.org>, <hsiangkao@linux.alibaba.com>,
	<huangjianan@oppo.com>, <mark@fasheh.com>, <jlbec@evilplan.org>,
	<joseph.qi@linux.alibaba.com>, <akpm@linux-foundation.org>,
	<alexander.deucher@amd.com>, <luben.tuikov@amd.com>,
	<richard@nod.at>, <liushixin2@huawei.com>
Subject: [PATCH 11/11] ubifs: Fix memory leak in ubifs_sysfs_init()
Date: Fri, 21 Oct 2022 10:21:02 +0800	[thread overview]
Message-ID: <20221021022102.2231464-12-yangyingliang@huawei.com> (raw)
In-Reply-To: <20221021022102.2231464-1-yangyingliang@huawei.com>

From: Liu Shixin <liushixin2@huawei.com>

When insmod ubifs.ko, a kmemleak reported as below:

 unreferenced object 0xffff88817fb1a780 (size 8):
   comm "insmod", pid 25265, jiffies 4295239702 (age 100.130s)
   hex dump (first 8 bytes):
     75 62 69 66 73 00 ff ff                          ubifs...
   backtrace:
     [<ffffffff81b3fc4c>] slab_post_alloc_hook+0x9c/0x3c0
     [<ffffffff81b44bf3>] __kmalloc_track_caller+0x183/0x410
     [<ffffffff8198d3da>] kstrdup+0x3a/0x80
     [<ffffffff8198d486>] kstrdup_const+0x66/0x80
     [<ffffffff83989325>] kvasprintf_const+0x155/0x190
     [<ffffffff83bf55bb>] kobject_set_name_vargs+0x5b/0x150
     [<ffffffff83bf576b>] kobject_set_name+0xbb/0xf0
     [<ffffffff8100204c>] do_one_initcall+0x14c/0x5a0
     [<ffffffff8157e380>] do_init_module+0x1f0/0x660
     [<ffffffff815857be>] load_module+0x6d7e/0x7590
     [<ffffffff8158644f>] __do_sys_finit_module+0x19f/0x230
     [<ffffffff815866b3>] __x64_sys_finit_module+0x73/0xb0
     [<ffffffff88c98e85>] do_syscall_64+0x35/0x80
     [<ffffffff88e00087>] entry_SYSCALL_64_after_hwframe+0x63/0xcd

When kset_register() failed, we should call kset_put to cleanup it.

Fixes: 2e3cbf425804 ("ubifs: Export filesystem error counters")
Signed-off-by: Liu Shixin <liushixin2@huawei.com>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
---
 fs/ubifs/sysfs.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/fs/ubifs/sysfs.c b/fs/ubifs/sysfs.c
index 06ad8fa1fcfb..54270ad36321 100644
--- a/fs/ubifs/sysfs.c
+++ b/fs/ubifs/sysfs.c
@@ -144,6 +144,8 @@ int __init ubifs_sysfs_init(void)
 	kobject_set_name(&ubifs_kset.kobj, "ubifs");
 	ubifs_kset.kobj.parent = fs_kobj;
 	ret = kset_register(&ubifs_kset);
+	if (ret)
+		kset_put(&ubifs_kset);
 
 	return ret;
 }
-- 
2.25.1


______________________________________________________
Linux MTD discussion mailing list
http://lists.infradead.org/mailman/listinfo/linux-mtd/

WARNING: multiple messages have this Message-ID (diff)
From: Yang Yingliang via Linux-erofs <linux-erofs@lists.ozlabs.org>
To: <linux-kernel@vger.kernel.org>, <qemu-devel@nongnu.org>,
	<linux-f2fs-devel@lists.sourceforge.net>,
	<linux-erofs@lists.ozlabs.org>, <ocfs2-devel@oss.oracle.com>,
	<linux-mtd@lists.infradead.org>, <amd-gfx@lists.freedesktop.org>
Cc: alexander.deucher@amd.com, richard@nod.at, mst@redhat.com,
	gregkh@linuxfoundation.org, somlo@cmu.edu, huangjianan@oppo.com,
	liushixin2@huawei.com, joseph.qi@linux.alibaba.com,
	luben.tuikov@amd.com, jlbec@evilplan.org,
	hsiangkao@linux.alibaba.com, rafael@kernel.org,
	jaegeuk@kernel.org, akpm@linux-foundation.org, mark@fasheh.com
Subject: [PATCH 11/11] ubifs: Fix memory leak in ubifs_sysfs_init()
Date: Fri, 21 Oct 2022 10:21:02 +0800	[thread overview]
Message-ID: <20221021022102.2231464-12-yangyingliang@huawei.com> (raw)
In-Reply-To: <20221021022102.2231464-1-yangyingliang@huawei.com>

From: Liu Shixin <liushixin2@huawei.com>

When insmod ubifs.ko, a kmemleak reported as below:

 unreferenced object 0xffff88817fb1a780 (size 8):
   comm "insmod", pid 25265, jiffies 4295239702 (age 100.130s)
   hex dump (first 8 bytes):
     75 62 69 66 73 00 ff ff                          ubifs...
   backtrace:
     [<ffffffff81b3fc4c>] slab_post_alloc_hook+0x9c/0x3c0
     [<ffffffff81b44bf3>] __kmalloc_track_caller+0x183/0x410
     [<ffffffff8198d3da>] kstrdup+0x3a/0x80
     [<ffffffff8198d486>] kstrdup_const+0x66/0x80
     [<ffffffff83989325>] kvasprintf_const+0x155/0x190
     [<ffffffff83bf55bb>] kobject_set_name_vargs+0x5b/0x150
     [<ffffffff83bf576b>] kobject_set_name+0xbb/0xf0
     [<ffffffff8100204c>] do_one_initcall+0x14c/0x5a0
     [<ffffffff8157e380>] do_init_module+0x1f0/0x660
     [<ffffffff815857be>] load_module+0x6d7e/0x7590
     [<ffffffff8158644f>] __do_sys_finit_module+0x19f/0x230
     [<ffffffff815866b3>] __x64_sys_finit_module+0x73/0xb0
     [<ffffffff88c98e85>] do_syscall_64+0x35/0x80
     [<ffffffff88e00087>] entry_SYSCALL_64_after_hwframe+0x63/0xcd

When kset_register() failed, we should call kset_put to cleanup it.

Fixes: 2e3cbf425804 ("ubifs: Export filesystem error counters")
Signed-off-by: Liu Shixin <liushixin2@huawei.com>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
---
 fs/ubifs/sysfs.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/fs/ubifs/sysfs.c b/fs/ubifs/sysfs.c
index 06ad8fa1fcfb..54270ad36321 100644
--- a/fs/ubifs/sysfs.c
+++ b/fs/ubifs/sysfs.c
@@ -144,6 +144,8 @@ int __init ubifs_sysfs_init(void)
 	kobject_set_name(&ubifs_kset.kobj, "ubifs");
 	ubifs_kset.kobj.parent = fs_kobj;
 	ret = kset_register(&ubifs_kset);
+	if (ret)
+		kset_put(&ubifs_kset);
 
 	return ret;
 }
-- 
2.25.1


WARNING: multiple messages have this Message-ID (diff)
From: Yang Yingliang via <qemu-devel@nongnu.org>
To: <linux-kernel@vger.kernel.org>, <qemu-devel@nongnu.org>,
	<linux-f2fs-devel@lists.sourceforge.net>,
	<linux-erofs@lists.ozlabs.org>, <ocfs2-devel@oss.oracle.com>,
	<linux-mtd@lists.infradead.org>, <amd-gfx@lists.freedesktop.org>
Cc: <gregkh@linuxfoundation.org>, <rafael@kernel.org>,
	<somlo@cmu.edu>, <mst@redhat.com>, <jaegeuk@kernel.org>,
	<chao@kernel.org>, <hsiangkao@linux.alibaba.com>,
	<huangjianan@oppo.com>, <mark@fasheh.com>, <jlbec@evilplan.org>,
	<joseph.qi@linux.alibaba.com>, <akpm@linux-foundation.org>,
	<alexander.deucher@amd.com>, <luben.tuikov@amd.com>,
	<richard@nod.at>, <liushixin2@huawei.com>
Subject: [PATCH 11/11] ubifs: Fix memory leak in ubifs_sysfs_init()
Date: Fri, 21 Oct 2022 10:21:02 +0800	[thread overview]
Message-ID: <20221021022102.2231464-12-yangyingliang@huawei.com> (raw)
In-Reply-To: <20221021022102.2231464-1-yangyingliang@huawei.com>

From: Liu Shixin <liushixin2@huawei.com>

When insmod ubifs.ko, a kmemleak reported as below:

 unreferenced object 0xffff88817fb1a780 (size 8):
   comm "insmod", pid 25265, jiffies 4295239702 (age 100.130s)
   hex dump (first 8 bytes):
     75 62 69 66 73 00 ff ff                          ubifs...
   backtrace:
     [<ffffffff81b3fc4c>] slab_post_alloc_hook+0x9c/0x3c0
     [<ffffffff81b44bf3>] __kmalloc_track_caller+0x183/0x410
     [<ffffffff8198d3da>] kstrdup+0x3a/0x80
     [<ffffffff8198d486>] kstrdup_const+0x66/0x80
     [<ffffffff83989325>] kvasprintf_const+0x155/0x190
     [<ffffffff83bf55bb>] kobject_set_name_vargs+0x5b/0x150
     [<ffffffff83bf576b>] kobject_set_name+0xbb/0xf0
     [<ffffffff8100204c>] do_one_initcall+0x14c/0x5a0
     [<ffffffff8157e380>] do_init_module+0x1f0/0x660
     [<ffffffff815857be>] load_module+0x6d7e/0x7590
     [<ffffffff8158644f>] __do_sys_finit_module+0x19f/0x230
     [<ffffffff815866b3>] __x64_sys_finit_module+0x73/0xb0
     [<ffffffff88c98e85>] do_syscall_64+0x35/0x80
     [<ffffffff88e00087>] entry_SYSCALL_64_after_hwframe+0x63/0xcd

When kset_register() failed, we should call kset_put to cleanup it.

Fixes: 2e3cbf425804 ("ubifs: Export filesystem error counters")
Signed-off-by: Liu Shixin <liushixin2@huawei.com>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
---
 fs/ubifs/sysfs.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/fs/ubifs/sysfs.c b/fs/ubifs/sysfs.c
index 06ad8fa1fcfb..54270ad36321 100644
--- a/fs/ubifs/sysfs.c
+++ b/fs/ubifs/sysfs.c
@@ -144,6 +144,8 @@ int __init ubifs_sysfs_init(void)
 	kobject_set_name(&ubifs_kset.kobj, "ubifs");
 	ubifs_kset.kobj.parent = fs_kobj;
 	ret = kset_register(&ubifs_kset);
+	if (ret)
+		kset_put(&ubifs_kset);
 
 	return ret;
 }
-- 
2.25.1



WARNING: multiple messages have this Message-ID (diff)
From: Yang Yingliang <yangyingliang@huawei.com>
To: <linux-kernel@vger.kernel.org>, <qemu-devel@nongnu.org>,
	<linux-f2fs-devel@lists.sourceforge.net>,
	<linux-erofs@lists.ozlabs.org>, <ocfs2-devel@oss.oracle.com>,
	<linux-mtd@lists.infradead.org>, <amd-gfx@lists.freedesktop.org>
Cc: alexander.deucher@amd.com, richard@nod.at, mst@redhat.com,
	gregkh@linuxfoundation.org, somlo@cmu.edu, chao@kernel.org,
	huangjianan@oppo.com, liushixin2@huawei.com,
	joseph.qi@linux.alibaba.com, luben.tuikov@amd.com,
	jlbec@evilplan.org, hsiangkao@linux.alibaba.com,
	rafael@kernel.org, jaegeuk@kernel.org, akpm@linux-foundation.org,
	mark@fasheh.com
Subject: [PATCH 11/11] ubifs: Fix memory leak in ubifs_sysfs_init()
Date: Fri, 21 Oct 2022 10:21:02 +0800	[thread overview]
Message-ID: <20221021022102.2231464-12-yangyingliang@huawei.com> (raw)
In-Reply-To: <20221021022102.2231464-1-yangyingliang@huawei.com>

From: Liu Shixin <liushixin2@huawei.com>

When insmod ubifs.ko, a kmemleak reported as below:

 unreferenced object 0xffff88817fb1a780 (size 8):
   comm "insmod", pid 25265, jiffies 4295239702 (age 100.130s)
   hex dump (first 8 bytes):
     75 62 69 66 73 00 ff ff                          ubifs...
   backtrace:
     [<ffffffff81b3fc4c>] slab_post_alloc_hook+0x9c/0x3c0
     [<ffffffff81b44bf3>] __kmalloc_track_caller+0x183/0x410
     [<ffffffff8198d3da>] kstrdup+0x3a/0x80
     [<ffffffff8198d486>] kstrdup_const+0x66/0x80
     [<ffffffff83989325>] kvasprintf_const+0x155/0x190
     [<ffffffff83bf55bb>] kobject_set_name_vargs+0x5b/0x150
     [<ffffffff83bf576b>] kobject_set_name+0xbb/0xf0
     [<ffffffff8100204c>] do_one_initcall+0x14c/0x5a0
     [<ffffffff8157e380>] do_init_module+0x1f0/0x660
     [<ffffffff815857be>] load_module+0x6d7e/0x7590
     [<ffffffff8158644f>] __do_sys_finit_module+0x19f/0x230
     [<ffffffff815866b3>] __x64_sys_finit_module+0x73/0xb0
     [<ffffffff88c98e85>] do_syscall_64+0x35/0x80
     [<ffffffff88e00087>] entry_SYSCALL_64_after_hwframe+0x63/0xcd

When kset_register() failed, we should call kset_put to cleanup it.

Fixes: 2e3cbf425804 ("ubifs: Export filesystem error counters")
Signed-off-by: Liu Shixin <liushixin2@huawei.com>
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
---
 fs/ubifs/sysfs.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/fs/ubifs/sysfs.c b/fs/ubifs/sysfs.c
index 06ad8fa1fcfb..54270ad36321 100644
--- a/fs/ubifs/sysfs.c
+++ b/fs/ubifs/sysfs.c
@@ -144,6 +144,8 @@ int __init ubifs_sysfs_init(void)
 	kobject_set_name(&ubifs_kset.kobj, "ubifs");
 	ubifs_kset.kobj.parent = fs_kobj;
 	ret = kset_register(&ubifs_kset);
+	if (ret)
+		kset_put(&ubifs_kset);
 
 	return ret;
 }
-- 
2.25.1


  parent reply	other threads:[~2022-10-21  2:23 UTC|newest]

Thread overview: 216+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-10-21  2:20 [PATCH 00/11] fix memory leak while kset_register() fails Yang Yingliang
2022-10-21  2:20 ` Yang Yingliang
2022-10-21  2:20 ` Yang Yingliang via
2022-10-21  2:20 ` Yang Yingliang via Linux-erofs
2022-10-21  2:20 ` Yang Yingliang
2022-10-21  2:20 ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:20 ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  2:20 ` [PATCH 01/11] kset: fix documentation for kset_register() Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang via
2022-10-21  2:20   ` Yang Yingliang via Linux-erofs
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:20   ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  5:34   ` Luben Tuikov
2022-10-21  5:34     ` [Ocfs2-devel] " Luben Tuikov via Ocfs2-devel
2022-10-21  5:34     ` [f2fs-dev] " Luben Tuikov via Linux-f2fs-devel
2022-10-21  5:34     ` Luben Tuikov
2022-10-21  5:34     ` Luben Tuikov via Linux-erofs
2022-10-21  5:34     ` Luben Tuikov
2022-10-21  8:05     ` Yang Yingliang
2022-10-21  8:05       ` Yang Yingliang
2022-10-21  8:05       ` Yang Yingliang via
2022-10-21  8:05       ` Yang Yingliang via Linux-erofs
2022-10-21  8:05       ` Yang Yingliang
2022-10-21  8:05       ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  8:05       ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  8:16       ` Greg KH
2022-10-21  8:16         ` Greg KH
2022-10-21  8:16         ` Greg KH
2022-10-21  8:16         ` Greg KH
2022-10-21  8:16         ` [f2fs-dev] " Greg KH
2022-10-21  8:16         ` [Ocfs2-devel] " Greg KH via Ocfs2-devel
2022-10-21  8:18       ` Luben Tuikov
2022-10-21  8:18         ` [Ocfs2-devel] " Luben Tuikov via Ocfs2-devel
2022-10-21  8:18         ` Luben Tuikov
2022-10-21  8:18         ` Luben Tuikov via Linux-erofs
2022-10-21  8:18         ` Luben Tuikov
2022-10-21  8:18         ` [f2fs-dev] " Luben Tuikov via Linux-f2fs-devel
2022-10-21  2:20 ` [PATCH 02/11] kset: add null pointer check in kset_put() Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang via
2022-10-21  2:20   ` Yang Yingliang via Linux-erofs
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:20   ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  2:20 ` [PATCH 03/11] bus: fix possible memory leak in bus_register() Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang via
2022-10-21  2:20   ` Yang Yingliang via Linux-erofs
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:20   ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  2:20 ` [PATCH 04/11] kobject: fix possible memory leak in kset_create_and_add() Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang via
2022-10-21  2:20   ` Yang Yingliang via Linux-erofs
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:20   ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  2:20 ` [Ocfs2-devel] [PATCH 05/11] class: fix possible memory leak in __class_register() Yang Yingliang via Ocfs2-devel
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang via
2022-10-21  2:20   ` Yang Yingliang via Linux-erofs
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:20 ` [PATCH 06/11] firmware: qemu_fw_cfg: fix possible memory leak in fw_cfg_build_symlink() Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang via
2022-10-21  2:20   ` Yang Yingliang via Linux-erofs
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:20   ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  2:20 ` [PATCH 07/11] f2fs: fix possible memory leak in f2fs_init_sysfs() Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang via
2022-10-21  2:20   ` Yang Yingliang via Linux-erofs
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:20   ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  2:20 ` [PATCH 08/11] erofs: fix possible memory leak in erofs_init_sysfs() Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang via
2022-10-21  2:20   ` Yang Yingliang
2022-10-21  2:20   ` Yang Yingliang via Linux-erofs
2022-10-21  2:20   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:20   ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  2:21 ` [Ocfs2-devel] [PATCH 09/11] ocfs2: possible memory leak in mlog_sys_init() Yang Yingliang via Ocfs2-devel
2022-10-21  2:21   ` Yang Yingliang
2022-10-21  2:21   ` Yang Yingliang via Linux-erofs
2022-10-21  2:21   ` Yang Yingliang via
2022-10-21  2:21   ` Yang Yingliang
2022-10-21  2:21   ` Yang Yingliang
2022-10-21  2:21   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:21 ` [PATCH 10/11] drm/amdgpu/discovery: fix possible memory leak Yang Yingliang
2022-10-21  2:21   ` Yang Yingliang
2022-10-21  2:21   ` Yang Yingliang via
2022-10-21  2:21   ` Yang Yingliang via Linux-erofs
2022-10-21  2:21   ` Yang Yingliang
2022-10-21  2:21   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:21   ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  2:21 ` Yang Yingliang [this message]
2022-10-21  2:21   ` [PATCH 11/11] ubifs: Fix memory leak in ubifs_sysfs_init() Yang Yingliang
2022-10-21  2:21   ` Yang Yingliang via
2022-10-21  2:21   ` Yang Yingliang via Linux-erofs
2022-10-21  2:21   ` Yang Yingliang
2022-10-21  2:21   ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  2:21   ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  5:29 ` [PATCH 00/11] fix memory leak while kset_register() fails Luben Tuikov
2022-10-21  5:29   ` [Ocfs2-devel] " Luben Tuikov via Ocfs2-devel
2022-10-21  5:29   ` [f2fs-dev] " Luben Tuikov via Linux-f2fs-devel
2022-10-21  5:29   ` Luben Tuikov via Linux-erofs
2022-10-21  5:29   ` Luben Tuikov
2022-10-21  5:29   ` Luben Tuikov
2022-10-21  5:37   ` Greg KH
2022-10-21  5:37     ` Greg KH
2022-10-21  5:37     ` Greg KH
2022-10-21  5:37     ` Greg KH
2022-10-21  5:37     ` [f2fs-dev] " Greg KH
2022-10-21  5:37     ` [Ocfs2-devel] " Greg KH via Ocfs2-devel
2022-10-21  7:55     ` Luben Tuikov
2022-10-21  7:55       ` [Ocfs2-devel] " Luben Tuikov via Ocfs2-devel
2022-10-21  7:55       ` Luben Tuikov
2022-10-21  7:55       ` Luben Tuikov via Linux-erofs
2022-10-21  7:55       ` Luben Tuikov
2022-10-21  8:18       ` Greg KH
2022-10-21  8:18         ` Greg KH
2022-10-21  8:18         ` Greg KH
2022-10-21  8:18         ` Greg KH
2022-10-21  8:18         ` [f2fs-dev] " Greg KH
2022-10-21  8:18         ` [Ocfs2-devel] " Greg KH via Ocfs2-devel
2022-10-21  8:24         ` Luben Tuikov
2022-10-21  8:24           ` [Ocfs2-devel] " Luben Tuikov via Ocfs2-devel
2022-10-21  8:24           ` [f2fs-dev] " Luben Tuikov via Linux-f2fs-devel
2022-10-21  8:24           ` Luben Tuikov
2022-10-21  8:24           ` Luben Tuikov
2022-10-21  8:24           ` Luben Tuikov via Linux-erofs
2022-10-21  8:41           ` Luben Tuikov
2022-10-21  8:41             ` [Ocfs2-devel] " Luben Tuikov via Ocfs2-devel
2022-10-21  8:41             ` [f2fs-dev] " Luben Tuikov via Linux-f2fs-devel
2022-10-21  8:41             ` Luben Tuikov via Linux-erofs
2022-10-21  8:41             ` Luben Tuikov
2022-10-21  8:41             ` Luben Tuikov
2022-10-21  9:23             ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  9:23               ` Yang Yingliang
2022-10-21  9:23               ` Yang Yingliang via
2022-10-21  9:23               ` Yang Yingliang
2022-10-21  9:23               ` Yang Yingliang via Linux-erofs
2022-10-21  9:23               ` Yang Yingliang
2022-10-21  9:23               ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  8:24     ` Yang Yingliang
2022-10-21  8:24       ` Yang Yingliang
2022-10-21  8:24       ` Yang Yingliang via
2022-10-21  8:24       ` Yang Yingliang
2022-10-21  8:24       ` Yang Yingliang via Linux-erofs
2022-10-21  8:24       ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  8:24       ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  8:36       ` Greg KH
2022-10-21  8:36         ` Greg KH
2022-10-21  8:36         ` Greg KH
2022-10-21  8:36         ` Greg KH
2022-10-21  8:36         ` [f2fs-dev] " Greg KH
2022-10-21  8:36         ` [Ocfs2-devel] " Greg KH via Ocfs2-devel
2022-10-21  8:52         ` Luben Tuikov
2022-10-21  8:52           ` [Ocfs2-devel] " Luben Tuikov via Ocfs2-devel
2022-10-21  8:52           ` Luben Tuikov
2022-10-21  8:52           ` Luben Tuikov via Linux-erofs
2022-10-21  8:52           ` Luben Tuikov
2022-10-21  8:52           ` [f2fs-dev] " Luben Tuikov via Linux-f2fs-devel
2022-10-21  8:59         ` Yang Yingliang
2022-10-21  8:59           ` Yang Yingliang
2022-10-21  8:59           ` Yang Yingliang via
2022-10-21  8:59           ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  8:59           ` Yang Yingliang via Linux-erofs
2022-10-21  8:59           ` Yang Yingliang
2022-10-21  8:59           ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  9:08           ` Luben Tuikov
2022-10-21  9:08             ` [Ocfs2-devel] " Luben Tuikov via Ocfs2-devel
2022-10-21  9:08             ` Luben Tuikov
2022-10-21  9:08             ` Luben Tuikov via Linux-erofs
2022-10-21  9:08             ` Luben Tuikov
2022-10-21  9:08             ` [f2fs-dev] " Luben Tuikov via Linux-f2fs-devel
2022-10-21  9:56             ` Yang Yingliang
2022-10-21  9:56               ` Yang Yingliang
2022-10-21  9:56               ` Yang Yingliang via
2022-10-21  9:56               ` Yang Yingliang via Linux-erofs
2022-10-21  9:56               ` Yang Yingliang
2022-10-21  9:56               ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  9:56               ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21 23:45               ` Luben Tuikov
2022-10-21 23:45                 ` Luben Tuikov
2022-10-21 23:45                 ` Luben Tuikov via Linux-erofs
2022-10-21 23:45                 ` Luben Tuikov
2022-10-21 23:45                 ` [f2fs-dev] " Luben Tuikov via Linux-f2fs-devel
2022-10-21 23:45                 ` [Ocfs2-devel] " Luben Tuikov via Ocfs2-devel
2022-10-21  9:12         ` Yang Yingliang
2022-10-21  9:12           ` Yang Yingliang
2022-10-21  9:12           ` Yang Yingliang via
2022-10-21  9:12           ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel
2022-10-21  9:12           ` Yang Yingliang
2022-10-21  9:12           ` Yang Yingliang via Linux-erofs
2022-10-21  9:12           ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21 23:48           ` Luben Tuikov
2022-10-21 23:48             ` [f2fs-dev] " Luben Tuikov via Linux-f2fs-devel
2022-10-21 23:48             ` Luben Tuikov via Linux-erofs
2022-10-21 23:48             ` Luben Tuikov
2022-10-21 23:48             ` [Ocfs2-devel] " Luben Tuikov via Ocfs2-devel
2022-10-21 23:48             ` Luben Tuikov
2022-10-21  7:25   ` Yang Yingliang
2022-10-21  7:25     ` Yang Yingliang
2022-10-21  7:25     ` Yang Yingliang via
2022-10-21  7:25     ` Yang Yingliang
2022-10-21  7:25     ` Yang Yingliang via Linux-erofs
2022-10-21  7:25     ` [f2fs-dev] " Yang Yingliang via Linux-f2fs-devel
2022-10-21  7:25     ` [Ocfs2-devel] " Yang Yingliang via Ocfs2-devel

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=20221021022102.2231464-12-yangyingliang@huawei.com \
    --to=yangyingliang@huawei.com \
    --cc=akpm@linux-foundation.org \
    --cc=alexander.deucher@amd.com \
    --cc=amd-gfx@lists.freedesktop.org \
    --cc=chao@kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=hsiangkao@linux.alibaba.com \
    --cc=huangjianan@oppo.com \
    --cc=jaegeuk@kernel.org \
    --cc=jlbec@evilplan.org \
    --cc=joseph.qi@linux.alibaba.com \
    --cc=linux-erofs@lists.ozlabs.org \
    --cc=linux-f2fs-devel@lists.sourceforge.net \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mtd@lists.infradead.org \
    --cc=liushixin2@huawei.com \
    --cc=luben.tuikov@amd.com \
    --cc=mark@fasheh.com \
    --cc=mst@redhat.com \
    --cc=ocfs2-devel@oss.oracle.com \
    --cc=qemu-devel@nongnu.org \
    --cc=rafael@kernel.org \
    --cc=richard@nod.at \
    --cc=somlo@cmu.edu \
    /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.