Revert "ALSA: usb-audio: purge needless variable length array"
authorTakashi Iwai <tiwai@suse.de>
Tue, 30 May 2017 07:23:41 +0000 (09:23 +0200)
committerTakashi Iwai <tiwai@suse.de>
Wed, 31 May 2017 06:46:14 +0000 (08:46 +0200)
commit64188cfbe5245d412de2139a3864e4e00b4136f0
tree7551a8af8eb7594e467e6051e96ffdb81b9f02a2
parent1fc2e41f7af4572b07190f9dec28396b418e9a36
Revert "ALSA: usb-audio: purge needless variable length array"

This reverts commit 89b593c30e83 ("ALSA: usb-audio: purge needless
variable length array").  The patch turned out to cause a severe
regression, triggering an Oops at snd_usb_ctl_msg().  It was overseen
that snd_usb_ctl_msg() writes back the response to the given buffer,
while the patch changed it to a read-only const buffer.  (One should
always double-check when an extra pointer cast is present...)

As a simple fix, just revert the affected commit.  It was merely a
cleanup.  Although it brings VLA again, it's clearer as a fix.  We'll
address the VLA later in another patch.

Fixes: 89b593c30e83 ("ALSA: usb-audio: purge needless variable length array")
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=195875
Cc: <stable@vger.kernel.org> # v4.11+
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/usb/mixer_us16x08.c