netdev - Re: [PATCH net] dccp: Fix memory leak in dccp_feat_change

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-Id: 
 <173321751975.4083494.12180230037136307201.git-patchwork-notify@kernel.org>
Date: Tue, 03 Dec 2024 09:18:39 +0000
From: patchwork-bot+netdevbpf@...nel.org
To: Ivan Solodovnikov <solodovnikov.ia@...stech.edu>
Cc: gerrit@....abdn.ac.uk, davem@...emloft.net, kuba@...nel.org,
 dccp@...r.kernel.org, netdev@...r.kernel.org, linux-kernel@...r.kernel.org,
 lvc-project@...uxtesting.org
Subject: Re: [PATCH net] dccp: Fix memory leak in dccp_feat_change_recv

Hello:

This patch was applied to netdev/net.git (main)
by Paolo Abeni <pabeni@...hat.com>:

On Tue, 26 Nov 2024 17:39:02 +0300 you wrote:
> If dccp_feat_push_confirm() fails after new value for SP feature was accepted
> without reconciliation ('entry == NULL' branch), memory allocated for that value
> with dccp_feat_clone_sp_val() is never freed.
> 
> Here is the kmemleak stack for this:
> 
> unreferenced object 0xffff88801d4ab488 (size 8):
>   comm "syz-executor310", pid 1127, jiffies 4295085598 (age 41.666s)
>   hex dump (first 8 bytes):
>     01 b4 4a 1d 80 88 ff ff                          ..J.....
>   backtrace:
>     [<00000000db7cabfe>] kmemdup+0x23/0x50 mm/util.c:128
>     [<0000000019b38405>] kmemdup include/linux/string.h:465 [inline]
>     [<0000000019b38405>] dccp_feat_clone_sp_val net/dccp/feat.c:371 [inline]
>     [<0000000019b38405>] dccp_feat_clone_sp_val net/dccp/feat.c:367 [inline]
>     [<0000000019b38405>] dccp_feat_change_recv net/dccp/feat.c:1145 [inline]
>     [<0000000019b38405>] dccp_feat_parse_options+0x1196/0x2180 net/dccp/feat.c:1416
>     [<00000000b1f6d94a>] dccp_parse_options+0xa2a/0x1260 net/dccp/options.c:125
>     [<0000000030d7b621>] dccp_rcv_state_process+0x197/0x13d0 net/dccp/input.c:650
>     [<000000001f74c72e>] dccp_v4_do_rcv+0xf9/0x1a0 net/dccp/ipv4.c:688
>     [<00000000a6c24128>] sk_backlog_rcv include/net/sock.h:1041 [inline]
>     [<00000000a6c24128>] __release_sock+0x139/0x3b0 net/core/sock.c:2570
>     [<00000000cf1f3a53>] release_sock+0x54/0x1b0 net/core/sock.c:3111
>     [<000000008422fa23>] inet_wait_for_connect net/ipv4/af_inet.c:603 [inline]
>     [<000000008422fa23>] __inet_stream_connect+0x5d0/0xf70 net/ipv4/af_inet.c:696
>     [<0000000015b6f64d>] inet_stream_connect+0x53/0xa0 net/ipv4/af_inet.c:735
>     [<0000000010122488>] __sys_connect_file+0x15c/0x1a0 net/socket.c:1865
>     [<00000000b4b70023>] __sys_connect+0x165/0x1a0 net/socket.c:1882
>     [<00000000f4cb3815>] __do_sys_connect net/socket.c:1892 [inline]
>     [<00000000f4cb3815>] __se_sys_connect net/socket.c:1889 [inline]
>     [<00000000f4cb3815>] __x64_sys_connect+0x6e/0xb0 net/socket.c:1889
>     [<00000000e7b1e839>] do_syscall_64+0x33/0x40 arch/x86/entry/common.c:46
>     [<0000000055e91434>] entry_SYSCALL_64_after_hwframe+0x67/0xd1
> 
> [...]

Here is the summary with links:
  - [net] dccp: Fix memory leak in dccp_feat_change_recv
    https://git.kernel.org/netdev/net/c/22be4727a8f8

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html