* [PATCH net v2] netdevsim: Fix memory leak of nsim_dev->fa_cookie
@ 2022-11-15 9:30 Wang Yufen
2022-11-16 20:40 ` patchwork-bot+netdevbpf
0 siblings, 1 reply; 2+ messages in thread
From: Wang Yufen @ 2022-11-15 9:30 UTC (permalink / raw)
To: netdev; +Cc: kuba, davem, edumazet, pabeni, jiri, Wang Yufen
kmemleak reports this issue:
unreferenced object 0xffff8881bac872d0 (size 8):
comm "sh", pid 58603, jiffies 4481524462 (age 68.065s)
hex dump (first 8 bytes):
04 00 00 00 de ad be ef ........
backtrace:
[<00000000c80b8577>] __kmalloc+0x49/0x150
[<000000005292b8c6>] nsim_dev_trap_fa_cookie_write+0xc1/0x210 [netdevsim]
[<0000000093d78e77>] full_proxy_write+0xf3/0x180
[<000000005a662c16>] vfs_write+0x1c5/0xaf0
[<000000007aabf84a>] ksys_write+0xed/0x1c0
[<000000005f1d2e47>] do_syscall_64+0x3b/0x90
[<000000006001c6ec>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
The issue occurs in the following scenarios:
nsim_dev_trap_fa_cookie_write()
kmalloc() fa_cookie
nsim_dev->fa_cookie = fa_cookie
..
nsim_drv_remove()
The fa_cookie allocked in nsim_dev_trap_fa_cookie_write() is not freed. To
fix, add kfree(nsim_dev->fa_cookie) to nsim_drv_remove().
Fixes: d3cbb907ae57 ("netdevsim: add ACL trap reporting cookie as a metadata")
Signed-off-by: Wang Yufen <wangyufen@huawei.com>
Cc: Jiri Pirko <jiri@mellanox.com>
---
drivers/net/netdevsim/dev.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/net/netdevsim/dev.c b/drivers/net/netdevsim/dev.c
index a7880c7..68e56e4 100644
--- a/drivers/net/netdevsim/dev.c
+++ b/drivers/net/netdevsim/dev.c
@@ -1683,6 +1683,7 @@ void nsim_drv_remove(struct nsim_bus_dev *nsim_bus_dev)
ARRAY_SIZE(nsim_devlink_params));
devl_resources_unregister(devlink);
kfree(nsim_dev->vfconfigs);
+ kfree(nsim_dev->fa_cookie);
devl_unlock(devlink);
devlink_free(devlink);
dev_set_drvdata(&nsim_bus_dev->dev, NULL);
--
1.8.3.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net v2] netdevsim: Fix memory leak of nsim_dev->fa_cookie
2022-11-15 9:30 [PATCH net v2] netdevsim: Fix memory leak of nsim_dev->fa_cookie Wang Yufen
@ 2022-11-16 20:40 ` patchwork-bot+netdevbpf
0 siblings, 0 replies; 2+ messages in thread
From: patchwork-bot+netdevbpf @ 2022-11-16 20:40 UTC (permalink / raw)
To: Wang Yufen; +Cc: netdev, kuba, davem, edumazet, pabeni, jiri
Hello:
This patch was applied to netdev/net.git (master)
by Jakub Kicinski <kuba@kernel.org>:
On Tue, 15 Nov 2022 17:30:25 +0800 you wrote:
> kmemleak reports this issue:
>
> unreferenced object 0xffff8881bac872d0 (size 8):
> comm "sh", pid 58603, jiffies 4481524462 (age 68.065s)
> hex dump (first 8 bytes):
> 04 00 00 00 de ad be ef ........
> backtrace:
> [<00000000c80b8577>] __kmalloc+0x49/0x150
> [<000000005292b8c6>] nsim_dev_trap_fa_cookie_write+0xc1/0x210 [netdevsim]
> [<0000000093d78e77>] full_proxy_write+0xf3/0x180
> [<000000005a662c16>] vfs_write+0x1c5/0xaf0
> [<000000007aabf84a>] ksys_write+0xed/0x1c0
> [<000000005f1d2e47>] do_syscall_64+0x3b/0x90
> [<000000006001c6ec>] entry_SYSCALL_64_after_hwframe+0x63/0xcd
>
> [...]
Here is the summary with links:
- [net,v2] netdevsim: Fix memory leak of nsim_dev->fa_cookie
https://git.kernel.org/netdev/net/c/064bc7312bd0
You are awesome, thank you!
--
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-11-16 20:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-15 9:30 [PATCH net v2] netdevsim: Fix memory leak of nsim_dev->fa_cookie Wang Yufen
2022-11-16 20:40 ` patchwork-bot+netdevbpf
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.