Re: [PATCH net-next] udp: properly deal with xfrm encap and ADDRFORM

[patchwork-bot+netdevbpf@kernel.org]

Hello:

This patch was applied to netdev/net-next.git (main)
by Jakub Kicinski <kuba@kernel.org>:

On Wed,  9 Apr 2025 22:00:56 +0200 you wrote:
> UDP GRO accounting assumes that the GRO receive callback is always
> set when the UDP tunnel is enabled, but syzkaller proved otherwise,
> leading tot the following splat:
> 
> WARNING: CPU: 0 PID: 5837 at net/ipv4/udp_offload.c:123 udp_tunnel_update_gro_rcv+0x28d/0x4c0 net/ipv4/udp_offload.c:123
> Modules linked in:
> CPU: 0 UID: 0 PID: 5837 Comm: syz-executor850 Not tainted 6.14.0-syzkaller-13320-g420aabef3ab5 #0 PREEMPT(full)
> Hardware name: Google Compute Engine/Google Compute Engine, BIOS Google 02/12/2025
> RIP: 0010:udp_tunnel_update_gro_rcv+0x28d/0x4c0 net/ipv4/udp_offload.c:123
> Code: 00 00 e8 c6 5a 2f f7 48 c1 e5 04 48 8d b5 20 53 c7 9a ba 10
>       00 00 00 4c 89 ff e8 ce 87 99 f7 e9 ce 00 00 00 e8 a4 5a 2f
>       f7 90 <0f> 0b 90 e9 de fd ff ff bf 01 00 00 00 89 ee e8 cf
>       5e 2f f7 85 ed
> RSP: 0018:ffffc90003effa88 EFLAGS: 00010293
> RAX: ffffffff8a93fc9c RBX: 0000000000000000 RCX: ffff8880306f9e00
> RDX: 0000000000000000 RSI: 0000000000000000 RDI: 0000000000000000
> RBP: 0000000000000000 R08: ffffffff8a93fabe R09: 1ffffffff20bfb2e
> R10: dffffc0000000000 R11: fffffbfff20bfb2f R12: ffff88814ef21738
> R13: dffffc0000000000 R14: ffff88814ef21778 R15: 1ffff11029de42ef
> FS:  0000000000000000(0000) GS:ffff888124f96000(0000) knlGS:0000000000000000
> CS:  0010 DS: 0000 ES: 0000 CR0: 0000000080050033
> CR2: 00007f04eec760d0 CR3: 000000000eb38000 CR4: 00000000003526f0
> DR0: 0000000000000000 DR1: 0000000000000000 DR2: 0000000000000000
> DR3: 0000000000000000 DR6: 00000000fffe0ff0 DR7: 0000000000000400
> Call Trace:
>  <TASK>
>  udp_tunnel_cleanup_gro include/net/udp_tunnel.h:205 [inline]
>  udpv6_destroy_sock+0x212/0x270 net/ipv6/udp.c:1829
>  sk_common_release+0x71/0x2e0 net/core/sock.c:3896
>  inet_release+0x17d/0x200 net/ipv4/af_inet.c:435
>  __sock_release net/socket.c:647 [inline]
>  sock_close+0xbc/0x240 net/socket.c:1391
>  __fput+0x3e9/0x9f0 fs/file_table.c:465
>  task_work_run+0x251/0x310 kernel/task_work.c:227
>  exit_task_work include/linux/task_work.h:40 [inline]
>  do_exit+0xa11/0x27f0 kernel/exit.c:953
>  do_group_exit+0x207/0x2c0 kernel/exit.c:1102
>  __do_sys_exit_group kernel/exit.c:1113 [inline]
>  __se_sys_exit_group kernel/exit.c:1111 [inline]
>  __x64_sys_exit_group+0x3f/0x40 kernel/exit.c:1111
>  x64_sys_call+0x26c3/0x26d0 arch/x86/include/generated/asm/syscalls_64.h:232
>  do_syscall_x64 arch/x86/entry/syscall_64.c:63 [inline]
>  do_syscall_64+0xf3/0x230 arch/x86/entry/syscall_64.c:94
>  entry_SYSCALL_64_after_hwframe+0x77/0x7f
> RIP: 0033:0x7f04eebfac79
> Code: Unable to access opcode bytes at 0x7f04eebfac4f.
> RSP: 002b:00007fffdcaa34a8 EFLAGS: 00000246 ORIG_RAX: 00000000000000e7
> RAX: ffffffffffffffda RBX: 0000000000000000 RCX: 00007f04eebfac79
> RDX: 000000000000003c RSI: 00000000000000e7 RDI: 0000000000000000
> RBP: 00007f04eec75270 R08: ffffffffffffffb8 R09: 00007fffdcaa36c8
> R10: 0000200000000000 R11: 0000000000000246 R12: 00007f04eec75270
> R13: 0000000000000000 R14: 00007f04eec75cc0 R15: 00007f04eebcca70
> 
> [...]

Here is the summary with links:
  - [net-next] udp: properly deal with xfrm encap and ADDRFORM
    https://git.kernel.org/netdev/net-next/c/c26c192c3d48

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