]> www.infradead.org Git - users/dwmw2/linux.git/commit
ASoC: dapm: fix bounds checker error in dapm_widget_list_create
authorAleksei Vetrov <vvvvvv@google.com>
Mon, 28 Oct 2024 22:50:30 +0000 (22:50 +0000)
committerMark Brown <broonie@kernel.org>
Mon, 28 Oct 2024 23:30:33 +0000 (23:30 +0000)
commit2ef9439f7a19fd3d43b288d38b1c6e55b668a4fe
tree656123a42a13fefa57125fe742f5f4f8fff49394
parentc1895ba181e560144601fafe46aeedbafdf4dbc4
ASoC: dapm: fix bounds checker error in dapm_widget_list_create

The widgets array in the snd_soc_dapm_widget_list has a __counted_by
attribute attached to it, which points to the num_widgets variable. This
attribute is used in bounds checking, and if it is not set before the
array is filled, then the bounds sanitizer will issue a warning or a
kernel panic if CONFIG_UBSAN_TRAP is set.

This patch sets the size of the widgets list calculated with
list_for_each as the initial value for num_widgets as it is used for
allocating memory for the array. It is updated with the actual number of
added elements after the array is filled.

Signed-off-by: Aleksei Vetrov <vvvvvv@google.com>
Fixes: 80e698e2df5b ("ASoC: soc-dapm: Annotate struct snd_soc_dapm_widget_list with __counted_by")
Link: https://patch.msgid.link/20241028-soc-dapm-bounds-checker-fix-v1-1-262b0394e89e@google.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/soc-dapm.c