drivers/gpu/drm/msm/adreno/a3xx_gpu.c:530 a3xx_gpu_init() warn: variable dereferenced before check 'pdev' (see line 525)
From: Dan Carpenter
Date: Thu Mar 19 2026 - 05:46:09 EST
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 8a30aeb0d1b4e4aaf7f7bae72f20f2ae75385ccb
commit: 491fadb2b818e1d900229a1f8243de6458381306 drm/msm/adreno: Move adreno_gpu_func to catalogue
config: arm64-randconfig-r073-20260319 (https://download.01.org/0day-ci/archive/20260319/202603191102.hueYyyPw-lkp@xxxxxxxxx/config)
compiler: clang version 23.0.0git (https://github.com/llvm/llvm-project 4abb927bacf37f18f6359a41639a6d1b3bffffb5)
smatch: v0.5.0-9004-gb810ac53
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@xxxxxxxxx>
| Reported-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>
| Closes: https://lore.kernel.org/r/202603191102.hueYyyPw-lkp@xxxxxxxxx/
smatch warnings:
drivers/gpu/drm/msm/adreno/a3xx_gpu.c:530 a3xx_gpu_init() warn: variable dereferenced before check 'pdev' (see line 525)
drivers/gpu/drm/msm/adreno/a2xx_gpu.c:503 a2xx_gpu_init() warn: variable dereferenced before check 'pdev' (see line 500)
drivers/gpu/drm/msm/adreno/a4xx_gpu.c:642 a4xx_gpu_init() warn: variable dereferenced before check 'pdev' (see line 637)
vim +/pdev +530 drivers/gpu/drm/msm/adreno/a3xx_gpu.c
491fadb2b818e1 Akhil P Oommen 2025-11-18 518 static struct msm_gpu *a3xx_gpu_init(struct drm_device *dev)
7198e6b03155f6 Rob Clark 2013-07-19 519 {
7198e6b03155f6 Rob Clark 2013-07-19 520 struct a3xx_gpu *a3xx_gpu = NULL;
55459968176f13 Rob Clark 2013-12-05 521 struct adreno_gpu *adreno_gpu;
7198e6b03155f6 Rob Clark 2013-07-19 522 struct msm_gpu *gpu;
060530f1ea6740 Rob Clark 2014-03-03 523 struct msm_drm_private *priv = dev->dev_private;
060530f1ea6740 Rob Clark 2014-03-03 524 struct platform_device *pdev = priv->gpu_pdev;
491fadb2b818e1 Akhil P Oommen 2025-11-18 @525 struct adreno_platform_config *config = pdev->dev.platform_data;
^^^^^^
Patch adds a dereference.
5785dd7a8ef0de Akhil P Oommen 2020-10-28 526 struct icc_path *ocmem_icc_path;
5785dd7a8ef0de Akhil P Oommen 2020-10-28 527 struct icc_path *icc_path;
7198e6b03155f6 Rob Clark 2013-07-19 528 int ret;
7198e6b03155f6 Rob Clark 2013-07-19 529
7198e6b03155f6 Rob Clark 2013-07-19 @530 if (!pdev) {
^^^^
But the old code assumed pdev could be NULL. Probably the NULL checks
are unnecessary.
6a41da17e87dee Mamta Shukla 2018-10-20 531 DRM_DEV_ERROR(dev->dev, "no a3xx device\n");
7198e6b03155f6 Rob Clark 2013-07-19 532 ret = -ENXIO;
7198e6b03155f6 Rob Clark 2013-07-19 533 goto fail;
7198e6b03155f6 Rob Clark 2013-07-19 534 }
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki