diff options
| author | Rakesh Ughreja <rakesh.a.ughreja@intel.com> | 2017-10-24 18:26:47 +0530 |
|---|---|---|
| committer | Takashi Iwai <tiwai@suse.de> | 2017-10-24 08:13:55 +0200 |
| commit | b676da70c495acb2515de76300596e9147806ead (patch) | |
| tree | 2c5707619daeed3fac126c478b138433e6781307 | |
| parent | 9780ded39bef5d22a84bdc39112df93f70a58bdd (diff) | |
| download | op-kernel-dev-b676da70c495acb2515de76300596e9147806ead.zip op-kernel-dev-b676da70c495acb2515de76300596e9147806ead.tar.gz | |
ALSA: hda: Abort capability probe on invalid capability
On reading wrong capability pointer values driver may crash, so whenever
driver discovers unknown HDA capability, log it as error and stop traversing
the link list further.
Signed-off-by: Rakesh Ughreja <rakesh.a.ughreja@intel.com>
Acked-by: Vinod Koul <vinod.koul@intel.com>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
| -rw-r--r-- | sound/hda/hdac_controller.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/sound/hda/hdac_controller.c b/sound/hda/hdac_controller.c index 978dc18..8f7d0d9 100644 --- a/sound/hda/hdac_controller.c +++ b/sound/hda/hdac_controller.c @@ -314,7 +314,8 @@ int snd_hdac_bus_parse_capabilities(struct hdac_bus *bus) break; default: - dev_dbg(bus->dev, "Unknown capability %d\n", cur_cap); + dev_err(bus->dev, "Unknown capability %d\n", cur_cap); + cur_cap = 0; break; } |
