| 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
| ||
|
Message-ID: <202309260305.LRL7ALTy-lkp@intel.com>
Date: Tue, 26 Sep 2023 03:27:13 +0800
From: kernel test robot <lkp@...el.com>
To: Qing Zhang <zhangqing@...ngson.cn>
Cc: oe-kbuild-all@...ts.linux.dev, linux-kernel@...r.kernel.org,
Huacai Chen <chenhuacai@...nel.org>
Subject: drivers/net/wireless/atmel/atmel.c:2654:17: warning: 'strncpy'
output may be truncated copying 31 bytes from a string of length 31
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 6465e260f48790807eef06b583b38ca9789b6072
commit: 5aa4ac64e6add3e40d5049e31275b2822daf885d LoongArch: Add KASAN (Kernel Address Sanitizer) support
date: 3 weeks ago
config: loongarch-randconfig-002-20230926 (https://download.01.org/0day-ci/archive/20230926/202309260305.LRL7ALTy-lkp@intel.com/config)
compiler: loongarch64-linux-gcc (GCC) 13.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20230926/202309260305.LRL7ALTy-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/202309260305.LRL7ALTy-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/net/wireless/atmel/atmel.c: In function 'atmel_ioctl':
>> drivers/net/wireless/atmel/atmel.c:2654:17: warning: 'strncpy' output may be truncated copying 31 bytes from a string of length 31 [-Wstringop-truncation]
2654 | strncpy(priv->firmware_id, com.id, 31);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
vim +/strncpy +2654 drivers/net/wireless/atmel/atmel.c
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2618
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2619 static int atmel_ioctl(struct net_device *dev, struct ifreq *rq, int cmd)
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2620 {
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2621 int i, rc = 0;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2622 struct atmel_private *priv = netdev_priv(dev);
f0db82a5775d1c drivers/net/wireless/atmel.c Himangi Saraogi 2014-08-09 2623 struct atmel_priv_ioctl com;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2624 struct iwreq *wrq = (struct iwreq *) rq;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2625 unsigned char *new_firmware;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2626 char domain[REGDOMAINSZ + 1];
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2627
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2628 switch (cmd) {
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2629 case ATMELIDIFC:
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2630 wrq->u.param.value = ATMELMAGIC;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2631 break;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2632
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2633 case ATMELFWL:
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2634 if (copy_from_user(&com, rq->ifr_data, sizeof(com))) {
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2635 rc = -EFAULT;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2636 break;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2637 }
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2638
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2639 if (!capable(CAP_NET_ADMIN)) {
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2640 rc = -EPERM;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2641 break;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2642 }
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2643
8668f9a57c8c55 drivers/net/wireless/atmel/atmel.c YueHaibing 2018-06-04 2644 new_firmware = memdup_user(com.data, com.len);
8668f9a57c8c55 drivers/net/wireless/atmel/atmel.c YueHaibing 2018-06-04 2645 if (IS_ERR(new_firmware)) {
8668f9a57c8c55 drivers/net/wireless/atmel/atmel.c YueHaibing 2018-06-04 2646 rc = PTR_ERR(new_firmware);
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2647 break;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2648 }
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2649
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2650 kfree(priv->firmware);
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2651
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2652 priv->firmware = new_firmware;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2653 priv->firmware_length = com.len;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 @2654 strncpy(priv->firmware_id, com.id, 31);
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2655 priv->firmware_id[31] = '\0';
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2656 break;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2657
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2658 case ATMELRD:
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2659 if (copy_from_user(domain, rq->ifr_data, REGDOMAINSZ)) {
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2660 rc = -EFAULT;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2661 break;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2662 }
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2663
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2664 if (!capable(CAP_NET_ADMIN)) {
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2665 rc = -EPERM;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2666 break;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2667 }
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2668
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2669 domain[REGDOMAINSZ] = 0;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2670 rc = -EINVAL;
b4341135794fda drivers/net/wireless/atmel.c Dmitry Torokhov 2006-10-08 2671 for (i = 0; i < ARRAY_SIZE(channel_table); i++) {
908414af255ea9 drivers/net/wireless/atmel.c Rasmus Villemoes 2015-01-21 2672 if (!strcasecmp(channel_table[i].name, domain)) {
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2673 priv->config_reg_domain = channel_table[i].reg_domain;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2674 rc = 0;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2675 }
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2676 }
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2677
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2678 if (rc == 0 && priv->station_state != STATION_STATE_DOWN)
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2679 rc = atmel_open(dev);
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2680 break;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2681
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2682 default:
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2683 rc = -EOPNOTSUPP;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2684 }
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2685
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2686 return rc;
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2687 }
^1da177e4c3f41 drivers/net/wireless/atmel.c Linus Torvalds 2005-04-16 2688
:::::: The code at line 2654 was first introduced by commit
:::::: 1da177e4c3f41524e886b7f1b8a0c1fc7321cac2 Linux-2.6.12-rc2
:::::: TO: Linus Torvalds <torvalds@...970.osdl.org>
:::::: CC: Linus Torvalds <torvalds@...970.osdl.org>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists