aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAdrian Bunk <bunk@stusta.de>2007-03-19 05:26:32 -0400
committerGreg Kroah-Hartman <gregkh@suse.de>2007-04-27 16:28:36 -0400
commit1b55fa2d430d6c708d3e5efbc6134be4a0c32b15 (patch)
treecb512cfbd98b12e53ed5f638bb77312a2c4dfd6e
parentf81ee4d52880b08c213982df5041217212689960 (diff)
USB: gtco.c: fix a use-before-check
NULL checks should be before the first dereference. Spotted by the Coverity checker. Signed-off-by: Adrian Bunk <bunk@stusta.de> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
-rw-r--r--drivers/usb/input/gtco.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/usb/input/gtco.c b/drivers/usb/input/gtco.c
index 203cdc1bbba4..ae756e0afc99 100644
--- a/drivers/usb/input/gtco.c
+++ b/drivers/usb/input/gtco.c
@@ -1047,13 +1047,10 @@ static void gtco_disconnect(struct usb_interface *interface)
1047 1047
1048 /* Grab private device ptr */ 1048 /* Grab private device ptr */
1049 struct gtco *device = usb_get_intfdata (interface); 1049 struct gtco *device = usb_get_intfdata (interface);
1050 struct input_dev *inputdev;
1051
1052 inputdev = device->inputdevice;
1053 1050
1054 /* Now reverse all the registration stuff */ 1051 /* Now reverse all the registration stuff */
1055 if (device) { 1052 if (device) {
1056 input_unregister_device(inputdev); 1053 input_unregister_device(device->inputdevice);
1057 usb_kill_urb(device->urbinfo); 1054 usb_kill_urb(device->urbinfo);
1058 usb_free_urb(device->urbinfo); 1055 usb_free_urb(device->urbinfo);
1059 usb_buffer_free(device->usbdev, REPORT_MAX_SIZE, 1056 usb_buffer_free(device->usbdev, REPORT_MAX_SIZE,