net: change skb->mac_header when Generic XDP calls adjust_head
authorEdward Cree <ecree@solarflare.com>
Tue, 19 Sep 2017 17:45:56 +0000 (18:45 +0100)
committerDavid S. Miller <davem@davemloft.net>
Wed, 20 Sep 2017 22:44:22 +0000 (15:44 -0700)
Since XDP's view of the packet includes the MAC header, moving the start-
 of-packet with bpf_xdp_adjust_head needs to also update the offset of the
 MAC header (which is relative to skb->head, not to the skb->data that was
 changed).
Without this, tcpdump sees packets starting from the old MAC header rather
 than the new one, at least in my tests on the loopback device.

Fixes: b5cdae3291f7 ("net: Generic XDP")
Signed-off-by: Edward Cree <ecree@solarflare.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c

index fb766d906148e7d1e85a654eaf6048c1cdf49364..9a2254f9802f325de65fa569ceeaa5411a31dfcc 100644 (file)
@@ -3892,6 +3892,7 @@ static u32 netif_receive_generic_xdp(struct sk_buff *skb,
                __skb_pull(skb, off);
        else if (off < 0)
                __skb_push(skb, -off);
+       skb->mac_header += off;
 
        switch (act) {
        case XDP_REDIRECT: