diff options
author | Timothy Pearson <tpearson@raptorengineering.com> | 2018-06-17 23:59:51 -0500 |
---|---|---|
committer | root <root@rcs-power9-talos> | 2018-10-27 02:51:26 -0400 |
commit | 1f0e220a0875e611e426a6b55f546254714a8ca4 (patch) | |
tree | 5c63f506689b809f1ac48240225f9da0d03cf318 /drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c | |
parent | 9d8bed4d5db2e83b862d4c750e51dab3ce6d6bfc (diff) | |
download | op-kernel-dev-1f0e220a0875e611e426a6b55f546254714a8ca4.zip op-kernel-dev-1f0e220a0875e611e426a6b55f546254714a8ca4.tar.gz |
Fix undefined behaviour from signed integer overflow
Caught by UBSAN
Signed-off-by: Timothy Pearson <tpearson@raptorengineering.com>
Diffstat (limited to 'drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c')
-rw-r--r-- | drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c b/drivers/gpu/drm/amd/powerplay/smumgr/smu7_smumgr.c index a029e47..f7e56be 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); |