ALSA: hdsp: silence and underflow warning
Subsystems (2)
PATCH 1/1 ALSA: hdsp: silence and underflow warning
I believe this probably cannot happen, as the code suggests. There would have to be an kcontrol->index.id which was zero, otherwise this would be prevented in snd_ctl_find_id(). But snd_BUG_ON() is just a WARN() or a no-op so static checkers complain that we keep on going with a negative offset. Let's just handle the error as well as printing a warning.
sound/pci/rme9652/hdsp.c +2 -1
@@ -2806,7 +2806,8 @@ static int snd_hdsp_get_adat_sync_check(struct snd_kcontrol *kcontrol, struct snstruct hdsp *hdsp =snd_kcontrol_chip (kcontrol); offset = ucontrol->id.index -1 ; -snd_BUG_ON (offset <0 ); +if (snd_BUG_ON (offset <0 )) +return -EINVAL ;switch (hdsp->io_type) {case Digiface:
Takashi Iwai
AppliedQuoted 9 lines · Dan Carpenter
I believe this probably cannot happen, as the code suggests. There would have to be an
kcontrol->index.idwhich was zero, otherwise this would be prevented insnd_ctl_find_id(). Butsnd_BUG_ON()is just aWARN()or a no-op so static checkers complain that we keep on going with a negative offset. Let's just handle the error as well as printing a warning.Signed-off-by Dan Carpenter <[email protected]>
Applied, thanks.
Takashi
Quoted 16 lines
sound/pci/rme9652/hdsp.c +2
-1@@ -2806,7 +2806,8 @@ static int snd_hdsp_get_adat_sync_check(struct snd_kcontrol *kcontrol, struct snstruct hdsp *hdsp =snd_kcontrol_chip (kcontrol); offset = ucontrol->id.index -1 ; -snd_BUG_ON (offset <0 ); +if (snd_BUG_ON (offset <0 )) +return -EINVAL ;switch (hdsp->io_type) {case Digiface: