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)
committerAlex Deucher <alexander.deucher@amd.com>
Wed, 11 Aug 2021 21:19:54 +0000 (17:19 -0400)
commita211260c34cfadc6068fece8c9e99e0fe1e2a2b6
tree0fb6c8f655adc3eb400a6d87b1a4a5a5919b7066
parentb53ef0df1ba8001b17da2f972cbc1f6091d1774a
gpu: drm: amd: amdgpu: amdgpu_i2c: fix possible uninitialized-variable access in amdgpu_i2c_router_select_ddc_port()

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>
drivers/gpu/drm/amd/amdgpu/amdgpu_i2c.c