]> git.baikalelectronics.ru Git - kernel.git/commit
udp: avoid ufo handling on IP payload compression packets
authorAlexey Kodanev <alexey.kodanev@oracle.com>
Thu, 9 Mar 2017 10:56:46 +0000 (13:56 +0300)
committerDavid S. Miller <davem@davemloft.net>
Fri, 10 Mar 2017 02:28:42 +0000 (18:28 -0800)
commit32b37ab7a3d75d8edb79f2d350561ebecd534194
treea04c2242337f96ec9ae4c9d63fc81846eba56a09
parent6c83d1ccf6800366447f5dab6adfbc1acb512e49
udp: avoid ufo handling on IP payload compression packets

commit 987366c905cb ("ipv4: Don't use ufo handling on later transformed
packets") and commit 00b764294f91 ("ipv6: Don't use ufo handling on
later transformed packets") added a check that 'rt->dst.header_len' isn't
zero in order to skip UFO, but it doesn't include IPcomp in transport mode
where it equals zero.

Packets, after payload compression, may not require further fragmentation,
and if original length exceeds MTU, later compressed packets will be
transmitted incorrectly. This can be reproduced with LTP udp_ipsec.sh test
on veth device with enabled UFO, MTU is 1500 and UDP payload is 2000:

* IPv4 case, offset is wrong + unnecessary fragmentation
    udp_ipsec.sh -p comp -m transport -s 2000 &
    tcpdump -ni ltp_ns_veth2
    ...
    IP (tos 0x0, ttl 64, id 45203, offset 0, flags [+],
      proto Compressed IP (108), length 49)
      10.0.0.2 > 10.0.0.1: IPComp(cpi=0x1000)
    IP (tos 0x0, ttl 64, id 45203, offset 1480, flags [none],
      proto UDP (17), length 21) 10.0.0.2 > 10.0.0.1: ip-proto-17

* IPv6 case, sending small fragments
    udp_ipsec.sh -6 -p comp -m transport -s 2000 &
    tcpdump -ni ltp_ns_veth2
    ...
    IP6 (flowlabel 0x6b9ba, hlim 64, next-header Compressed IP (108)
      payload length: 37) fd00::2 > fd00::1: IPComp(cpi=0x1000)
    IP6 (flowlabel 0x6b9ba, hlim 64, next-header Compressed IP (108)
      payload length: 21) fd00::2 > fd00::1: IPComp(cpi=0x1000)

Fix it by checking 'rt->dst.xfrm' pointer to 'xfrm_state' struct, skip UFO
if xfrm is set. So the new check will include both cases: IPcomp and IPsec.

Fixes: 987366c905cb ("ipv4: Don't use ufo handling on later transformed packets")
Fixes: 00b764294f91 ("ipv6: Don't use ufo handling on later transformed packets")
Signed-off-by: Alexey Kodanev <alexey.kodanev@oracle.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ip_output.c
net/ipv6/ip6_output.c