diff options
| author | David Thompson <davthompson@nvidia.com> | 2024-09-30 11:10:56 -0400 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2024-12-05 14:01:16 +0100 |
| commit | 000930193fe5eb79ce5563ee2e9ddb0c6e4e1bb5 (patch) | |
| tree | fc67e905020e35a247f59120720d8377488c1030 /init | |
| parent | 8d8404774465e9b2ea201121b9bfc81606feb5aa (diff) | |
| download | linux-000930193fe5eb79ce5563ee2e9ddb0c6e4e1bb5.tar.gz linux-000930193fe5eb79ce5563ee2e9ddb0c6e4e1bb5.tar.bz2 linux-000930193fe5eb79ce5563ee2e9ddb0c6e4e1bb5.zip | |
EDAC/bluefield: Fix potential integer overflow
[ Upstream commit 1fe774a93b46bb029b8f6fa9d1f25affa53f06c6 ]
The 64-bit argument for the "get DIMM info" SMC call consists of mem_ctrl_idx
left-shifted 16 bits and OR-ed with DIMM index. With mem_ctrl_idx defined as
32-bits wide the left-shift operation truncates the upper 16 bits of
information during the calculation of the SMC argument.
The mem_ctrl_idx stack variable must be defined as 64-bits wide to prevent any
potential integer overflow, i.e. loss of data from upper 16 bits.
Fixes: 82413e562ea6 ("EDAC, mellanox: Add ECC support for BlueField DDR4")
Signed-off-by: David Thompson <davthompson@nvidia.com>
Signed-off-by: Borislav Petkov (AMD) <bp@alien8.de>
Reviewed-by: Shravan Kumar Ramani <shravankr@nvidia.com>
Link: https://lore.kernel.org/r/20240930151056.10158-1-davthompson@nvidia.com
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'init')
0 files changed, 0 insertions, 0 deletions