Reference in a new issue

Blame

Copy permalink

Reference in a new issue

Copy permalink

Andrey Vatoropin 9f3891867e power: supply: da9150-fg: fix potential overflow ... [ Upstream commit 3fb3cb4350 ] Size of variable sd_gain equals four bytes - DA9150_QIF_SD_GAIN_SIZE. Size of variable shunt_val equals two bytes - DA9150_QIF_SHUNT_VAL_SIZE. The expression sd_gain * shunt_val is currently being evaluated using 32-bit arithmetic. So during the multiplication an overflow may occur. As the value of type 'u64' is used as storage for the eventual result, put ULL variable at the first position of each expression in order to give the compiler complete information about the proper arithmetic to use. According to C99 the guaranteed width for a variable of type 'unsigned long long' >= 64 bits. Remove the explicit cast to u64 as it is meaningless. Just for the sake of consistency, perform the similar trick with another expression concerning 'iavg'. Found by Linux Verification Center (linuxtesting.org) with SVACE. Fixes: a419b4fd91 ("power: Add support for DA9150 Fuel-Gauge") Signed-off-by: Andrey Vatoropin <a.vatoropin@crpt.ru> Link: https://lore.kernel.org/r/20250130090030.53422-1-a.vatoropin@crpt.ru Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com> Signed-off-by: Sasha Levin <sashal@kernel.org>		History 2025-03-13 12:50:53 +01:00
..
reset	power: reset: brcmstb: Do not go into infinite loop if reset fails	2024-10-17 15:11:29 +02:00
supply	power: supply: da9150-fg: fix potential overflow	2025-03-13 12:50:53 +01:00
Kconfig	PM: AVS: Drop the avs directory and the corresponding Kconfig	2020-10-22 18:39:32 +02:00
Makefile	PM: AVS: Drop the avs directory and the corresponding Kconfig	2020-10-22 18:39:32 +02:00