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 /sound/hda/hdac_controller.c | |
| parent | 9780ded39bef5d22a84bdc39112df93f70a58bdd (diff) | |
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>
Diffstat (limited to 'sound/hda/hdac_controller.c')
| -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 978dc1801b3a..8f7d0d9ed762 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; } |