diff options
| author | Johannes Berg <[email protected]> | 2024-06-07 15:53:32 +0000 |
|---|---|---|
| committer | Jakub Kicinski <[email protected]> | 2024-06-11 02:36:49 +0000 |
| commit | 44180feaccf266d9b0b28cc4ceaac019817deb5c (patch) | |
| tree | 17cae917ba6f0002a17d93e878861926da9405f1 /net/ipv4/tcp_timer.c | |
| parent | Merge branch 'geneve-fixes' (diff) | |
| download | kernel-44180feaccf266d9b0b28cc4ceaac019817deb5c.tar.gz kernel-44180feaccf266d9b0b28cc4ceaac019817deb5c.zip | |
net/sched: initialize noop_qdisc owner
When the noop_qdisc owner isn't initialized, then it will be 0,
so packets will erroneously be regarded as having been subject
to recursion as long as only CPU 0 queues them. For non-SMP,
that's all packets, of course. This causes a change in what's
reported to userspace, normally noop_qdisc would drop packets
silently, but with this change the syscall returns -ENOBUFS if
RECVERR is also set on the socket.
Fix this by initializing the owner field to -1, just like it
would be for dynamically allocated qdiscs by qdisc_alloc().
Fixes: 0f022d32c3ec ("net/sched: Fix mirred deadlock on device recursion")
Signed-off-by: Johannes Berg <[email protected]>
Reviewed-by: Eric Dumazet <[email protected]>
Link: https://lore.kernel.org/r/20240607175340.786bfb938803.I493bf8422e36be4454c08880a8d3703cea8e421a@changeid
Signed-off-by: Jakub Kicinski <[email protected]>
Diffstat (limited to 'net/ipv4/tcp_timer.c')
0 files changed, 0 insertions, 0 deletions