Browse Source

Staging: wlan-ng: Remove unnecessary checks for NULL before calling kfree()

Signed-off-by: Moritz Muehlenhoff <jmm@debian.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Moritz Muehlenhoff 16 years ago
parent
commit
7c98f71878
2 changed files with 5 additions and 10 deletions
  1. 3 6
      drivers/staging/wlan-ng/prism2sta.c
  2. 2 4
      drivers/staging/wlan-ng/prism2usb.c

+ 3 - 6
drivers/staging/wlan-ng/prism2sta.c

@@ -1134,8 +1134,7 @@ static void prism2sta_inf_hostscanresults(wlandevice_t *wlandev,
 	if (nbss > 32)
 		nbss = 32;
 
-	if (hw->scanresults)
-		kfree(hw->scanresults);
+	kfree(hw->scanresults);
 
 	hw->scanresults = kmalloc(sizeof(hfa384x_InfFrame_t), GFP_ATOMIC);
 	memcpy(hw->scanresults, inf, sizeof(hfa384x_InfFrame_t));
@@ -1966,10 +1965,8 @@ static wlandevice_t *create_wlan(void)
 
 	if (!wlandev || !hw) {
 		printk(KERN_ERR "%s: Memory allocation failure.\n", dev_info);
-		if (wlandev)
-			kfree(wlandev);
-		if (hw)
-			kfree(hw);
+		kfree(wlandev);
+		kfree(hw);
 		return NULL;
 	}
 

+ 2 - 4
drivers/staging/wlan-ng/prism2usb.c

@@ -165,10 +165,8 @@ static int prism2sta_probe_usb(struct usb_interface *interface,
 	goto done;
 
 failed:
-	if (wlandev)
-		kfree(wlandev);
-	if (hw)
-		kfree(hw);
+	kfree(wlandev);
+	kfree(hw);
 	wlandev = NULL;
 
 done: