diff options
| author | Alexei Starovoitov <ast@kernel.org> | 2026-09-10 16:58:40 -0700 |
|---|---|---|
| committer | Alexei Starovoitov <ast@kernel.org> | 2026-09-10 16:58:40 -0700 |
| commit | 15071f2a1263e82150c77eeb1e94dbfc31950a8e (patch) | |
| tree | b1630d699b422154bf52ef64e086180b2dfb41e7 /tools/perf/scripts/python | |
| parent | f77d21245710690f3fb02d19d8dd4dc17ee58c2c (diff) | |
| parent | 8036d3a5a6589ef0721d15e7c03976bc3996f7a8 (diff) | |
| download | linux-next-15071f2a1263e82150c77eeb1e94dbfc31950a8e.tar.gz linux-next-15071f2a1263e82150c77eeb1e94dbfc31950a8e.zip | |
Merge branch 'bpf-tcp-fix-bpf_sock_destroy-on-time_wait-and-listener-socks'
Jiayuan Chen says:
====================
bpf,tcp: Fix bpf_sock_destroy() on TIME_WAIT and listener socks
Fix two bugs in bpf_sock_destroy(). One is an out-of-bounds read of
sk->sk_protocol on TIME_WAIT and NEW_SYN_RECV socks, since the field is
not in struct sock_common. The other is a might_sleep splat when
destroying a listener with children in its accept queue, the
cond_resched() in inet_csk_listen_stop() runs under the iterator's
rcu_read_lock(). Patch 3 adds a subtest for each.
v2 -> v3:
- Patch 1: add Reviewed-by from Kuniyuki.
- Patch 2: reword why cond_resched() has to go, it can reschedule or
report a bogus quiescent state there.
- Patch 2: keep the cond_resched() for the non-BPF path rather than
removing it as Kuniyuki suggested. VOLUNTARY and NONE are still
there on some arches, and on x86 in stable where this goes, so a
big listener close() still relies on it. Can go once those modes
are gone.
- Selftest: don't leak the fd if accept() unexpectedly succeeds.
v2: https://lore.kernel.org/bpf/20260906074135.185212-1-jiayuan.chen@linux.dev/
v1 -> v2:
- Patch 1: fix the return comment too.
- Patch 2: new.
- Selftest: server recv()s EOF before close so the FINs can't cross,
comment style, keep the blank line before RUN_TESTS(), add the
tcp_listen_pending subtest.
v1: https://lore.kernel.org/bpf/20260903125306.299943-1-jiayuan.chen@linux.dev/
====================
Link: https://patch.msgid.link/20260910112107.148770-1-jiayuan.chen@linux.dev
Signed-off-by: Alexei Starovoitov <ast@kernel.org>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions
