summaryrefslogtreecommitdiffstats
path: root/drivers/usb/misc
diff options
context:
space:
mode:
authorJulien Brunel <brunel@diku.dk>2008-09-01 10:57:27 +0200
committerGreg Kroah-Hartman <gregkh@suse.de>2008-10-17 14:40:53 -0700
commitbedf0883cbe3015d21aec5ed47ddffb429f6cca7 (patch)
tree5a31171dcfa203f33dde74d040b704bccfc0fceb /drivers/usb/misc
parent58209018cfd6f1fa8d0139af7c39dbfe73de21fd (diff)
downloadop-kernel-dev-bedf0883cbe3015d21aec5ed47ddffb429f6cca7.zip
op-kernel-dev-bedf0883cbe3015d21aec5ed47ddffb429f6cca7.tar.gz
USB: drivers/usb/misc: Use an IS_ERR test rather than a NULL test
In case of error, the function backlight_device_register returns an ERR pointer, but never returns a NULL pointer. So a NULL test that may come after a call to this function should be strengthened by an IS_ERR test. The semantic match that finds this problem is as follows: (http://www.emn.fr/x-info/coccinelle/) // <smpl> @match_bad_null_test@ expression x, E; statement S1,S2; @@ x = backlight_device_register(...) ... when != x = E * if (x != NULL) S1 else S2 // </smpl> Signed-off-by: Julien Brunel <brunel@diku.dk> Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Diffstat (limited to 'drivers/usb/misc')
-rw-r--r--drivers/usb/misc/appledisplay.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/usb/misc/appledisplay.c b/drivers/usb/misc/appledisplay.c
index a076c24..71d672e 100644
--- a/drivers/usb/misc/appledisplay.c
+++ b/drivers/usb/misc/appledisplay.c
@@ -314,7 +314,7 @@ error:
pdata->urbdata, pdata->urb->transfer_dma);
usb_free_urb(pdata->urb);
}
- if (pdata->bd)
+ if (pdata->bd && !IS_ERR(pdata->bd))
backlight_device_unregister(pdata->bd);
kfree(pdata->msgdata);
}
OpenPOWER on IntegriCloud