linux-kernel - drivers/firmware/stratix10-rsu.c:293:21: sparse: sparse: Using plain integer as NULL pointer

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]

Date:   Wed, 3 Jun 2020 05:09:53 +0800
From:   kbuild test robot <lkp@...el.com>
To:     "Richard, Gong," <richard.gong@...el.com>
Cc:     kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>
Subject: drivers/firmware/stratix10-rsu.c:293:21: sparse: sparse: Using plain
 integer as NULL pointer

Hi Richard,

First bad commit (maybe != root cause):

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   f359287765c04711ff54fbd11645271d8e5ff763
commit: 231d901d1e368c47c4786e826195520e0000a5ea firmware: intel_stratix10_service: add depend on agilex
date:   3 months ago
config: arm64-randconfig-s032-20200602 (attached as .config)
compiler: aarch64-linux-gcc (GCC) 9.3.0
reproduce:
        # apt-get install sparse
        # sparse version: v0.6.1-244-g0ee050a8-dirty
        git checkout 231d901d1e368c47c4786e826195520e0000a5ea
        # save the attached .config to linux build tree
        make W=1 C=1 ARCH=arm64 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'

If you fix the issue, kindly add following tag as appropriate
Reported-by: kbuild test robot <lkp@...el.com>


sparse warnings: (new ones prefixed by >>)

>> drivers/firmware/stratix10-rsu.c:293:21: sparse: sparse: Using plain integer as NULL pointer
   drivers/firmware/stratix10-rsu.c:318:21: sparse: sparse: Using plain integer as NULL pointer

vim +293 drivers/firmware/stratix10-rsu.c

4526ebbc77732b Richard Gong 2019-09-03  284  
4526ebbc77732b Richard Gong 2019-09-03  285  static ssize_t reboot_image_store(struct device *dev,
4526ebbc77732b Richard Gong 2019-09-03  286  				  struct device_attribute *attr,
4526ebbc77732b Richard Gong 2019-09-03  287  				  const char *buf, size_t count)
4526ebbc77732b Richard Gong 2019-09-03  288  {
4526ebbc77732b Richard Gong 2019-09-03  289  	struct stratix10_rsu_priv *priv = dev_get_drvdata(dev);
4526ebbc77732b Richard Gong 2019-09-03  290  	unsigned long address;
4526ebbc77732b Richard Gong 2019-09-03  291  	int ret;
4526ebbc77732b Richard Gong 2019-09-03  292  
4526ebbc77732b Richard Gong 2019-09-03 @293  	if (priv == 0)
4526ebbc77732b Richard Gong 2019-09-03  294  		return -ENODEV;
4526ebbc77732b Richard Gong 2019-09-03  295  
4526ebbc77732b Richard Gong 2019-09-03  296  	ret = kstrtoul(buf, 0, &address);
4526ebbc77732b Richard Gong 2019-09-03  297  	if (ret)
4526ebbc77732b Richard Gong 2019-09-03  298  		return ret;
4526ebbc77732b Richard Gong 2019-09-03  299  
4526ebbc77732b Richard Gong 2019-09-03  300  	ret = rsu_send_msg(priv, COMMAND_RSU_UPDATE,
4526ebbc77732b Richard Gong 2019-09-03  301  			   address, rsu_command_callback);
4526ebbc77732b Richard Gong 2019-09-03  302  	if (ret) {
4526ebbc77732b Richard Gong 2019-09-03  303  		dev_err(dev, "Error, RSU update returned %i\n", ret);
4526ebbc77732b Richard Gong 2019-09-03  304  		return ret;
4526ebbc77732b Richard Gong 2019-09-03  305  	}
4526ebbc77732b Richard Gong 2019-09-03  306  
4526ebbc77732b Richard Gong 2019-09-03  307  	return count;
4526ebbc77732b Richard Gong 2019-09-03  308  }
4526ebbc77732b Richard Gong 2019-09-03  309  

:::::: The code at line 293 was first introduced by commit
:::::: 4526ebbc77732bcae965ee374cf3e8d86436b2ad firmware: add Intel Stratix10 remote system update driver

:::::: TO: Richard Gong <richard.gong@...el.com>
:::::: CC: Greg Kroah-Hartman <gregkh@...uxfoundation.org>

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org

Download attachment ".config.gz" of type "application/gzip" (32093 bytes)