diff options
author | Adrian Bunk <bunk@stusta.de> | 2007-03-19 05:26:32 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@suse.de> | 2007-04-27 16:28:36 -0400 |
commit | 1b55fa2d430d6c708d3e5efbc6134be4a0c32b15 (patch) | |
tree | cb512cfbd98b12e53ed5f638bb77312a2c4dfd6e /drivers/usb | |
parent | f81ee4d52880b08c213982df5041217212689960 (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>
Diffstat (limited to 'drivers/usb')
-rw-r--r-- | drivers/usb/input/gtco.c | 5 |
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, |