* drivers/iio/temperature/max31865.c:238:22: warning: use of uninitialized value '<unknown>' [CWE-457]
@ 2022-08-08 14:30 kernel test robot
0 siblings, 0 replies; only message in thread
From: kernel test robot @ 2022-08-08 14:30 UTC (permalink / raw)
To: kbuild
[-- Attachment #1: Type: text/plain, Size: 11146 bytes --]
::::::
:::::: Manual check reason: "low confidence bisect report"
:::::: Manual check reason: "low confidence static check warning: drivers/iio/temperature/max31865.c:238:22: warning: use of uninitialized value '<unknown>' [CWE-457] [-Wanalyzer-use-of-uninitialized-value]"
::::::
BCC: lkp(a)intel.com
CC: kbuild-all(a)lists.01.org
CC: linux-kernel(a)vger.kernel.org
TO: Navin Sankar Velliangiri <navin@linumiz.com>
CC: Jonathan Cameron <Jonathan.Cameron@huawei.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 4e23eeebb2e57f5a28b36221aa776b5a1122dde5
commit: e112dc4e18eafc5ee9d5700e3c059ac9897ae2a1 iio: temperature: Add MAX31865 RTD Support
date: 11 months ago
:::::: branch date: 14 hours ago
:::::: commit date: 11 months ago
config: arm-randconfig-c002-20220807 (https://download.01.org/0day-ci/archive/20220808/202208082213.GvFaWHca-lkp(a)intel.com/config)
compiler: arm-linux-gnueabi-gcc (GCC) 12.1.0
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
# https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=e112dc4e18eafc5ee9d5700e3c059ac9897ae2a1
git remote add linus https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git
git fetch --no-tags linus master
git checkout e112dc4e18eafc5ee9d5700e3c059ac9897ae2a1
# save the config file
COMPILER_INSTALL_PATH=$HOME/0day COMPILER=gcc-12.1.0 make.cross ARCH=arm KBUILD_USERCFLAGS='-fanalyzer -Wno-error'
If you fix the issue, kindly add following tag where applicable
Reported-by: kernel test robot <lkp@intel.com>
gcc-analyzer warnings: (new ones prefixed by >>)
| | ~
| | |
| | (43) following 'false' branch (when 'ret == 0')...
|......
| 131 | if (data->filter_50hz) {
| | ~~~~~~~~~~~~~~~~~
| | |
| | (44) ...to here
|......
| 140 | if (ret)
| | ~
| | |
| | (45) following 'false' branch (when 'ret == 0')...
|
'max31865_rtd_read': event 46
|
|include/asm-generic/unaligned.h:14:15:
| 14 | __pptr->x; \
| | ~~~~~~^~~
| | |
| | (46) ...to here
include/uapi/linux/swab.h:102:54: note: in definition of macro '__swab16'
| 102 | #define __swab16(x) (__u16)__builtin_bswap16((__u16)(x))
| | ^
include/linux/byteorder/generic.h:97:21: note: in expansion of macro '__be16_to_cpu'
| 97 | #define be16_to_cpu __be16_to_cpu
| | ^~~~~~~~~~~~~
include/asm-generic/unaligned.h:57:28: note: in expansion of macro '__get_unaligned_t'
| 57 | return be16_to_cpu(__get_unaligned_t(__be16, p));
| | ^~~~~~~~~~~~~~~~~
|
'max31865_rtd_read': event 47
|
|drivers/iio/temperature/max31865.c:145:16:
| 145 | return disable_bias(data);
| | ^~~~~~~~~~~~~~~~~~
| | |
| | (47) calling 'disable_bias' from 'max31865_rtd_read'
|
+--> 'disable_bias': events 48-51
|
| 87 | static int disable_bias(struct max31865_data *data)
| | ^~~~~~~~~~~~
| | |
| | (48) entry to 'disable_bias'
|......
| 93 | if (ret)
| | ~
| | |
| | (49) following 'false' branch (when 'ret == 0')...
|......
| 96 | cfg = data->buf[0];
| | ~~~~~~~~~~~~~~~~~~
| | |
| | (50) ...to here
|......
| 102 | return max31865_write(data, 2);
| | ~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (51) calling 'max31865_write' from 'disable_bias'
|
+--> 'max31865_write': events 52-53
|
| 65 | static int max31865_write(struct max31865_data *data, size_t len)
| | ^~~~~~~~~~~~~~
| | |
| | (52) entry to 'max31865_write'
| 66 | {
| 67 | return spi_write(data->spi, data->buf, len);
| | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (53) calling 'spi_write' from 'max31865_write'
|
+--> 'spi_write': events 54-55
|
|include/linux/spi/spi.h:1265:1:
| 1265 | spi_write(struct spi_device *spi, const void *buf, size_t len)
| | ^~~~~~~~~
| | |
| | (54) entry to 'spi_write'
|......
| 1272 | return spi_sync_transfer(spi, &t, 1);
| | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| | |
| | (55) calling 'spi_sync_transfer' from 'spi_write'
|
+--> 'spi_sync_transfer': events 56-58
|
| 1242 | spi_sync_transfer(struct spi_device *spi, struct spi_transfer *xfers,
| | ^~~~~~~~~~~~~~~~~
| | |
| | (56) entry to 'spi_sync_transfer'
|......
| 1245 | struct spi_message msg;
| | ~~~
| | |
| | (57) region created on stack here
| | (58) use of uninitialized value '*(int *)((char *)&msg + offsetof(struct spi_message, is_dma_mapped))' here
|
drivers/iio/temperature/max31865.c: In function 'set_filter':
>> drivers/iio/temperature/max31865.c:238:22: warning: use of uninitialized value '<unknown>' [CWE-457] [-Wanalyzer-use-of-uninitialized-value]
238 | unsigned int freq;
| ^~~~
'set_filter': event 1
|
| 238 | unsigned int freq;
| | ^~~~
| | |
| | (1) use of uninitialized value '<unknown>' here
|
vim +238 drivers/iio/temperature/max31865.c
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 230
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 231 static ssize_t set_filter(struct device *dev,
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 232 struct device_attribute *attr,
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 233 const char *buf,
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 234 size_t len)
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 235 {
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 236 struct iio_dev *indio_dev = dev_to_iio_dev(dev);
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 237 struct max31865_data *data = iio_priv(indio_dev);
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 @238 unsigned int freq;
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 239 int ret;
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 240
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 241 ret = kstrtouint(buf, 10, &freq);
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 242 if (ret)
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 243 return ret;
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 244
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 245 switch (freq) {
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 246 case 50:
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 247 data->filter_50hz = true;
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 248 break;
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 249 case 60:
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 250 data->filter_50hz = false;
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 251 break;
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 252 default:
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 253 return -EINVAL;
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 254 }
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 255
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 256 mutex_lock(&data->lock);
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 257 ret = max31865_init(data);
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 258 mutex_unlock(&data->lock);
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 259 if (ret)
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 260 return ret;
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 261
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 262 return len;
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 263 }
e112dc4e18eafc5 Navin Sankar Velliangiri 2021-08-24 264
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2022-08-08 14:30 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-08-08 14:30 drivers/iio/temperature/max31865.c:238:22: warning: use of uninitialized value '<unknown>' [CWE-457] kernel test robot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.