net: Remove unnecessary net action assertion
authorjamal <hadi@cyberus.ca>
Fri, 4 Jun 2010 02:06:22 +0000 (02:06 +0000)
committerDavid S. Miller <davem@davemloft.net>
Mon, 7 Jun 2010 08:10:44 +0000 (01:10 -0700)
The extra assertion to allow packet munging only when there are
no other ptypes listening which may have worked around an old bug
is unnecessary. It is sufficient to check if the skb is cloned before
trampling on it. Thanks to Herbert Xu for being persistent and patient
in getting this across.
[Note that cloning checks and assertions are the general rule used
by tc actions (documentation/networking/tc-actions-env-rules.txt)].

Signed-off-by: Jamal Hadi Salim <hadi@cyberus.ca>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/core/dev.c

index b65347c2cf2a2469f4a55d9cb2dd25811728a260..c8d127718ff191a78c80acf81eec26d95c791cda 100644 (file)
@@ -2663,9 +2663,6 @@ static inline struct sk_buff *handle_ing(struct sk_buff *skb,
        if (*pt_prev) {
                *ret = deliver_skb(skb, *pt_prev, orig_dev);
                *pt_prev = NULL;
-       } else {
-               /* Huh? Why does turning on AF_PACKET affect this? */
-               skb->tc_verd = SET_TC_OK2MUNGE(skb->tc_verd);
        }
 
        switch (ing_filter(skb)) {