ip_gre: fix error path when erspan_rcv failed
authorHaishuang Yan <yanhaishuang@cmss.chinamobile.com>
Wed, 20 Dec 2017 02:21:46 +0000 (10:21 +0800)
committerDavid S. Miller <davem@davemloft.net>
Wed, 20 Dec 2017 18:51:46 +0000 (13:51 -0500)
When erspan_rcv call return PACKET_REJECT, we shoudn't call ipgre_rcv to
process packets again, instead send icmp unreachable message in error
path.

Fixes: 84e54fe0a5ea ("gre: introduce native tunnel support for ERSPAN")
Acked-by: William Tu <u9012063@gmail.com>
Cc: William Tu <u9012063@gmail.com>
Signed-off-by: Haishuang Yan <yanhaishuang@cmss.chinamobile.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/ip_gre.c

index fd4d6e96da7e953b8bb24758a5a691051a3e1e6a..5c36b6745dbce6a96a5aa6b9fa914faf65e4fc74 100644 (file)
@@ -434,11 +434,13 @@ static int gre_rcv(struct sk_buff *skb)
                     tpi.proto == htons(ETH_P_ERSPAN2))) {
                if (erspan_rcv(skb, &tpi, hdr_len) == PACKET_RCVD)
                        return 0;
+               goto out;
        }
 
        if (ipgre_rcv(skb, &tpi, hdr_len) == PACKET_RCVD)
                return 0;
 
+out:
        icmp_send(skb, ICMP_DEST_UNREACH, ICMP_PORT_UNREACH, 0);
 drop:
        kfree_skb(skb);