staging: hv: Remove NULL check before kfree
This patch was generated by the following semantic patch:
// <smpl>
@@ expression E; @@
- if (E != NULL) { kfree(E); }
+ kfree(E);
@@ expression E; @@
- if (E != NULL) { kfree(E); E = NULL; }
+ kfree(E);
+ E = NULL;
// </smpl>
Signed-off-by: Ilia Mirkin <imirkin@alum.mit.edu>
Cc: K. Y. Srinivasan <kys@microsoft.com>
Cc: Haiyang Zhang <haiyangz@microsoft.com>
Cc: Mike Sterling <mike.sterling@microsoft.com>
Cc: Abhishek Kane <v-abkane@microsoft.com>
Cc: Hank Janssen <hjanssen@microsoft.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
This commit is contained in:
committed by
Greg Kroah-Hartman
parent
e13a0b5a4b
commit
dd9b15dc03
@@ -829,8 +829,7 @@ int vmbus_request_offers(void)
|
||||
|
||||
|
||||
cleanup:
|
||||
if (msginfo)
|
||||
kfree(msginfo);
|
||||
kfree(msginfo);
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
@@ -186,9 +186,7 @@ Cleanup:
|
||||
vmbus_connection.monitor_pages = NULL;
|
||||
}
|
||||
|
||||
if (msginfo) {
|
||||
kfree(msginfo);
|
||||
}
|
||||
kfree(msginfo);
|
||||
|
||||
return ret;
|
||||
}
|
||||
|
||||
@@ -402,15 +402,11 @@ static void MousevscOnReceiveDeviceInfo(struct mousevsc_dev *InputDevice, struct
|
||||
return;
|
||||
|
||||
Cleanup:
|
||||
if (InputDevice->HidDesc) {
|
||||
kfree(InputDevice->HidDesc);
|
||||
InputDevice->HidDesc = NULL;
|
||||
}
|
||||
kfree(InputDevice->HidDesc);
|
||||
InputDevice->HidDesc = NULL;
|
||||
|
||||
if (InputDevice->ReportDesc) {
|
||||
kfree(InputDevice->ReportDesc);
|
||||
InputDevice->ReportDesc = NULL;
|
||||
}
|
||||
kfree(InputDevice->ReportDesc);
|
||||
InputDevice->ReportDesc = NULL;
|
||||
|
||||
InputDevice->DeviceInfoStatus = -1;
|
||||
InputDevice->device_wait_condition = 1;
|
||||
|
||||
Reference in New Issue
Block a user