diff options
| author | Hannes Frederic Sowa <hannes@stressinduktion.org> | 2016-02-22 18:43:25 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2016-02-24 14:11:34 -0500 |
| commit | a8c4a2522a0808c5c2143612909717d1115c40cf (patch) | |
| tree | 630d9bda2a7e98752fbb442f1d1a96d92035ef98 /lib/parser.c | |
| parent | c7dfe3abf40ec9a298884a4c0775062eac27afc9 (diff) | |
| download | op-kernel-dev-a8c4a2522a0808c5c2143612909717d1115c40cf.zip op-kernel-dev-a8c4a2522a0808c5c2143612909717d1115c40cf.tar.gz | |
ipv4: only create late gso-skb if skb is already set up with CHECKSUM_PARTIAL
Otherwise we break the contract with GSO to only pass CHECKSUM_PARTIAL
skbs down. This can easily happen with UDP+IPv4 sockets with the first
MSG_MORE write smaller than the MTU, second write is a sendfile.
Returning -EOPNOTSUPP lets the callers fall back into normal sendmsg path,
were we calculate the checksum manually during copying.
Commit d749c9cbffd6 ("ipv4: no CHECKSUM_PARTIAL on MSG_MORE corked
sockets") started to exposes this bug.
Fixes: d749c9cbffd6 ("ipv4: no CHECKSUM_PARTIAL on MSG_MORE corked sockets")
Reported-by: Jiri Benc <jbenc@redhat.com>
Cc: Jiri Benc <jbenc@redhat.com>
Reported-by: Wakko Warner <wakko@animx.eu.org>
Cc: Wakko Warner <wakko@animx.eu.org>
Signed-off-by: Hannes Frederic Sowa <hannes@stressinduktion.org>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'lib/parser.c')
0 files changed, 0 insertions, 0 deletions
