diff options
| author | Willem de Bruijn <willemb@google.com> | 2017-09-26 12:20:17 -0400 |
|---|---|---|
| committer | Ben Hutchings <ben@decadent.org.uk> | 2018-01-01 20:51:44 +0000 |
| commit | 38b2f505e56f7496a3bddfab910d9001f3c6142b (patch) | |
| tree | bc9123b2d47ec0db12796061aefeeec3038e94ac /kernel/sysctl.c | |
| parent | 0ad43afc28bab51dcfc002d36afc541b7b32d5b1 (diff) | |
packet: only test po->has_vnet_hdr once in packet_snd
commit da7c9561015e93d10fe6aab73e9288e0d09d65a6 upstream.
Packet socket option po->has_vnet_hdr can be updated concurrently with
other operations if no ring is attached.
Do not test the option twice in packet_snd, as the value may change in
between calls. A race on setsockopt disable may cause a packet > mtu
to be sent without having GSO options set.
Fixes: bfd5f4a3d605 ("packet: Add GSO/csum offload support.")
Signed-off-by: Willem de Bruijn <willemb@google.com>
Reviewed-by: Eric Dumazet <edumazet@google.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
[bwh: Backported to 3.16: adjust context]
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
Diffstat (limited to 'kernel/sysctl.c')
0 files changed, 0 insertions, 0 deletions