ALSA: line6: Fix memory leak at line6_init_pcm() error path

I forgot to release the allocated object at the early error path in
line6_init_pcm().  For addressing it, slightly shuffle the code so
that the PCM destructor (pcm->private_free) is assigned properly
before all error paths.

Fixes: 3450121997ce ("ALSA: line6: Fix write on zero-sized buffer")
Cc: <stable@vger.kernel.org>
Signed-off-by: Takashi Iwai <tiwai@suse.de>

1 files changed, 9 insertions, 9 deletions

diff --git a/sound/usb/line6/pcm.c b/sound/usb/line6/pcm.c
index 2c03e0f6bf72..f70211e6b174 100644
--- a/sound/usb/line6/pcm.c
+++ b/sound/usb/line6/pcm.c
@@ -550,6 +550,15 @@ int line6_init_pcm(struct usb_line6 *line6,
         line6pcm->volume_monitor = 255;
         line6pcm->line6 = line6;
 
+        spin_lock_init(&line6pcm->out.lock);
+        spin_lock_init(&line6pcm->in.lock);
+        line6pcm->impulse_period = LINE6_IMPULSE_DEFAULT_PERIOD;
+
+        line6->line6pcm = line6pcm;
+
+        pcm->private_data = line6pcm;
+        pcm->private_free = line6_cleanup_pcm;
+
         line6pcm->max_packet_size_in =
                 usb_maxpacket(line6->usbdev,
                         usb_rcvisocpipe(line6->usbdev, ep_read), 0);
@@ -562,15 +571,6 @@ int line6_init_pcm(struct usb_line6 *line6,
                 return -EINVAL;
         }
 
-        spin_lock_init(&line6pcm->out.lock);
-        spin_lock_init(&line6pcm->in.lock);
-        line6pcm->impulse_period = LINE6_IMPULSE_DEFAULT_PERIOD;
-
-        line6->line6pcm = line6pcm;
-
-        pcm->private_data = line6pcm;
-        pcm->private_free = line6_cleanup_pcm;
-
         err = line6_create_audio_out_urbs(line6pcm);
         if (err < 0)
                 return err;