staging: line6: fix memory leak in .hw_params()
authorStefan Hajnoczi <stefanha@gmail.com>
Sat, 10 Dec 2011 01:12:27 +0000 (02:12 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Sat, 10 Dec 2011 03:26:09 +0000 (19:26 -0800)
The .hw_params() pcm callback can be invoked multiple times in a row.
Ensure that the URB data buffer is only allocated once.

Signed-off-by: Stefan Hajnoczi <stefanha@gmail.com>
Signed-off-by: Markus Grabner <grabner@icg.tugraz.at>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/staging/line6/capture.c
drivers/staging/line6/playback.c

index ba441ed..8f59ff3 100644 (file)
@@ -316,8 +316,11 @@ static int snd_line6_capture_hw_params(struct snd_pcm_substream *substream,
        }
        /* -- [FD] end */
 
-       line6pcm->buffer_in = kmalloc(LINE6_ISO_BUFFERS * LINE6_ISO_PACKETS *
-                                     line6pcm->max_packet_size, GFP_KERNEL);
+       /* We may be invoked multiple times in a row so allocate once only */
+       if (!line6pcm->buffer_in)
+               line6pcm->buffer_in =
+                       kmalloc(LINE6_ISO_BUFFERS * LINE6_ISO_PACKETS *
+                               line6pcm->max_packet_size, GFP_KERNEL);
 
        if (!line6pcm->buffer_in) {
                dev_err(line6pcm->line6->ifcdev,
index e495b32..ed1b9bd 100644 (file)
@@ -462,8 +462,11 @@ static int snd_line6_playback_hw_params(struct snd_pcm_substream *substream,
        }
        /* -- [FD] end */
 
-       line6pcm->buffer_out = kmalloc(LINE6_ISO_BUFFERS * LINE6_ISO_PACKETS *
-                                      line6pcm->max_packet_size, GFP_KERNEL);
+       /* We may be invoked multiple times in a row so allocate once only */
+       if (!line6pcm->buffer_out)
+               line6pcm->buffer_out =
+                       kmalloc(LINE6_ISO_BUFFERS * LINE6_ISO_PACKETS *
+                               line6pcm->max_packet_size, GFP_KERNEL);
 
        if (!line6pcm->buffer_out) {
                dev_err(line6pcm->line6->ifcdev,