diff options
| author | Raphaël Beamonte | 2015-08-18 18:58:13 +0200 |
|---|---|---|
| committer | Greg Kroah-Hartman | 2015-09-13 03:24:34 +0200 |
| commit | efe8a7fad59f4e157485e91d84210fe7496afa03 (patch) | |
| tree | 2fb850a6a4a854d76fea88b265e5a9c7b460b1e2 /drivers/staging/rtl8192u | |
| parent | staging: rtl8192u: r8192U_core: remove forward declarations in .c file (diff) | |
| download | kernel-qcow2-linux-efe8a7fad59f4e157485e91d84210fe7496afa03.tar.gz kernel-qcow2-linux-efe8a7fad59f4e157485e91d84210fe7496afa03.tar.xz kernel-qcow2-linux-efe8a7fad59f4e157485e91d84210fe7496afa03.zip | |
staging: rtl8192u: r8192U_core: fix unnecessary check before kfree code style issue
kfree(NULL) is safe and the checks were not required.
Signed-off-by: Raphaël Beamonte <raphael.beamonte@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'drivers/staging/rtl8192u')
| -rw-r--r-- | drivers/staging/rtl8192u/r8192U_core.c | 13 |
1 files changed, 5 insertions, 8 deletions
diff --git a/drivers/staging/rtl8192u/r8192U_core.c b/drivers/staging/rtl8192u/r8192U_core.c index cd14c0a680da..d0020f1caa26 100644 --- a/drivers/staging/rtl8192u/r8192U_core.c +++ b/drivers/staging/rtl8192u/r8192U_core.c @@ -1724,10 +1724,9 @@ static void rtl8192_usb_deleteendpoints(struct net_device *dev) } kfree(priv->oldaddr); priv->oldaddr = NULL; - if (priv->pp_rxskb) { - kfree(priv->pp_rxskb); - priv->pp_rxskb = NULL; - } + + kfree(priv->pp_rxskb); + priv->pp_rxskb = NULL; } #else void rtl8192_usb_deleteendpoints(struct net_device *dev) @@ -1752,11 +1751,9 @@ void rtl8192_usb_deleteendpoints(struct net_device *dev) priv->rx_urb = NULL; kfree(priv->oldaddr); priv->oldaddr = NULL; - if (priv->pp_rxskb) { - kfree(priv->pp_rxskb); - priv->pp_rxskb = 0; - } + kfree(priv->pp_rxskb); + priv->pp_rxskb = 0; #endif } |