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/usbvideo.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/usbvideo.c')
-rw-r--r-- | drivers/usb/media/usbvideo.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/drivers/usb/media/usbvideo.c b/drivers/usb/media/usbvideo.c index 298484aa27d2..24efb21969c6 100644 --- a/drivers/usb/media/usbvideo.c +++ b/drivers/usb/media/usbvideo.c | |||
@@ -1169,10 +1169,8 @@ static int usbvideo_v4l_open(struct inode *inode, struct file *file) | |||
1169 | } | 1169 | } |
1170 | RingQueue_Free(&uvd->dp); | 1170 | RingQueue_Free(&uvd->dp); |
1171 | for (i=0; i < USBVIDEO_NUMSBUF; i++) { | 1171 | for (i=0; i < USBVIDEO_NUMSBUF; i++) { |
1172 | if (uvd->sbuf[i].data != NULL) { | 1172 | kfree(uvd->sbuf[i].data); |
1173 | kfree (uvd->sbuf[i].data); | 1173 | uvd->sbuf[i].data = NULL; |
1174 | uvd->sbuf[i].data = NULL; | ||
1175 | } | ||
1176 | } | 1174 | } |
1177 | } | 1175 | } |
1178 | } | 1176 | } |