diff options
| author | Eric Dumazet <edumazet@google.com> | 2018-05-02 19:03:30 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2018-05-02 22:37:38 +0200 |
| commit | 7df40c2673a1307c3260aab6f9d4b9bf97ca8fd7 (patch) | |
| tree | e2da27920cae457805549786292b4dc6e0b592d8 /Documentation/netlabel | |
| parent | ipv6: Revert "ipv6: Allow non-gateway ECMP for IPv6" (diff) | |
| download | linux-7df40c2673a1307c3260aab6f9d4b9bf97ca8fd7.tar.xz linux-7df40c2673a1307c3260aab6f9d4b9bf97ca8fd7.zip | |
net_sched: fq: take care of throttled flows before reuse
Normally, a socket can not be freed/reused unless all its TX packets
left qdisc and were TX-completed. However connect(AF_UNSPEC) allows
this to happen.
With commit fc59d5bdf1e3 ("pkt_sched: fq: clear time_next_packet for
reused flows") we cleared f->time_next_packet but took no special
action if the flow was still in the throttled rb-tree.
Since f->time_next_packet is the key used in the rb-tree searches,
blindly clearing it might break rb-tree integrity. We need to make
sure the flow is no longer in the rb-tree to avoid this problem.
Fixes: fc59d5bdf1e3 ("pkt_sched: fq: clear time_next_packet for reused flows")
Signed-off-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'Documentation/netlabel')
0 files changed, 0 insertions, 0 deletions