diff options
| author | jamal <hadi@cyberus.ca> | 2010-06-04 04:43:06 +0200 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2010-06-07 10:10:43 +0200 |
| commit | 9dacaf17a60101a55d456cc7b00e269d8145aa0d (patch) | |
| tree | ed5996e4d799559f8f91fdb09ec989042b8f4c0e /net/sched/act_pedit.c | |
| parent | htb: remove two unnecessary assignments (diff) | |
| download | linux-9dacaf17a60101a55d456cc7b00e269d8145aa0d.tar.xz linux-9dacaf17a60101a55d456cc7b00e269d8145aa0d.zip | |
net sched: make pedit check for clones instead
Now that the core path doesnt set OK to munge we detect
writable skbs by looking to see if they are cloned.
Signed-off-by: Jamal Hadi Salim <hadi@cyberus.ca>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/act_pedit.c')
| -rw-r--r-- | net/sched/act_pedit.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/sched/act_pedit.c b/net/sched/act_pedit.c index 50e3d945e1f4..a0593c9640db 100644 --- a/net/sched/act_pedit.c +++ b/net/sched/act_pedit.c @@ -127,8 +127,7 @@ static int tcf_pedit(struct sk_buff *skb, struct tc_action *a, int i, munged = 0; unsigned int off; - if (!(skb->tc_verd & TC_OK2MUNGE)) { - /* should we set skb->cloned? */ + if (skb_cloned(skb)) { if (pskb_expand_head(skb, 0, 0, GFP_ATOMIC)) { return p->tcf_action; } |