diff options
author | Devendra Naga <devendra.aaru@gmail.com> | 2012-09-09 18:40:59 +0530 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2012-09-10 15:42:51 -0700 |
commit | 311e24fb1a654a260697793880979142fbbd185d (patch) | |
tree | a9b65e130479a1d157f2240204ec84ecb7014dac | |
parent | 7694538cfa3636d217bdafa1328fb0cf85119540 (diff) | |
download | linux-3.10-311e24fb1a654a260697793880979142fbbd185d.tar.gz linux-3.10-311e24fb1a654a260697793880979142fbbd185d.tar.bz2 linux-3.10-311e24fb1a654a260697793880979142fbbd185d.zip |
staging:wlan-ng: cleanup p80211skb_free and p80211skb_rxmeta_detach
these functions doesn't need return at the end of the function
Signed-off-by: Devendra Naga <devendra.aaru@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/wlan-ng/p80211conv.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/drivers/staging/wlan-ng/p80211conv.c b/drivers/staging/wlan-ng/p80211conv.c index f53a27a2e3f..3df753b51e8 100644 --- a/drivers/staging/wlan-ng/p80211conv.c +++ b/drivers/staging/wlan-ng/p80211conv.c @@ -559,17 +559,17 @@ void p80211skb_rxmeta_detach(struct sk_buff *skb) /* Sanity checks */ if (skb == NULL) { /* bad skb */ pr_debug("Called w/ null skb.\n"); - goto exit; + return; } frmmeta = P80211SKB_FRMMETA(skb); if (frmmeta == NULL) { /* no magic */ pr_debug("Called w/ bad frmmeta magic.\n"); - goto exit; + return; } rxmeta = frmmeta->rx; if (rxmeta == NULL) { /* bad meta ptr */ pr_debug("Called w/ bad rxmeta ptr.\n"); - goto exit; + return; } /* Free rxmeta */ @@ -577,8 +577,6 @@ void p80211skb_rxmeta_detach(struct sk_buff *skb) /* Clear skb->cb */ memset(skb->cb, 0, sizeof(skb->cb)); -exit: - return; } /*---------------------------------------------------------------- @@ -660,5 +658,4 @@ void p80211skb_free(struct wlandevice *wlandev, struct sk_buff *skb) else printk(KERN_ERR "Freeing an skb (%p) w/ no frmmeta.\n", skb); dev_kfree_skb(skb); - return; } |