]> 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>
Wed, 22 Sep 2021 09:41:25 +0000 (11:41 +0200)
commit7ea5848b32d0e21e7bc7b19b6c0ab5ca618f3f89
tree1efe5f235de1f5479e0bf90af3b163e7e3109140
parent470f7a682dbd120b9a7b92715d0e57c9b98cc7cc
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