ALSA: hda: intel-nhlt: Correct the handling of fmt_config flexible array
authorPeter Ujfalusi <peter.ujfalusi@linux.intel.com>
Tue, 23 Aug 2022 12:24:05 +0000 (15:24 +0300)
committerTakashi Iwai <tiwai@suse.de>
Wed, 24 Aug 2022 05:59:10 +0000 (07:59 +0200)
The struct nhlt_format's fmt_config is a flexible array, it must not be
used as normal array.
When moving to the next nhlt_fmt_cfg we need to take into account the data
behind the ->config.caps (indicated by ->config.size).

Fixes: a864e8f159b13 ("ALSA: hda: intel-nhlt: verify config type")
Signed-off-by: Peter Ujfalusi <peter.ujfalusi@linux.intel.com>
Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Reviewed-by: Jaska Uimonen <jaska.uimonen@linux.intel.com>
Link: https://lore.kernel.org/r/20220823122405.18464-1-peter.ujfalusi@linux.intel.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
sound/hda/intel-nhlt.c

index 9db5ccd9aa2db12fcbea22c974e4dfb6f5f53b7e..13bb0ccfb36c032e9e4a14b35b2218f0116d0f92 100644 (file)
@@ -55,16 +55,22 @@ int intel_nhlt_get_dmic_geo(struct device *dev, struct nhlt_acpi_table *nhlt)
 
                /* find max number of channels based on format_configuration */
                if (fmt_configs->fmt_count) {
+                       struct nhlt_fmt_cfg *fmt_cfg = fmt_configs->fmt_config;
+
                        dev_dbg(dev, "found %d format definitions\n",
                                fmt_configs->fmt_count);
 
                        for (i = 0; i < fmt_configs->fmt_count; i++) {
                                struct wav_fmt_ext *fmt_ext;
 
-                               fmt_ext = &fmt_configs->fmt_config[i].fmt_ext;
+                               fmt_ext = &fmt_cfg->fmt_ext;
 
                                if (fmt_ext->fmt.channels > max_ch)
                                        max_ch = fmt_ext->fmt.channels;
+
+                               /* Move to the next nhlt_fmt_cfg */
+                               fmt_cfg = (struct nhlt_fmt_cfg *)(fmt_cfg->config.caps +
+                                                                 fmt_cfg->config.size);
                        }
                        dev_dbg(dev, "max channels found %d\n", max_ch);
                } else {