Unverified Commit 16719d48 authored by Qasim Ijaz's avatar Qasim Ijaz Committed by Mark Brown
Browse files

ASoC: codecs: fix out-of-bounds access on invalid clock config 



get_coeff() returns –EINVAL when no table entry matches.
The driver then uses that value as an index into coeff_div[],
causing an OOB access.

To fix lets abort the hw_params call instead.

Fixes: de2b3119 ("ASoC: codecs: add support for ES8375")
Signed-off-by: default avatarQasim Ijaz <qasdev00@gmail.com>
Link: https://patch.msgid.link/20250526191820.72577-1-qasdev00@gmail.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent f4ba2ea5

sound/soc/codecs/es8375.c

+1 −0
Original line number Diff line number Diff line
@@ -319,6 +319,7 @@ static int es8375_hw_params(struct snd_pcm_substream *substream, 
	coeff = get_coeff(es8375->vddd, dmic_enable, es8375->mclk_freq, params_rate(params));

	if (coeff < 0) {

		dev_warn(component->dev, "Clock coefficients do not match");

		return coeff;

	}

	regmap_write(es8375->regmap, ES8375_CLK_MGR4,

			coeff_div[coeff].Reg0x04);