lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [day] [month] [year] [list]
Message-ID: <202207311957.wVEYCh2c-lkp@intel.com>
Date:   Sun, 31 Jul 2022 19:56:46 +0800
From:   kernel test robot <lkp@...el.com>
To:     Dmitry Baryshkov <dmitry.baryshkov@...aro.org>
Cc:     llvm@...ts.linux.dev, kbuild-all@...ts.01.org,
        linux-kernel@...r.kernel.org, Steev Klimaszewski <steev@...i.org>
Subject: [steev:linux-v5.19.0-rc8-tests 146/182]
 drivers/gpu/drm/msm/dsi/dsi_host.c:1997:14: warning: variable 'device_node'
 is uninitialized when used here

tree:   https://github.com/steev/linux linux-v5.19.0-rc8-tests
head:   171125613e081f0d0d9aeb710dc7469ce2c6a219
commit: d99732349a84f6046d58df883c6c68fd7351faa9 [146/182] drm/msm/dsi: switch to DRM_PANEL_BRIDGE
config: arm64-buildonly-randconfig-r005-20220731 (https://download.01.org/0day-ci/archive/20220731/202207311957.wVEYCh2c-lkp@intel.com/config)
compiler: clang version 16.0.0 (https://github.com/llvm/llvm-project 52cd00cabf479aa7eb6dbb063b7ba41ea57bce9e)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm64 cross compiling tool for clang build
        # apt-get install binutils-aarch64-linux-gnu
        # https://github.com/steev/linux/commit/d99732349a84f6046d58df883c6c68fd7351faa9
        git remote add steev https://github.com/steev/linux
        git fetch --no-tags steev linux-v5.19.0-rc8-tests
        git checkout d99732349a84f6046d58df883c6c68fd7351faa9
        # save the config file
        mkdir build_dir && cp config build_dir/.config
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm64 SHELL=/bin/bash drivers/gpu/drm/msm/

If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@...el.com>

All warnings (new ones prefixed by >>):

>> drivers/gpu/drm/msm/dsi/dsi_host.c:1997:14: warning: variable 'device_node' is uninitialized when used here [-Wuninitialized]
           of_node_put(device_node);
                       ^~~~~~~~~~~
   drivers/gpu/drm/msm/dsi/dsi_host.c:1964:44: note: initialize the variable 'device_node' to silence this warning
           struct device_node *endpoint, *device_node;
                                                     ^
                                                      = NULL
   1 warning generated.
--
>> drivers/gpu/drm/msm/dsi/dsi_manager.c:533:7: warning: variable 'connector' is used uninitialized whenever 'if' condition is false [-Wsometimes-uninitialized]
                   if (ret < 0)
                       ^~~~~~~
   drivers/gpu/drm/msm/dsi/dsi_manager.c:551:9: note: uninitialized use occurs here
           return connector;
                  ^~~~~~~~~
   drivers/gpu/drm/msm/dsi/dsi_manager.c:533:3: note: remove the 'if' if its condition is always true
                   if (ret < 0)
                   ^~~~~~~~~~~~
   drivers/gpu/drm/msm/dsi/dsi_manager.c:509:33: note: initialize the variable 'connector' to silence this warning
           struct drm_connector *connector;
                                          ^
                                           = NULL
   1 warning generated.


vim +/device_node +1997 drivers/gpu/drm/msm/dsi/dsi_host.c

b9080324d6ca24 Vinod Koul    2022-04-06  1959  
f7009d266d8b2f Archit Taneja 2015-06-25  1960  static int dsi_host_parse_dt(struct msm_dsi_host *msm_host)
f7009d266d8b2f Archit Taneja 2015-06-25  1961  {
f7009d266d8b2f Archit Taneja 2015-06-25  1962  	struct device *dev = &msm_host->pdev->dev;
f7009d266d8b2f Archit Taneja 2015-06-25  1963  	struct device_node *np = dev->of_node;
a9ddac9c576571 Archit Taneja 2015-08-03  1964  	struct device_node *endpoint, *device_node;
a1b1a4f7e4c3cf Archit Taneja 2017-01-04  1965  	int ret = 0;
f7009d266d8b2f Archit Taneja 2015-06-25  1966  
f7009d266d8b2f Archit Taneja 2015-06-25  1967  	/*
b9ac76f6ac9841 Archit Taneja 2016-04-27  1968  	 * Get the endpoint of the output port of the DSI host. In our case,
b9ac76f6ac9841 Archit Taneja 2016-04-27  1969  	 * this is mapped to port number with reg = 1. Don't return an error if
b9ac76f6ac9841 Archit Taneja 2016-04-27  1970  	 * the remote endpoint isn't defined. It's possible that there is
b9ac76f6ac9841 Archit Taneja 2016-04-27  1971  	 * nothing connected to the dsi output.
f7009d266d8b2f Archit Taneja 2015-06-25  1972  	 */
b9ac76f6ac9841 Archit Taneja 2016-04-27  1973  	endpoint = of_graph_get_endpoint_by_regs(np, 1, -1);
f7009d266d8b2f Archit Taneja 2015-06-25  1974  	if (!endpoint) {
6a41da17e87dee Mamta Shukla  2018-10-20  1975  		DRM_DEV_DEBUG(dev, "%s: no endpoint\n", __func__);
f7009d266d8b2f Archit Taneja 2015-06-25  1976  		return 0;
f7009d266d8b2f Archit Taneja 2015-06-25  1977  	}
f7009d266d8b2f Archit Taneja 2015-06-25  1978  
26f7d1f4d9ab10 Archit Taneja 2016-02-25  1979  	ret = dsi_host_parse_lane_data(msm_host, endpoint);
26f7d1f4d9ab10 Archit Taneja 2016-02-25  1980  	if (ret) {
6a41da17e87dee Mamta Shukla  2018-10-20  1981  		DRM_DEV_ERROR(dev, "%s: invalid lane configuration %d\n",
26f7d1f4d9ab10 Archit Taneja 2016-02-25  1982  			__func__, ret);
feb085ec8a3dc1 Sean Paul     2018-07-30  1983  		ret = -EINVAL;
26f7d1f4d9ab10 Archit Taneja 2016-02-25  1984  		goto err;
26f7d1f4d9ab10 Archit Taneja 2016-02-25  1985  	}
26f7d1f4d9ab10 Archit Taneja 2016-02-25  1986  
0c7df47f6a4c5d Archit Taneja 2015-10-14  1987  	if (of_property_read_bool(np, "syscon-sfpb")) {
0c7df47f6a4c5d Archit Taneja 2015-10-14  1988  		msm_host->sfpb = syscon_regmap_lookup_by_phandle(np,
0c7df47f6a4c5d Archit Taneja 2015-10-14  1989  					"syscon-sfpb");
0c7df47f6a4c5d Archit Taneja 2015-10-14  1990  		if (IS_ERR(msm_host->sfpb)) {
6a41da17e87dee Mamta Shukla  2018-10-20  1991  			DRM_DEV_ERROR(dev, "%s: failed to get sfpb regmap\n",
0c7df47f6a4c5d Archit Taneja 2015-10-14  1992  				__func__);
26f7d1f4d9ab10 Archit Taneja 2016-02-25  1993  			ret = PTR_ERR(msm_host->sfpb);
0c7df47f6a4c5d Archit Taneja 2015-10-14  1994  		}
0c7df47f6a4c5d Archit Taneja 2015-10-14  1995  	}
0c7df47f6a4c5d Archit Taneja 2015-10-14  1996  
26f7d1f4d9ab10 Archit Taneja 2016-02-25 @1997  	of_node_put(device_node);
26f7d1f4d9ab10 Archit Taneja 2016-02-25  1998  
26f7d1f4d9ab10 Archit Taneja 2016-02-25  1999  err:
26f7d1f4d9ab10 Archit Taneja 2016-02-25  2000  	of_node_put(endpoint);
26f7d1f4d9ab10 Archit Taneja 2016-02-25  2001  
26f7d1f4d9ab10 Archit Taneja 2016-02-25  2002  	return ret;
f7009d266d8b2f Archit Taneja 2015-06-25  2003  }
f7009d266d8b2f Archit Taneja 2015-06-25  2004  

:::::: The code at line 1997 was first introduced by commit
:::::: 26f7d1f4d9ab108b28b8d0f340b98941073c6f4d drm/msm/dsi: Parse DSI lanes via DT

:::::: TO: Archit Taneja <architt@...eaurora.org>
:::::: CC: Rob Clark <robdclark@...il.com>

-- 
0-DAY CI Kernel Test Service
https://01.org/lkp

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ