diff options
| author |   Adrian Bunk <bunk@stusta.de> | 2007-03-19 10:26:32 +0100 |
|---|---|---|
| committer |   Greg Kroah-Hartman <gregkh@suse.de> | 2007-04-27 13:28:36 -0700 |
| commit | 1b55fa2d430d6c708d3e5efbc6134be4a0c32b15 (patch) | |
| tree | cb512cfbd98b12e53ed5f638bb77312a2c4dfd6e /drivers/usb | |
| parent | USB: iowarrior.c: fix NULL dereference (diff) | |
| download | linux-dev-1b55fa2d430d6c708d3e5efbc6134be4a0c32b15.tar.xz linux-dev-1b55fa2d430d6c708d3e5efbc6134be4a0c32b15.zip | |
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) /* Grab private device ptr */ struct gtco *device = usb_get_intfdata (interface); - struct input_dev *inputdev; - - inputdev = device->inputdevice; /* Now reverse all the registration stuff */ if (device) { - input_unregister_device(inputdev); + input_unregister_device(device->inputdevice); usb_kill_urb(device->urbinfo); usb_free_urb(device->urbinfo); usb_buffer_free(device->usbdev, REPORT_MAX_SIZE, |