]> git.baikalelectronics.ru Git - kernel.git/commit
drm/radeon: fix potential buffer overflow in ni_set_mc_special_registers()
authorAlexey Kodanev <aleksei.kodanev@bell-sw.com>
Mon, 6 Jun 2022 13:50:54 +0000 (16:50 +0300)
committerAlex Deucher <alexander.deucher@amd.com>
Mon, 6 Jun 2022 18:43:41 +0000 (14:43 -0400)
commit903ce2616fa8a3ac00fb52b1f2cc97d3b6a7d878
treeb72b5a6bbdabfbe2537726a2205c66ca1b44256f
parentfa95e3349603d7ac5f0ecab2415957b1717d2d21
drm/radeon: fix potential buffer overflow in ni_set_mc_special_registers()

The last case label can write two buffers 'mc_reg_address[j]' and
'mc_data[j]' with 'j' offset equal to SMC_NISLANDS_MC_REGISTER_ARRAY_SIZE
since there are no checks for this value in both case labels after the
last 'j++'.

Instead of changing '>' to '>=' there, add the bounds check at the start
of the second 'case' (the first one already has it).

Also, remove redundant last checks for 'j' index bigger than array size.
The expression is always false. Moreover, before or after the patch
'table->last' can be equal to SMC_NISLANDS_MC_REGISTER_ARRAY_SIZE and it
seems it can be a valid value.

Detected using the static analysis tool - Svace.
Fixes: 63214a79b637 ("drm/radeon/kms: add dpm support for cayman (v5)")
Signed-off-by: Alexey Kodanev <aleksei.kodanev@bell-sw.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
drivers/gpu/drm/radeon/ni_dpm.c