af_packet: make tpacket_rcv to not set status value before run_filter
authorAlexander Drozdov <al.drozdov@gmail.com>
Mon, 23 Mar 2015 06:11:12 +0000 (09:11 +0300)
committerDavid S. Miller <davem@davemloft.net>
Tue, 24 Mar 2015 02:00:36 +0000 (22:00 -0400)
It is just an optimization. We don't need the value of status variable
if the packet is filtered.

Signed-off-by: Alexander Drozdov <al.drozdov@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/packet/af_packet.c

index b91ac5946ad1a4b88123c63dc33106ade6ce5bd3..9d854c5ce0b5065a78a5a684ec3ccac09d66f826 100644 (file)
@@ -1916,14 +1916,15 @@ static int tpacket_rcv(struct sk_buff *skb, struct net_device *dev,
                }
        }
 
-       if (skb->ip_summed == CHECKSUM_PARTIAL)
-               status |= TP_STATUS_CSUMNOTREADY;
-
        snaplen = skb->len;
 
        res = run_filter(skb, sk, snaplen);
        if (!res)
                goto drop_n_restore;
+
+       if (skb->ip_summed == CHECKSUM_PARTIAL)
+               status |= TP_STATUS_CSUMNOTREADY;
+
        if (snaplen > res)
                snaplen = res;