From: Jesper Juhl There's no need to check pointers passed to vfree() for NULL. Signed-off-by: Jesper Juhl Cc: Takashi Iwai Cc: Jaroslav Kysela Cc: Greg KH Signed-off-by: Andrew Morton --- sound/usb/usbaudio.c | 7 +++---- 1 files changed, 3 insertions(+), 4 deletions(-) diff -puN sound/usb/usbaudio.c~dont-check-vfree-arg-for-null-in-usbaudio sound/usb/usbaudio.c --- 25/sound/usb/usbaudio.c~dont-check-vfree-arg-for-null-in-usbaudio Fri Feb 24 15:10:16 2006 +++ 25-akpm/sound/usb/usbaudio.c Fri Feb 24 15:10:16 2006 @@ -720,10 +720,9 @@ static int snd_pcm_alloc_vmalloc_buffer( static int snd_pcm_free_vmalloc_buffer(struct snd_pcm_substream *subs) { struct snd_pcm_runtime *runtime = subs->runtime; - if (runtime->dma_area) { - vfree(runtime->dma_area); - runtime->dma_area = NULL; - } + + vfree(runtime->dma_area); + runtime->dma_area = NULL; return 0; } _