diff options
| author | Takashi Iwai <tiwai@suse.de> | 2014-11-20 15:39:14 -0500 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2014-11-20 15:44:49 -0500 |
| commit | 01cb156edbbd4e6c4fd8db0d05f18c62c424f9aa (patch) | |
| tree | 51cad682b4615ca9f2722e9f39ed68d4d012be96 | |
| parent | 6676f3081f7e3dae64e05b87d47a041b782f898a (diff) | |
ALSA: usb-audio: Use snd_usb_ctl_msg() for Native Instruments quirk
snd_nativeinstruments_control_get() uses a stack as a buffer for
usb_control_msg(), but it's basically not allowed. Replace the call
with a safer helper, snd_usb_ctl_msg(), instead.
Signed-off-by: Takashi Iwai <tiwai@suse.de>
| -rw-r--r-- | sound/usb/mixer_quirks.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/sound/usb/mixer_quirks.c b/sound/usb/mixer_quirks.c index 7c83bab69dee..8c9bf4b7aaf0 100644 --- a/sound/usb/mixer_quirks.c +++ b/sound/usb/mixer_quirks.c | |||
| @@ -593,10 +593,10 @@ static int snd_nativeinstruments_control_get(struct snd_kcontrol *kcontrol, | |||
| 593 | if (mixer->chip->shutdown) | 593 | if (mixer->chip->shutdown) |
| 594 | ret = -ENODEV; | 594 | ret = -ENODEV; |
| 595 | else | 595 | else |
| 596 | ret = usb_control_msg(dev, usb_rcvctrlpipe(dev, 0), bRequest, | 596 | ret = snd_usb_ctl_msg(dev, usb_rcvctrlpipe(dev, 0), bRequest, |
| 597 | USB_TYPE_VENDOR | USB_RECIP_DEVICE | USB_DIR_IN, | 597 | USB_TYPE_VENDOR | USB_RECIP_DEVICE | USB_DIR_IN, |
| 598 | 0, wIndex, | 598 | 0, wIndex, |
| 599 | &tmp, sizeof(tmp), 1000); | 599 | &tmp, sizeof(tmp)); |
| 600 | up_read(&mixer->chip->shutdown_rwsem); | 600 | up_read(&mixer->chip->shutdown_rwsem); |
| 601 | 601 | ||
| 602 | if (ret < 0) { | 602 | if (ret < 0) { |
