* drivers/hid/hid-sensor-custom.c:598:64: warning: 'snprintf' output may be truncated before the last format character
@ 2023-11-16 4:07 kernel test robot
2023-11-16 6:00 ` Liu, Yujie
0 siblings, 1 reply; 2+ messages in thread
From: kernel test robot @ 2023-11-16 4:07 UTC (permalink / raw)
To: Sebastian Andrzej Siewior; +Cc: oe-kbuild-all, linux-kernel, Michal Marek
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: c42d9eeef8e5ba9292eda36fd8e3c11f35ee065c
commit: 8ae94224c9d72fc4d9aaac93b2d7833cf46d7141 kbuild: add -fno-PIE
date: 7 years ago
config: x86_64-randconfig-m001-20230717 (https://download.01.org/0day-ci/archive/20231116/202311161224.pnGqzMx0-lkp@intel.com/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231116/202311161224.pnGqzMx0-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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202311161224.pnGqzMx0-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/hid/hid-sensor-custom.c: In function 'store_value':
drivers/hid/hid-sensor-custom.c:409:21: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
409 | int ret;
| ^~~
drivers/hid/hid-sensor-custom.c: In function 'hid_sensor_custom_add_attributes':
>> drivers/hid/hid-sensor-custom.c:598:64: warning: 'snprintf' output may be truncated before the last format character [-Wformat-truncation=]
598 | HID_CUSTOM_NAME_LENGTH, "%s-%s",
| ^
drivers/hid/hid-sensor-custom.c:597:25: note: 'snprintf' output 2 or more bytes (assuming 65) into a destination of size 64
597 | snprintf((char *)&sensor_inst->fields[i].attr_name[j],
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
598 | HID_CUSTOM_NAME_LENGTH, "%s-%s",
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
599 | sensor_inst->fields[i].group_name,
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
600 | hid_custom_attrs[j].name);
| ~~~~~~~~~~~~~~~~~~~~~~~~~
arch/x86/include/asm/bitops.h: Assembler messages:
arch/x86/include/asm/bitops.h:211: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
--
In file included from fs/orangefs/protocol.h:336,
from fs/orangefs/acl.c:7:
>> fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
89 | static const int num_kmod_keyword_mask_map = (int)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from fs/orangefs/protocol.h:336,
from fs/orangefs/dcache.c:11:
>> fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
89 | static const int num_kmod_keyword_mask_map = (int)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
fs/orangefs/dcache.c: In function 'orangefs_revalidate_lookup':
fs/orangefs/dcache.c:33:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
33 | strncpy(new_op->upcall.req.lookup.d_name,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
34 | dentry->d_name.name,
| ~~~~~~~~~~~~~~~~~~~~
35 | ORANGEFS_NAME_MAX);
| ~~~~~~~~~~~~~~~~~~
--
In file included from fs/orangefs/protocol.h:336,
from fs/orangefs/inode.c:11:
>> fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
89 | static const int num_kmod_keyword_mask_map = (int)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
arch/x86/include/asm/bitops.h: Assembler messages:
arch/x86/include/asm/bitops.h:96: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
arch/x86/include/asm/bitops.h:139: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
--
In file included from fs/orangefs/protocol.h:336,
from fs/orangefs/super.c:7:
>> fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
89 | static const int num_kmod_keyword_mask_map = (int)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
fs/orangefs/super.c: In function 'orangefs_mount':
fs/orangefs/super.c:451:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
451 | strncpy(new_op->upcall.req.fs_mount.orangefs_config_server,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
452 | devname,
| ~~~~~~~~
453 | ORANGEFS_MAX_SERVER_ADDR_LEN);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/orangefs/super.c:491:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
491 | strncpy(ORANGEFS_SB(sb)->devname,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
492 | devname,
| ~~~~~~~~
493 | ORANGEFS_MAX_SERVER_ADDR_LEN);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
--
In file included from fs/orangefs/protocol.h:336,
from fs/orangefs/namei.c:11:
>> fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
89 | static const int num_kmod_keyword_mask_map = (int)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
fs/orangefs/namei.c: In function 'orangefs_rename':
fs/orangefs/namei.c:434:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
434 | strncpy(new_op->upcall.req.rename.d_old_name,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
435 | old_dentry->d_name.name,
| ~~~~~~~~~~~~~~~~~~~~~~~~
436 | ORANGEFS_NAME_MAX);
| ~~~~~~~~~~~~~~~~~~
fs/orangefs/namei.c:437:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
437 | strncpy(new_op->upcall.req.rename.d_new_name,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
438 | new_dentry->d_name.name,
| ~~~~~~~~~~~~~~~~~~~~~~~~
439 | ORANGEFS_NAME_MAX);
| ~~~~~~~~~~~~~~~~~~
fs/orangefs/namei.c: In function 'orangefs_unlink':
fs/orangefs/namei.c:240:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
240 | strncpy(new_op->upcall.req.remove.d_name, dentry->d_name.name,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
241 | ORANGEFS_NAME_MAX);
| ~~~~~~~~~~~~~~~~~~
fs/orangefs/namei.c: In function 'orangefs_lookup':
fs/orangefs/namei.c:137:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
137 | strncpy(new_op->upcall.req.lookup.d_name, dentry->d_name.name,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
138 | ORANGEFS_NAME_MAX);
| ~~~~~~~~~~~~~~~~~~
fs/orangefs/namei.c:137:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
fs/orangefs/namei.c: In function 'orangefs_mkdir':
fs/orangefs/namei.c:358:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
358 | strncpy(new_op->upcall.req.mkdir.d_name,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
359 | dentry->d_name.name, ORANGEFS_NAME_MAX);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/orangefs/namei.c: In function 'orangefs_create':
fs/orangefs/namei.c:40:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
40 | strncpy(new_op->upcall.req.create.d_name,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
41 | dentry->d_name.name, ORANGEFS_NAME_MAX);
| ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
fs/orangefs/namei.c: In function 'orangefs_symlink':
fs/orangefs/namei.c:291:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
291 | strncpy(new_op->upcall.req.sym.entry_name,
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
292 | dentry->d_name.name,
| ~~~~~~~~~~~~~~~~~~~~
293 | ORANGEFS_NAME_MAX);
| ~~~~~~~~~~~~~~~~~~
fs/orangefs/namei.c:294:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
294 | strncpy(new_op->upcall.req.sym.target, symname, ORANGEFS_NAME_MAX);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
--
fs/orangefs/orangefs-bufmap.c:225:5: warning: no previous prototype for 'orangefs_get_bufmap_init' [-Wmissing-prototypes]
225 | int orangefs_get_bufmap_init(void)
| ^~~~~~~~~~~~~~~~~~~~~~~~
In file included from fs/orangefs/protocol.h:336,
from fs/orangefs/orangefs-bufmap.c:6:
>> fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
89 | static const int num_kmod_keyword_mask_map = (int)
| ^~~~~~~~~~~~~~~~~~~~~~~~~
vim +/snprintf +598 drivers/hid/hid-sensor-custom.c
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 568
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 569 static int hid_sensor_custom_add_attributes(struct hid_sensor_custom
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 570 *sensor_inst)
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 571 {
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 572 struct hid_sensor_hub_device *hsdev = sensor_inst->hsdev;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 573 struct hid_device *hdev = hsdev->hdev;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 574 int ret = -1;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 575 int i, j;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 576
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 577 for (j = 0; j < HID_REPORT_TYPES; ++j) {
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 578 if (j == HID_OUTPUT_REPORT)
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 579 continue;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 580
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 581 ret = hid_sensor_custom_add_fields(sensor_inst,
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 582 &hdev->report_enum[j], j);
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 583 if (ret)
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 584 return ret;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 585
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 586 }
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 587
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 588 /* Create sysfs attributes */
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 589 for (i = 0; i < sensor_inst->sensor_field_count; ++i) {
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 590 j = 0;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 591 while (j < HID_CUSTOM_TOTAL_ATTRS &&
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 592 hid_custom_attrs[j].name) {
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 593 struct device_attribute *device_attr;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 594
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 595 device_attr = &sensor_inst->fields[i].sd_attrs[j];
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 596
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 597 snprintf((char *)&sensor_inst->fields[i].attr_name[j],
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 @598 HID_CUSTOM_NAME_LENGTH, "%s-%s",
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 599 sensor_inst->fields[i].group_name,
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 600 hid_custom_attrs[j].name);
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 601 sysfs_attr_init(&device_attr->attr);
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 602 device_attr->attr.name =
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 603 (char *)&sensor_inst->fields[i].attr_name[j];
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 604 device_attr->attr.mode = hid_custom_attrs[j].mode;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 605 device_attr->show = show_value;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 606 if (hid_custom_attrs[j].mode & S_IWUSR)
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 607 device_attr->store = store_value;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 608 sensor_inst->fields[i].attrs[j] = &device_attr->attr;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 609 ++j;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 610 }
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 611 sensor_inst->fields[i].attrs[j] = NULL;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 612 sensor_inst->fields[i].hid_custom_attribute_group.attrs =
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 613 sensor_inst->fields[i].attrs;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 614 sensor_inst->fields[i].hid_custom_attribute_group.name =
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 615 sensor_inst->fields[i].group_name;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 616 ret = sysfs_create_group(&sensor_inst->pdev->dev.kobj,
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 617 &sensor_inst->fields[i].
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 618 hid_custom_attribute_group);
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 619 if (ret)
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 620 break;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 621
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 622 /* For power or report field store indexes */
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 623 if (sensor_inst->fields[i].attribute.attrib_id ==
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 624 HID_USAGE_SENSOR_PROY_POWER_STATE)
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 625 sensor_inst->power_state = &sensor_inst->fields[i];
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 626 else if (sensor_inst->fields[i].attribute.attrib_id ==
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 627 HID_USAGE_SENSOR_PROP_REPORT_STATE)
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 628 sensor_inst->report_state = &sensor_inst->fields[i];
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 629 }
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 630
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 631 return ret;
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 632 }
4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 633
:::::: The code at line 598 was first introduced by commit
:::::: 4a7de0519df5e8fb89cef6ee062330ffe4b50a4d HID: sensor: Custom and Generic sensor support
:::::: TO: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
:::::: CC: Jiri Kosina <jkosina@suse.cz>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: drivers/hid/hid-sensor-custom.c:598:64: warning: 'snprintf' output may be truncated before the last format character
2023-11-16 4:07 drivers/hid/hid-sensor-custom.c:598:64: warning: 'snprintf' output may be truncated before the last format character kernel test robot
@ 2023-11-16 6:00 ` Liu, Yujie
0 siblings, 0 replies; 2+ messages in thread
From: Liu, Yujie @ 2023-11-16 6:00 UTC (permalink / raw)
To: bigeasy@linutronix.de
Cc: mmarek@suse.com, linux-kernel@vger.kernel.org,
oe-kbuild-all@lists.linux.dev, lkp
On Thu, 2023-11-16 at 12:07 +0800, kernel test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> head: c42d9eeef8e5ba9292eda36fd8e3c11f35ee065c
> commit: 8ae94224c9d72fc4d9aaac93b2d7833cf46d7141 kbuild: add -fno-PIE
> date: 7 years ago
> config: x86_64-randconfig-m001-20230717 (https://download.01.org/0day-ci/archive/20231116/202311161224.pnGqzMx0-lkp@intel.com/config)
> compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20231116/202311161224.pnGqzMx0-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@intel.com>
> > Closes: https://lore.kernel.org/oe-kbuild-all/202311161224.pnGqzMx0-lkp@intel.com/
Please kindly ignore this report. The commit here is not the cause of
the warnings. Sorry for the false positive.
> All warnings (new ones prefixed by >>):
>
> drivers/hid/hid-sensor-custom.c: In function 'store_value':
> drivers/hid/hid-sensor-custom.c:409:21: warning: variable 'ret' set but not used [-Wunused-but-set-variable]
> 409 | int ret;
> | ^~~
> drivers/hid/hid-sensor-custom.c: In function 'hid_sensor_custom_add_attributes':
> > > drivers/hid/hid-sensor-custom.c:598:64: warning: 'snprintf' output may be truncated before the last format character [-Wformat-truncation=]
> 598 | HID_CUSTOM_NAME_LENGTH, "%s-%s",
> | ^
> drivers/hid/hid-sensor-custom.c:597:25: note: 'snprintf' output 2 or more bytes (assuming 65) into a destination of size 64
> 597 | snprintf((char *)&sensor_inst->fields[i].attr_name[j],
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 598 | HID_CUSTOM_NAME_LENGTH, "%s-%s",
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 599 | sensor_inst->fields[i].group_name,
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 600 | hid_custom_attrs[j].name);
> | ~~~~~~~~~~~~~~~~~~~~~~~~~
> arch/x86/include/asm/bitops.h: Assembler messages:
> arch/x86/include/asm/bitops.h:211: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
> --
> In file included from fs/orangefs/protocol.h:336,
> from fs/orangefs/acl.c:7:
> > > fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
> 89 | static const int num_kmod_keyword_mask_map = (int)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~
> --
> In file included from fs/orangefs/protocol.h:336,
> from fs/orangefs/dcache.c:11:
> > > fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
> 89 | static const int num_kmod_keyword_mask_map = (int)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~
> fs/orangefs/dcache.c: In function 'orangefs_revalidate_lookup':
> fs/orangefs/dcache.c:33:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 33 | strncpy(new_op->upcall.req.lookup.d_name,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 34 | dentry->d_name.name,
> | ~~~~~~~~~~~~~~~~~~~~
> 35 | ORANGEFS_NAME_MAX);
> | ~~~~~~~~~~~~~~~~~~
> --
> In file included from fs/orangefs/protocol.h:336,
> from fs/orangefs/inode.c:11:
> > > fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
> 89 | static const int num_kmod_keyword_mask_map = (int)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~
> arch/x86/include/asm/bitops.h: Assembler messages:
> arch/x86/include/asm/bitops.h:96: Warning: no instruction mnemonic suffix given and no register operands; using default for `bts'
> arch/x86/include/asm/bitops.h:139: Warning: no instruction mnemonic suffix given and no register operands; using default for `btr'
> --
> In file included from fs/orangefs/protocol.h:336,
> from fs/orangefs/super.c:7:
> > > fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
> 89 | static const int num_kmod_keyword_mask_map = (int)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~
> fs/orangefs/super.c: In function 'orangefs_mount':
> fs/orangefs/super.c:451:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 451 | strncpy(new_op->upcall.req.fs_mount.orangefs_config_server,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 452 | devname,
> | ~~~~~~~~
> 453 | ORANGEFS_MAX_SERVER_ADDR_LEN);
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> fs/orangefs/super.c:491:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 491 | strncpy(ORANGEFS_SB(sb)->devname,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 492 | devname,
> | ~~~~~~~~
> 493 | ORANGEFS_MAX_SERVER_ADDR_LEN);
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> --
> In file included from fs/orangefs/protocol.h:336,
> from fs/orangefs/namei.c:11:
> > > fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
> 89 | static const int num_kmod_keyword_mask_map = (int)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~
> fs/orangefs/namei.c: In function 'orangefs_rename':
> fs/orangefs/namei.c:434:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 434 | strncpy(new_op->upcall.req.rename.d_old_name,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 435 | old_dentry->d_name.name,
> | ~~~~~~~~~~~~~~~~~~~~~~~~
> 436 | ORANGEFS_NAME_MAX);
> | ~~~~~~~~~~~~~~~~~~
> fs/orangefs/namei.c:437:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 437 | strncpy(new_op->upcall.req.rename.d_new_name,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 438 | new_dentry->d_name.name,
> | ~~~~~~~~~~~~~~~~~~~~~~~~
> 439 | ORANGEFS_NAME_MAX);
> | ~~~~~~~~~~~~~~~~~~
> fs/orangefs/namei.c: In function 'orangefs_unlink':
> fs/orangefs/namei.c:240:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 240 | strncpy(new_op->upcall.req.remove.d_name, dentry->d_name.name,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 241 | ORANGEFS_NAME_MAX);
> | ~~~~~~~~~~~~~~~~~~
> fs/orangefs/namei.c: In function 'orangefs_lookup':
> fs/orangefs/namei.c:137:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 137 | strncpy(new_op->upcall.req.lookup.d_name, dentry->d_name.name,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 138 | ORANGEFS_NAME_MAX);
> | ~~~~~~~~~~~~~~~~~~
> fs/orangefs/namei.c:137:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> fs/orangefs/namei.c: In function 'orangefs_mkdir':
> fs/orangefs/namei.c:358:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 358 | strncpy(new_op->upcall.req.mkdir.d_name,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 359 | dentry->d_name.name, ORANGEFS_NAME_MAX);
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> fs/orangefs/namei.c: In function 'orangefs_create':
> fs/orangefs/namei.c:40:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 40 | strncpy(new_op->upcall.req.create.d_name,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 41 | dentry->d_name.name, ORANGEFS_NAME_MAX);
> | ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> fs/orangefs/namei.c: In function 'orangefs_symlink':
> fs/orangefs/namei.c:291:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 291 | strncpy(new_op->upcall.req.sym.entry_name,
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 292 | dentry->d_name.name,
> | ~~~~~~~~~~~~~~~~~~~~
> 293 | ORANGEFS_NAME_MAX);
> | ~~~~~~~~~~~~~~~~~~
> fs/orangefs/namei.c:294:9: warning: 'strncpy' specified bound 256 equals destination size [-Wstringop-truncation]
> 294 | strncpy(new_op->upcall.req.sym.target, symname, ORANGEFS_NAME_MAX);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> --
> fs/orangefs/orangefs-bufmap.c:225:5: warning: no previous prototype for 'orangefs_get_bufmap_init' [-Wmissing-prototypes]
> 225 | int orangefs_get_bufmap_init(void)
> | ^~~~~~~~~~~~~~~~~~~~~~~~
> In file included from fs/orangefs/protocol.h:336,
> from fs/orangefs/orangefs-bufmap.c:6:
> > > fs/orangefs/orangefs-debug.h:89:18: warning: 'num_kmod_keyword_mask_map' defined but not used [-Wunused-const-variable=]
> 89 | static const int num_kmod_keyword_mask_map = (int)
> | ^~~~~~~~~~~~~~~~~~~~~~~~~
>
>
> vim +/snprintf +598 drivers/hid/hid-sensor-custom.c
>
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 568
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 569 static int hid_sensor_custom_add_attributes(struct hid_sensor_custom
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 570 *sensor_inst)
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 571 {
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 572 struct hid_sensor_hub_device *hsdev = sensor_inst->hsdev;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 573 struct hid_device *hdev = hsdev->hdev;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 574 int ret = -1;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 575 int i, j;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 576
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 577 for (j = 0; j < HID_REPORT_TYPES; ++j) {
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 578 if (j == HID_OUTPUT_REPORT)
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 579 continue;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 580
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 581 ret = hid_sensor_custom_add_fields(sensor_inst,
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 582 &hdev->report_enum[j], j);
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 583 if (ret)
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 584 return ret;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 585
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 586 }
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 587
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 588 /* Create sysfs attributes */
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 589 for (i = 0; i < sensor_inst->sensor_field_count; ++i) {
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 590 j = 0;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 591 while (j < HID_CUSTOM_TOTAL_ATTRS &&
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 592 hid_custom_attrs[j].name) {
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 593 struct device_attribute *device_attr;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 594
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 595 device_attr = &sensor_inst->fields[i].sd_attrs[j];
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 596
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 597 snprintf((char *)&sensor_inst->fields[i].attr_name[j],
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 @598 HID_CUSTOM_NAME_LENGTH, "%s-%s",
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 599 sensor_inst->fields[i].group_name,
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 600 hid_custom_attrs[j].name);
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 601 sysfs_attr_init(&device_attr->attr);
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 602 device_attr->attr.name =
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 603 (char *)&sensor_inst->fields[i].attr_name[j];
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 604 device_attr->attr.mode = hid_custom_attrs[j].mode;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 605 device_attr->show = show_value;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 606 if (hid_custom_attrs[j].mode & S_IWUSR)
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 607 device_attr->store = store_value;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 608 sensor_inst->fields[i].attrs[j] = &device_attr->attr;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 609 ++j;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 610 }
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 611 sensor_inst->fields[i].attrs[j] = NULL;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 612 sensor_inst->fields[i].hid_custom_attribute_group.attrs =
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 613 sensor_inst->fields[i].attrs;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 614 sensor_inst->fields[i].hid_custom_attribute_group.name =
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 615 sensor_inst->fields[i].group_name;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 616 ret = sysfs_create_group(&sensor_inst->pdev->dev.kobj,
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 617 &sensor_inst->fields[i].
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 618 hid_custom_attribute_group);
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 619 if (ret)
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 620 break;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 621
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 622 /* For power or report field store indexes */
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 623 if (sensor_inst->fields[i].attribute.attrib_id ==
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 624 HID_USAGE_SENSOR_PROY_POWER_STATE)
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 625 sensor_inst->power_state = &sensor_inst->fields[i];
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 626 else if (sensor_inst->fields[i].attribute.attrib_id ==
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 627 HID_USAGE_SENSOR_PROP_REPORT_STATE)
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 628 sensor_inst->report_state = &sensor_inst->fields[i];
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 629 }
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 630
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 631 return ret;
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 632 }
> 4a7de0519df5e8 Srinivas Pandruvada 2015-04-10 633
>
> :::::: The code at line 598 was first introduced by commit
> :::::: 4a7de0519df5e8fb89cef6ee062330ffe4b50a4d HID: sensor: Custom and Generic sensor support
>
> :::::: TO: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com>
> :::::: CC: Jiri Kosina <jkosina@suse.cz>
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2023-11-16 6:01 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-11-16 4:07 drivers/hid/hid-sensor-custom.c:598:64: warning: 'snprintf' output may be truncated before the last format character kernel test robot
2023-11-16 6:00 ` Liu, Yujie
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox