diff options
| author | Jorge Boncompte [DTI2] <jorge@dti2.net> | 2011-11-21 11:25:56 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2011-11-22 22:15:34 +0100 |
| commit | 3b829366cc6d0adeb4df2c2d917926f6b41c573d (patch) | |
| tree | be99f35ab02c024fbb777644a0582ce25f35fc93 /net/atm | |
| parent | atm: clip: Don't move counters backwards (diff) | |
| download | linux-3b829366cc6d0adeb4df2c2d917926f6b41c573d.tar.xz linux-3b829366cc6d0adeb4df2c2d917926f6b41c573d.zip | |
atm: clip: move clip_devs check to clip_push
This will allow further cleanup.
Signed-off-by: Jorge Boncompte [DTI2] <jorge@dti2.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/atm')
| -rw-r--r-- | net/atm/clip.c | 17 |
1 files changed, 10 insertions, 7 deletions
diff --git a/net/atm/clip.c b/net/atm/clip.c index b3ab7ddc1185..e2de7c548b3a 100644 --- a/net/atm/clip.c +++ b/net/atm/clip.c @@ -189,6 +189,13 @@ static void clip_push(struct atm_vcc *vcc, struct sk_buff *skb) struct clip_vcc *clip_vcc = CLIP_VCC(vcc); pr_debug("\n"); + + if (!clip_devs) { + atm_return(vcc, skb->truesize); + kfree_skb(skb); + return; + } + if (!skb) { pr_debug("removing VCC %p\n", clip_vcc); if (clip_vcc->entry) @@ -480,13 +487,9 @@ static int clip_mkip(struct atm_vcc *vcc, int timeout) spin_unlock_irqrestore(&rq->lock, flags); /* re-process everything received between connection setup and MKIP */ - skb_queue_walk_safe(&queue, skb, tmp) { - if (!clip_devs) { - atm_return(vcc, skb->truesize); - kfree_skb(skb); - } else - clip_push(vcc, skb); - } + skb_queue_walk_safe(&queue, skb, tmp) + clip_push(vcc, skb); + return 0; } |