]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
gpu: drm: amd: amdgpu: amdgpu_i2c: fix possible uninitialized-variable access in...
authorTuo Li <islituo@gmail.com>
Wed, 11 Aug 2021 11:34:58 +0000 (04:34 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 18 Sep 2021 11:42:11 +0000 (13:42 +0200)
commit4a54350944be766f08f886cd2d8c34b766448c80
treed7165aa18cde2041df42648730e9f2b2680b0b09
parent453fd12458d79d206c6485d7d40be49f61aceda9
gpu: drm: amd: amdgpu: amdgpu_i2c: fix possible uninitialized-variable access in amdgpu_i2c_router_select_ddc_port()

[ Upstream commit a211260c34cfadc6068fece8c9e99e0fe1e2a2b6 ]

The variable val is declared without initialization, and its address is
passed to amdgpu_i2c_get_byte(). In this function, the value of val is
accessed in:
  DRM_DEBUG("i2c 0x%02x 0x%02x read failed\n",
       addr, *val);

Also, when amdgpu_i2c_get_byte() returns, val may remain uninitialized,
but it is accessed in:
  val &= ~amdgpu_connector->router.ddc_mux_control_pin;

To fix this possible uninitialized-variable access, initialize val to 0 in
amdgpu_i2c_router_select_ddc_port().

Reported-by: TOTE Robot <oslab@tsinghua.edu.cn>
Signed-off-by: Tuo Li <islituo@gmail.com>
Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/gpu/drm/amd/amdgpu/amdgpu_i2c.c