diff options
author | Jesper Juhl <juhl-lkml@dif.dk> | 2005-04-18 20:39:34 -0400 |
---|---|---|
committer | Greg K-H <gregkh@suse.de> | 2005-04-18 20:39:34 -0400 |
commit | 1bc3c9e1e44c2059fe2ffa6ff70ad0a925d7b05f (patch) | |
tree | 0bc14ec53acf3b4c08a9995c7ea335e236435558 /drivers/usb/media/se401.c | |
parent | 6fd19f4b55f7fd1c9d8650bd7f8df2c81b69c5ca (diff) |
[PATCH] USB: kfree cleanup for drivers/usb/* - no need to check for NULL
Get rid of a bunch of redundant NULL pointer checks in drivers/usb/*,
there's no need to check a pointer for NULL before calling kfree() on it.
Signed-off-by: Jesper Juhl <juhl-lkml@dif.dk>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
Index: gregkh-2.6/drivers/usb/class/audio.c
===================================================================
Diffstat (limited to 'drivers/usb/media/se401.c')
-rw-r--r-- | drivers/usb/media/se401.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/drivers/usb/media/se401.c b/drivers/usb/media/se401.c index 685bdae5cb62..f69e443cd1bc 100644 --- a/drivers/usb/media/se401.c +++ b/drivers/usb/media/se401.c | |||
@@ -868,13 +868,14 @@ static void usb_se401_remove_disconnected (struct usb_se401 *se401) | |||
868 | 868 | ||
869 | se401->dev = NULL; | 869 | se401->dev = NULL; |
870 | 870 | ||
871 | for (i=0; i<SE401_NUMSBUF; i++) if (se401->urb[i]) { | 871 | for (i=0; i<SE401_NUMSBUF; i++) |
872 | usb_kill_urb(se401->urb[i]); | 872 | if (se401->urb[i]) { |
873 | usb_free_urb(se401->urb[i]); | 873 | usb_kill_urb(se401->urb[i]); |
874 | se401->urb[i] = NULL; | 874 | usb_free_urb(se401->urb[i]); |
875 | kfree(se401->sbuf[i].data); | 875 | se401->urb[i] = NULL; |
876 | } | 876 | kfree(se401->sbuf[i].data); |
877 | for (i=0; i<SE401_NUMSCRATCH; i++) if (se401->scratch[i].data) { | 877 | } |
878 | for (i=0; i<SE401_NUMSCRATCH; i++) { | ||
878 | kfree(se401->scratch[i].data); | 879 | kfree(se401->scratch[i].data); |
879 | } | 880 | } |
880 | if (se401->inturb) { | 881 | if (se401->inturb) { |