| 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: <202309200044.5VZF1Ufb-lkp@intel.com>
Date: Wed, 20 Sep 2023 00:23:20 +0800
From: kernel test robot <lkp@...el.com>
To: WANG Xuerui <git@...0n.name>
Cc: oe-kbuild-all@...ts.linux.dev, linux-kernel@...r.kernel.org,
Huacai Chen <chenhuacai@...nel.org>
Subject: net/8021q/vlan.c:235:46: warning: '%.4i' directive output may be
truncated writing 4 bytes into a region of size between 0 and 15
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 2cf0f715623872823a72e451243bbf555d10d032
commit: 3f301dc292eb122eff61b8b2906e519154b0327f LoongArch: Replace -ffreestanding with finer-grained -fno-builtin's
date: 4 weeks ago
config: loongarch-defconfig (https://download.01.org/0day-ci/archive/20230920/202309200044.5VZF1Ufb-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/20230920/202309200044.5VZF1Ufb-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/202309200044.5VZF1Ufb-lkp@intel.com/
All warnings (new ones prefixed by >>):
net/8021q/vlan.c: In function 'register_vlan_device':
net/8021q/vlan.c:247:46: warning: '%i' directive output may be truncated writing between 1 and 4 bytes into a region of size between 0 and 15 [-Wformat-truncation=]
247 | snprintf(name, IFNAMSIZ, "%s.%i", real_dev->name, vlan_id);
| ^~
net/8021q/vlan.c:247:42: note: directive argument in the range [0, 4094]
247 | snprintf(name, IFNAMSIZ, "%s.%i", real_dev->name, vlan_id);
| ^~~~~~~
net/8021q/vlan.c:247:17: note: 'snprintf' output between 3 and 21 bytes into a destination of size 16
247 | snprintf(name, IFNAMSIZ, "%s.%i", real_dev->name, vlan_id);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
>> net/8021q/vlan.c:235:46: warning: '%.4i' directive output may be truncated writing 4 bytes into a region of size between 0 and 15 [-Wformat-truncation=]
235 | snprintf(name, IFNAMSIZ, "%s.%.4i", real_dev->name, vlan_id);
| ^~~~
net/8021q/vlan.c:235:42: note: directive argument in the range [0, 4094]
235 | snprintf(name, IFNAMSIZ, "%s.%.4i", real_dev->name, vlan_id);
| ^~~~~~~~~
net/8021q/vlan.c:235:17: note: 'snprintf' output between 6 and 21 bytes into a destination of size 16
235 | snprintf(name, IFNAMSIZ, "%s.%.4i", real_dev->name, vlan_id);
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
vim +235 net/8021q/vlan.c
e89fe42cd03c8f Patrick McHardy 2007-06-13 210
c1d3ee9925ca71 Patrick McHardy 2007-06-13 211 /* Attach a VLAN device to a mac address (ie Ethernet Card).
2ae0bf69b716d0 Patrick McHardy 2007-06-13 212 * Returns 0 if the device was created or a negative error code otherwise.
c1d3ee9925ca71 Patrick McHardy 2007-06-13 213 */
9bb8582efb5555 Patrick McHardy 2008-07-08 214 static int register_vlan_device(struct net_device *real_dev, u16 vlan_id)
c1d3ee9925ca71 Patrick McHardy 2007-06-13 215 {
c1d3ee9925ca71 Patrick McHardy 2007-06-13 216 struct net_device *new_dev;
0c0667a8548ef2 Wang Sheng-Hui 2013-08-03 217 struct vlan_dev_priv *vlan;
7a17a2f79f54a9 Pavel Emelyanov 2008-04-16 218 struct net *net = dev_net(real_dev);
7a17a2f79f54a9 Pavel Emelyanov 2008-04-16 219 struct vlan_net *vn = net_generic(net, vlan_net_id);
c1d3ee9925ca71 Patrick McHardy 2007-06-13 220 char name[IFNAMSIZ];
2ae0bf69b716d0 Patrick McHardy 2007-06-13 221 int err;
c1d3ee9925ca71 Patrick McHardy 2007-06-13 222
9bb8582efb5555 Patrick McHardy 2008-07-08 223 if (vlan_id >= VLAN_VID_MASK)
2ae0bf69b716d0 Patrick McHardy 2007-06-13 224 return -ERANGE;
c1d3ee9925ca71 Patrick McHardy 2007-06-13 225
33fa382324ebd1 David Ahern 2018-05-17 226 err = vlan_check_real_dev(real_dev, htons(ETH_P_8021Q), vlan_id,
33fa382324ebd1 David Ahern 2018-05-17 227 NULL);
2ae0bf69b716d0 Patrick McHardy 2007-06-13 228 if (err < 0)
2ae0bf69b716d0 Patrick McHardy 2007-06-13 229 return err;
c1d3ee9925ca71 Patrick McHardy 2007-06-13 230
^1da177e4c3f41 Linus Torvalds 2005-04-16 231 /* Gotta set up the fields for the device. */
7a17a2f79f54a9 Pavel Emelyanov 2008-04-16 232 switch (vn->name_type) {
^1da177e4c3f41 Linus Torvalds 2005-04-16 233 case VLAN_NAME_TYPE_RAW_PLUS_VID:
^1da177e4c3f41 Linus Torvalds 2005-04-16 234 /* name will look like: eth1.0005 */
9bb8582efb5555 Patrick McHardy 2008-07-08 @235 snprintf(name, IFNAMSIZ, "%s.%.4i", real_dev->name, vlan_id);
^1da177e4c3f41 Linus Torvalds 2005-04-16 236 break;
^1da177e4c3f41 Linus Torvalds 2005-04-16 237 case VLAN_NAME_TYPE_PLUS_VID_NO_PAD:
^1da177e4c3f41 Linus Torvalds 2005-04-16 238 /* Put our vlan.VID in the name.
^1da177e4c3f41 Linus Torvalds 2005-04-16 239 * Name will look like: vlan5
^1da177e4c3f41 Linus Torvalds 2005-04-16 240 */
9bb8582efb5555 Patrick McHardy 2008-07-08 241 snprintf(name, IFNAMSIZ, "vlan%i", vlan_id);
^1da177e4c3f41 Linus Torvalds 2005-04-16 242 break;
^1da177e4c3f41 Linus Torvalds 2005-04-16 243 case VLAN_NAME_TYPE_RAW_PLUS_VID_NO_PAD:
^1da177e4c3f41 Linus Torvalds 2005-04-16 244 /* Put our vlan.VID in the name.
^1da177e4c3f41 Linus Torvalds 2005-04-16 245 * Name will look like: eth0.5
^1da177e4c3f41 Linus Torvalds 2005-04-16 246 */
9bb8582efb5555 Patrick McHardy 2008-07-08 247 snprintf(name, IFNAMSIZ, "%s.%i", real_dev->name, vlan_id);
^1da177e4c3f41 Linus Torvalds 2005-04-16 248 break;
^1da177e4c3f41 Linus Torvalds 2005-04-16 249 case VLAN_NAME_TYPE_PLUS_VID:
^1da177e4c3f41 Linus Torvalds 2005-04-16 250 /* Put our vlan.VID in the name.
^1da177e4c3f41 Linus Torvalds 2005-04-16 251 * Name will look like: vlan0005
^1da177e4c3f41 Linus Torvalds 2005-04-16 252 */
^1da177e4c3f41 Linus Torvalds 2005-04-16 253 default:
9bb8582efb5555 Patrick McHardy 2008-07-08 254 snprintf(name, IFNAMSIZ, "vlan%.4i", vlan_id);
3ff50b7997fe06 Stephen Hemminger 2007-04-20 255 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 256
c835a677331495 Tom Gundersen 2014-07-14 257 new_dev = alloc_netdev(sizeof(struct vlan_dev_priv), name,
c835a677331495 Tom Gundersen 2014-07-14 258 NET_NAME_UNKNOWN, vlan_setup);
5dd8d1e9eb8b51 Arjan van de Ven 2006-07-03 259
^1da177e4c3f41 Linus Torvalds 2005-04-16 260 if (new_dev == NULL)
2ae0bf69b716d0 Patrick McHardy 2007-06-13 261 return -ENOBUFS;
^1da177e4c3f41 Linus Torvalds 2005-04-16 262
65d292a2ef2df6 Pavel Emelyanov 2008-04-16 263 dev_net_set(new_dev, net);
^1da177e4c3f41 Linus Torvalds 2005-04-16 264 /* need 4 bytes for extra VLAN header info,
^1da177e4c3f41 Linus Torvalds 2005-04-16 265 * hope the underlying device can handle it.
^1da177e4c3f41 Linus Torvalds 2005-04-16 266 */
^1da177e4c3f41 Linus Torvalds 2005-04-16 267 new_dev->mtu = real_dev->mtu;
^1da177e4c3f41 Linus Torvalds 2005-04-16 268
0c0667a8548ef2 Wang Sheng-Hui 2013-08-03 269 vlan = vlan_dev_priv(new_dev);
0c0667a8548ef2 Wang Sheng-Hui 2013-08-03 270 vlan->vlan_proto = htons(ETH_P_8021Q);
0c0667a8548ef2 Wang Sheng-Hui 2013-08-03 271 vlan->vlan_id = vlan_id;
0c0667a8548ef2 Wang Sheng-Hui 2013-08-03 272 vlan->real_dev = real_dev;
0c0667a8548ef2 Wang Sheng-Hui 2013-08-03 273 vlan->dent = NULL;
0c0667a8548ef2 Wang Sheng-Hui 2013-08-03 274 vlan->flags = VLAN_FLAG_REORDER_HDR;
^1da177e4c3f41 Linus Torvalds 2005-04-16 275
07b5b17e157b70 Patrick McHardy 2007-06-13 276 new_dev->rtnl_link_ops = &vlan_link_ops;
42ab19ee902929 David Ahern 2017-10-04 277 err = register_vlan_dev(new_dev, NULL);
2ae0bf69b716d0 Patrick McHardy 2007-06-13 278 if (err < 0)
42429aaee5eb44 Patrick McHardy 2007-06-13 279 goto out_free_newdev;
^1da177e4c3f41 Linus Torvalds 2005-04-16 280
2ae0bf69b716d0 Patrick McHardy 2007-06-13 281 return 0;
^1da177e4c3f41 Linus Torvalds 2005-04-16 282
^1da177e4c3f41 Linus Torvalds 2005-04-16 283 out_free_newdev:
^1da177e4c3f41 Linus Torvalds 2005-04-16 284 free_netdev(new_dev);
2ae0bf69b716d0 Patrick McHardy 2007-06-13 285 return err;
^1da177e4c3f41 Linus Torvalds 2005-04-16 286 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 287
:::::: The code at line 235 was first introduced by commit
:::::: 9bb8582efb555521c7eec595ebd34e835ddc34b8 vlan: TCI related type and naming cleanups
:::::: TO: Patrick McHardy <kaber@...sh.net>
:::::: CC: David S. Miller <davem@...emloft.net>
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists