Commit cfa7391a authored by Alex Deucher's avatar Alex Deucher Committed by Kamal Mostafa

drm/radeon/si: make sure mc ucode is loaded before checking the size

commit 8c79bae6 upstream.

Avoid a possible segfault.
Noticed-by: default avatarDan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: default avatarAlex Deucher <alexander.deucher@amd.com>
Signed-off-by: default avatarKamal Mostafa <kamal@canonical.com>
parent 78e11238
......@@ -1470,11 +1470,13 @@ static int si_mc_load_microcode(struct radeon_device *rdev)
const __be32 *fw_data;
u32 running, blackout = 0;
u32 *io_mc_regs;
int i, regs_size, ucode_size = rdev->mc_fw->size / 4;
int i, regs_size, ucode_size;
if (!rdev->mc_fw)
return -EINVAL;
ucode_size = rdev->mc_fw->size / 4;
switch (rdev->family) {
case CHIP_TAHITI:
io_mc_regs = (u32 *)&tahiti_io_mc_regs;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment