]> git.baikalelectronics.ru Git - kernel.git/commit
ALSA: update dell-wmi mic-mute registration to new world order
authorLinus Torvalds <torvalds@linux-foundation.org>
Thu, 16 Aug 2018 02:08:10 +0000 (19:08 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Thu, 16 Aug 2018 02:08:10 +0000 (19:08 -0700)
commit7834553f85f5335b9d5b3f99930d922513c8d0e1
treef8c024c5b683cf612a1fe7784bc50b2af0d4eabd
parent3695085341c24185f479611715640cc34ea2c097
ALSA: update dell-wmi mic-mute registration to new world order

Commit 4e3fbe9849e8 ("ALSA: hda - Allow multiple ADCs for mic mute LED
controls") changed the return value of the snd_hda_gen_add_micmute_led()
without actually updating the callers.

Admittedly, almost no callers actually cared about the return value.
But one call site very much did: the Dell wmi code.  It would see the
registration return zero, which _used_ to mean "failed" but now means
"success", and clear the dell_micmute_led_set_func pointer.

End result: the successful registration would end up calling the Dell
code that thought it had all failed, and call through a NULL pointer.

To make matters worse, it ends up being a tail-call, and with the
retpoline sequence you don't even see the caller (dell_micmute_update())
in the stack trace, so the error ended up way less obvious than it
should have been.

Fixes: 4e3fbe9849e8 "ALSA: hda - Allow multiple ADCs for mic mute LED controls"
Cc: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
sound/pci/hda/dell_wmi_helper.c