spi-imx: prevent overflow when estimating transfer time
authorAntonio Quartulli <antonio@mandelbit.com>
Fri, 15 Nov 2024 22:02:02 +0000 (23:02 +0100)
committerMark Brown <broonie@kernel.org>
Thu, 21 Nov 2024 13:14:33 +0000 (13:14 +0000)
commit9e141955ede223d82251a59644ff9448a5aba580
tree28483fcf10a112152ef438cd0f1927fcd03df686
parentc752e87b9c3982b78dddcdd70dcb826df3cfd75d
spi-imx: prevent overflow when estimating transfer time

The words delay is computed by multiplying two unsigned ints
and by adding up the result to a u64 variable.

The multiplication, however, is performed with 32bit math
thus losing data when the actual result is larger than UINT32_MAX.

Fix the operation by casting the first operand to u64, thus forcing
the multiplication to be performed with 64bit math.

This fixes 1 OVERFLOW_BEFORE_WIDEN issue reported by Coverity
Report: CID 1601859:  Integer handling issues  (OVERFLOW_BEFORE_WIDEN)

Cc: Mark Brown <broonie@kernel.org>
Cc: Shawn Guo <shawnguo@kernel.org>
Cc: Sascha Hauer <s.hauer@pengutronix.de>
Cc: Pengutronix Kernel Team <kernel@pengutronix.de>
Cc: Fabio Estevam <festevam@gmail.com>
Cc: imx@lists.linux.dev
Cc: linux-arm-kernel@lists.infradead.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Antonio Quartulli <antonio@mandelbit.com>
Link: https://patch.msgid.link/20241115220202.31086-1-antonio@mandelbit.com
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/spi/spi-imx.c