diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2023-12-04 13:42:07 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@kernel.org> | 2023-12-07 15:00:03 +0100 |
| commit | d685aea5e0a89b66679e5266320ab2ba4378c754 (patch) | |
| tree | ebaceaf43152cd18be5165632d6026097b93a9f7 /sound/soc/generic | |
| parent | ASoC: amd: acp: modify config flag read logic (diff) | |
| download | linux-d685aea5e0a89b66679e5266320ab2ba4378c754.tar.xz linux-d685aea5e0a89b66679e5266320ab2ba4378c754.zip | |
ASoC: audio-graph-card2: fix off by one in graph_parse_node_multi_nm()
The > comparison should be >= to avoid writing one element beyond the end
of the dai_link->ch_maps[] array. The dai_link->ch_maps[] array is
allocated in graph_parse_node_multi() and it has "nm_max" elements.
Fixes: e2de6808df4a ("ASoC: audio-graph-card2: add CPU:Codec = N:M support")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Acked-by: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://lore.kernel.org/r/1032216f-902f-48f9-aa49-9d5ece8e87f2@moroto.mountain
Signed-off-by: Mark Brown <broonie@kernel.org>
Diffstat (limited to 'sound/soc/generic')
| -rw-r--r-- | sound/soc/generic/audio-graph-card2.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/generic/audio-graph-card2.c b/sound/soc/generic/audio-graph-card2.c index d9e10308a508..78d9679decda 100644 --- a/sound/soc/generic/audio-graph-card2.c +++ b/sound/soc/generic/audio-graph-card2.c @@ -557,7 +557,7 @@ static int graph_parse_node_multi_nm(struct snd_soc_dai_link *dai_link, struct device_node *mcodec_port; int codec_idx; - if (*nm_idx > nm_max) + if (*nm_idx >= nm_max) break; mcpu_ep_n = of_get_next_child(mcpu_port, mcpu_ep_n); |