From 578e2d761554130f7c6abfe821c2509912a00ac6 Mon Sep 17 00:00:00 2001 From: Timothy Pearson Date: Sun, 17 Jun 2018 23:59:51 -0500 Subject: Fix undefined behaviour from signed integer overflow Caught by UBSAN Signed-off-by: Timothy Pearson --- drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c b/drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c index d644a9b..a9c386f 100644 --- a/drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c +++ b/drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c @@ -98,7 +98,7 @@ int smu7_copy_bytes_to_smc(struct pp_hwmgr *hwmgr, uint32_t smc_start_address, while (byte_count >= 4) { /* Bytes are written into the SMC addres space with the MSB first. */ - data = src[0] * 0x1000000 + src[1] * 0x10000 + src[2] * 0x100 + src[3]; + data = src[0] * 0x1000000U + src[1] * 0x10000U + src[2] * 0x100U + src[3]; result = smu7_set_smc_sram_address(hwmgr, addr, limit); -- cgit v1.1