:::::: :::::: 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: [email protected] CC: [email protected] CC: [email protected] TO: Navin Sankar Velliangiri <[email protected]> CC: Jonathan Cameron <[email protected]> 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/[email protected]/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 <[email protected]> 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 _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
