ip_gre: check packet length and mtu correctly in erspan_xmit
authorXin Long <lucien.xin@gmail.com>
Sun, 1 Oct 2017 14:00:54 +0000 (22:00 +0800)
committerDavid S. Miller <davem@davemloft.net>
Mon, 2 Oct 2017 05:30:32 +0000 (22:30 -0700)
As a ARPHRD_ETHER device, skb->len in erspan_xmit is the length
of the whole ether packet. So before checking if a packet size
exceeds the mtu, skb->len should subtract dev->hard_header_len.

Otherwise, all packets with max size according to mtu would be
trimmed to be truncated packet.

Fixes: 84e54fe0a5ea ("gre: introduce native tunnel support for ERSPAN")
Signed-off-by: Xin Long <lucien.xin@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ip_gre.c

index b25b1e5112d0de4a9b5050b9fa3e81f5ee074516..2a4ef9dc48ffb80d6ee0d7118867773a1f5907c5 100644 (file)
@@ -731,7 +731,7 @@ static netdev_tx_t erspan_xmit(struct sk_buff *skb,
        if (skb_cow_head(skb, dev->needed_headroom))
                goto free_skb;
 
-       if (skb->len > dev->mtu) {
+       if (skb->len - dev->hard_header_len > dev->mtu) {
                pskb_trim(skb, dev->mtu);
                truncate = true;
        }