fuse: Fix memory leak in fuse_dev_free()
Subsystems (2)
PATCH 1/1 fuse: Fix memory leak in fuse_dev_free()
When ntfs is unmounted, the following leak is
reported by kmemleak.
kmemleak report:
unreferenced object 0xffff880052bf4400 (size 4096):
comm "mount.ntfs", pid 16530, jiffies 4294861127 (age 3215.836s) hex dump (first 32 bytes):
00 44 bf 52 00 88 ff ff 00 44 bf 52 00 88 ff ff .D.R.....D.R....
10 44 bf 52 00 88 ff ff 10 44 bf 52 00 88 ff ff .D.R.....D.R....
backtrace:
[<00000000bf4a2f8d>] fuse_fill_super+0xb22/0x1da0 [fuse]
[<000000004dde0f0c>] mount_bdev+0x263/0x320
[<0000000025aebc66>] mount_fs+0x82/0x2bf
[<0000000042c5a6be>] vfs_kern_mount.part.33+0xbf/0x480
[<00000000ed10cd5b>] do_mount+0x3de/0x2ad0
[<00000000d59ff068>] ksys_mount+0xba/0xd0
[<000000001bda1bcc>] __x64_sys_mount+0xba/0x150
[<00000000ebe26304>] do_syscall_64+0x151/0x490
[<00000000d25f2b42>] entry_SYSCALL_64_after_hwframe+0x44/0xa9
[<000000002e0abd2c>] 0xfffffffffffffffffuse_dev_alloc() allocate fud->pq.processing.
But this hash table is not freed.
Fix this by freeing fud->pq.processing.
---
Dear Miklos Szeredi
I found kmemleak in fuse and try to create a patch. Please consider this kmemleak and patch.
Regards.
--- fs/fuse/inode.c | 1 + 1 file changed, 1 insertion(+)
fs/fuse/inode.c +1 -0
@@ -1068,6 +1068,7 @@ void fuse_dev_free(struct fuse_dev *fud)fuse_conn_put (fc); } +kfree (fud->pq.processing);kfree (fud); }EXPORT_SYMBOL_GPL (fuse_dev_free);
Signature · 1 line
2.19.1
Quoted 26 lines · Takeshi Misawa
When ntfs is unmounted, the following leak is
reported by kmemleak.kmemleak report:
unreferenced object
0xffff880052bf4400(size 4096):
comm "mount.ntfs", pid 16530, jiffies 4294861127 (age 3215.836s) hex dump (first 32 bytes):00 44 bf 52 00 88 ff ff 00 44 bf 52 00 88 ff ff .D.R.....D.R.... 10 44 bf 52 00 88 ff ff 10 44 bf 52 00 88 ff ff .D.R.....D.R.... backtrace: [<00000000bf4a2f8d>] fuse_fill_super+0xb22/0x1da0 [fuse] [<000000004dde0f0c>] mount_bdev+0x263/0x320 [<0000000025aebc66>] mount_fs+0x82/0x2bf [<0000000042c5a6be>] vfs_kern_mount.part.33+0xbf/0x480 [<00000000ed10cd5b>] do_mount+0x3de/0x2ad0 [<00000000d59ff068>] ksys_mount+0xba/0xd0 [<000000001bda1bcc>] __x64_sys_mount+0xba/0x150 [<00000000ebe26304>] do_syscall_64+0x151/0x490 [<00000000d25f2b42>] entry_SYSCALL_64_after_hwframe+0x44/0xa9 [<000000002e0abd2c>] 0xffffffffffffffff
fuse_dev_alloc()allocatefud->pq.processing.
But this hash table is not freed.
Quoted 8 lines
Fix this by freeing
fud->pq.processing.Signed-off-by Takeshi Misawa <[email protected]>---
Dear Miklos SzerediI found kmemleak in fuse and try to create a patch. Please consider this kmemleak and patch.
Thanks, applied.
Miklos
Quoted 21 lines
Regards.
--- fs/fuse/inode.c | 1 + 1 file changed, 1 insertion(+)fs/fuse/inode.c +1
-1@@ -1068,6 +1068,7 @@ void fuse_dev_free(struct fuse_dev *fud)fuse_conn_put (fc); } +kfree (fud->pq.processing);kfree (fud); }EXPORT_SYMBOL_GPL (fuse_dev_free); --2.19.1