linux-kernel - drivers/spi/spi-stm32-qspi.c:468:27: warning: 'op' is used uninitialized

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:   Fri, 28 Jul 2023 09:13:18 +0800
From:   kernel test robot <lkp@...el.com>
To:     Johannes Berg <johannes.berg@...el.com>
Cc:     oe-kbuild-all@...ts.linux.dev, linux-kernel@...r.kernel.org,
        Richard Weinberger <richard@....at>
Subject: drivers/spi/spi-stm32-qspi.c:468:27: warning: 'op' is used
 uninitialized

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   57012c57536f8814dec92e74197ee96c3498d24e
commit: 68f5d3f3b6543266b29e047cfaf9842333019b4c um: add PCI over virtio emulation driver
date:   2 years, 1 month ago
config: um-allyesconfig (https://download.01.org/0day-ci/archive/20230728/202307280915.PELlO3TQ-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce: (https://download.01.org/0day-ci/archive/20230728/202307280915.PELlO3TQ-lkp@intel.com/reproduce)

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@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202307280915.PELlO3TQ-lkp@intel.com/

All warnings (new ones prefixed by >>):

   drivers/spi/spi-stm32-qspi.c: In function 'stm32_qspi_dirmap_read':
>> drivers/spi/spi-stm32-qspi.c:468:27: warning: 'op' is used uninitialized [-Wuninitialized]
     468 |         struct spi_mem_op op;
         |                           ^~
   drivers/spi/spi-stm32-qspi.c:468:27: note: 'op' declared here
     468 |         struct spi_mem_op op;
         |                           ^~


vim +/op +468 drivers/spi/spi-stm32-qspi.c

18674dee3cd651 Patrice Chotard 2021-04-19  463  
18674dee3cd651 Patrice Chotard 2021-04-19  464  static ssize_t stm32_qspi_dirmap_read(struct spi_mem_dirmap_desc *desc,
18674dee3cd651 Patrice Chotard 2021-04-19  465  				      u64 offs, size_t len, void *buf)
18674dee3cd651 Patrice Chotard 2021-04-19  466  {
18674dee3cd651 Patrice Chotard 2021-04-19  467  	struct stm32_qspi *qspi = spi_controller_get_devdata(desc->mem->spi->master);
18674dee3cd651 Patrice Chotard 2021-04-19 @468  	struct spi_mem_op op;
18674dee3cd651 Patrice Chotard 2021-04-19  469  	u32 addr_max;
18674dee3cd651 Patrice Chotard 2021-04-19  470  	int ret;
18674dee3cd651 Patrice Chotard 2021-04-19  471  
18674dee3cd651 Patrice Chotard 2021-04-19  472  	ret = pm_runtime_get_sync(qspi->dev);
18674dee3cd651 Patrice Chotard 2021-04-19  473  	if (ret < 0) {
18674dee3cd651 Patrice Chotard 2021-04-19  474  		pm_runtime_put_noidle(qspi->dev);
18674dee3cd651 Patrice Chotard 2021-04-19  475  		return ret;
18674dee3cd651 Patrice Chotard 2021-04-19  476  	}
18674dee3cd651 Patrice Chotard 2021-04-19  477  
18674dee3cd651 Patrice Chotard 2021-04-19  478  	mutex_lock(&qspi->lock);
18674dee3cd651 Patrice Chotard 2021-04-19  479  	/* make a local copy of desc op_tmpl and complete dirmap rdesc
18674dee3cd651 Patrice Chotard 2021-04-19  480  	 * spi_mem_op template with offs, len and *buf in  order to get
18674dee3cd651 Patrice Chotard 2021-04-19  481  	 * all needed transfer information into struct spi_mem_op
18674dee3cd651 Patrice Chotard 2021-04-19  482  	 */
18674dee3cd651 Patrice Chotard 2021-04-19  483  	memcpy(&op, &desc->info.op_tmpl, sizeof(struct spi_mem_op));
14ef64ebdc2a45 Arnd Bergmann   2021-04-22  484  	dev_dbg(qspi->dev, "%s len = 0x%zx offs = 0x%llx buf = 0x%p\n", __func__, len, offs, buf);
18674dee3cd651 Patrice Chotard 2021-04-19  485  
18674dee3cd651 Patrice Chotard 2021-04-19  486  	op.data.nbytes = len;
18674dee3cd651 Patrice Chotard 2021-04-19  487  	op.addr.val = desc->info.offset + offs;
18674dee3cd651 Patrice Chotard 2021-04-19  488  	op.data.buf.in = buf;
18674dee3cd651 Patrice Chotard 2021-04-19  489  
18674dee3cd651 Patrice Chotard 2021-04-19  490  	addr_max = op.addr.val + op.data.nbytes + 1;
18674dee3cd651 Patrice Chotard 2021-04-19  491  	if (addr_max < qspi->mm_size && op.addr.buswidth)
18674dee3cd651 Patrice Chotard 2021-04-19  492  		qspi->fmode = CCR_FMODE_MM;
18674dee3cd651 Patrice Chotard 2021-04-19  493  	else
18674dee3cd651 Patrice Chotard 2021-04-19  494  		qspi->fmode = CCR_FMODE_INDR;
18674dee3cd651 Patrice Chotard 2021-04-19  495  
18674dee3cd651 Patrice Chotard 2021-04-19  496  	ret = stm32_qspi_send(desc->mem, &op);
18674dee3cd651 Patrice Chotard 2021-04-19  497  	mutex_unlock(&qspi->lock);
18674dee3cd651 Patrice Chotard 2021-04-19  498  
18674dee3cd651 Patrice Chotard 2021-04-19  499  	pm_runtime_mark_last_busy(qspi->dev);
18674dee3cd651 Patrice Chotard 2021-04-19  500  	pm_runtime_put_autosuspend(qspi->dev);
18674dee3cd651 Patrice Chotard 2021-04-19  501  
18674dee3cd651 Patrice Chotard 2021-04-19  502  	return ret ?: len;
18674dee3cd651 Patrice Chotard 2021-04-19  503  }
18674dee3cd651 Patrice Chotard 2021-04-19  504  

:::::: The code at line 468 was first introduced by commit
:::::: 18674dee3cd651279eb3d9ba789fe483ddfe1137 spi: stm32-qspi: Add dirmap support

:::::: TO: Patrice Chotard <patrice.chotard@...s.st.com>
:::::: CC: Mark Brown <broonie@...nel.org>

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki