bianbu-repo-mirror/bianbu-linux-6.6
1
0
Fork 0
mirror of https://gitee.com/bianbu-linux/linux-6.6 synced 2025-04-24 14:07:52 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

ALSA: usb-audio: Fix endianess in descriptor validation

Browse source 
I overlooked that some fields are words and need the converts from
LE in the recently added USB descriptor validation code.
This patch fixes those with the proper macro usages.

Fixes: 57f8770620 ("ALSA: usb-audio: More validations of descriptor units")
Cc: <stable@vger.kernel.org>
Link: https://lore.kernel.org/r/20200201080530.22390-1-tiwai@suse.de
Signed-off-by: Takashi Iwai <tiwai@suse.de>
This commit is contained in:
Takashi Iwai 2020-02-01 09:05:28 +01:00
parent 78be2228c1
commit f8e5f90b3a
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
sound/usb/validate.c
View file

@ -110,7 +110,7 @@ static bool validate_processing_unit(const void *p,
default: default:
if (v->type == UAC1_EXTENSION_UNIT) if (v->type == UAC1_EXTENSION_UNIT)
return true; /* OK */ return true; /* OK */
switch (d->wProcessType) { switch (le16_to_cpu(d->wProcessType)) {
case UAC_PROCESS_UP_DOWNMIX: case UAC_PROCESS_UP_DOWNMIX:
case UAC_PROCESS_DOLBY_PROLOGIC: case UAC_PROCESS_DOLBY_PROLOGIC:
if (d->bLength < len + 1) /* bNrModes */ if (d->bLength < len + 1) /* bNrModes */
@ -125,7 +125,7 @@ static bool validate_processing_unit(const void *p,
case UAC_VERSION_2: case UAC_VERSION_2:
if (v->type == UAC2_EXTENSION_UNIT_V2) if (v->type == UAC2_EXTENSION_UNIT_V2)
return true; /* OK */ return true; /* OK */
switch (d->wProcessType) { switch (le16_to_cpu(d->wProcessType)) {
case UAC2_PROCESS_UP_DOWNMIX: case UAC2_PROCESS_UP_DOWNMIX:
case UAC2_PROCESS_DOLBY_PROLOCIC: /* SiC! */ case UAC2_PROCESS_DOLBY_PROLOCIC: /* SiC! */
if (d->bLength < len + 1) /* bNrModes */ if (d->bLength < len + 1) /* bNrModes */
@ -142,7 +142,7 @@ static bool validate_processing_unit(const void *p,
len += 2; /* wClusterDescrID */ len += 2; /* wClusterDescrID */
break; break;
} }
switch (d->wProcessType) { switch (le16_to_cpu(d->wProcessType)) {
case UAC3_PROCESS_UP_DOWNMIX: case UAC3_PROCESS_UP_DOWNMIX:
if (d->bLength < len + 1) /* bNrModes */ if (d->bLength < len + 1) /* bNrModes */
return false; return false;