]> git.ipfire.org Git - thirdparty/kernel/linux.git/commit
ASoC: cs-amp-lib: Fix wrong sizeof() in _cs_amp_set_efi_calibration_data()
authorRichard Fitzgerald <rf@opensource.cirrus.com>
Thu, 21 May 2026 12:25:09 +0000 (13:25 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 21 May 2026 14:13:48 +0000 (15:13 +0100)
commit67a52d3ebb5a0ae0c0e23ffa99470d9463179c9f
treeec6bd91d6f534fb505da6776752a7e5707c4198a
parent5200f5f493f79f14bbdc349e402a40dfb32f23c8
ASoC: cs-amp-lib: Fix wrong sizeof() in _cs_amp_set_efi_calibration_data()

When calculating data->count replace the incorrect sizeof(data) with use
of struct_offset().

The faulty sizeof(data) was incorrectly calculating the size of the
pointer instead of the size of the struct pointed to. As it happens, both
values are 8 on a 64-bit CPU. In the unlikely event of using this code on
a 32-bit CPU the number of available bytes would be calculated 4 larger
than is actually available.

Instead of changing to sizeof(*data) it has been replaced by
struct_offset() because it has better chance of detecting these sorts of
typos. Also the offset of the data[] array is actually what we want to know
here anyway.

Signed-off-by: Richard Fitzgerald <rf@opensource.cirrus.com>
Fixes: 2b62e66626f0 ("ASoC: cs-amp-lib: Add function to write calibration to UEFI")
Link: https://patch.msgid.link/20260521122511.987322-2-rf@opensource.cirrus.com
Signed-off-by: Mark Brown <broonie@kernel.org>
sound/soc/codecs/cs-amp-lib.c