[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <202512090202.P59kzmhm-lkp@intel.com>
Date: Tue, 9 Dec 2025 03:11:31 +0800
From: kernel test robot <lkp@...el.com>
To: Dharanitharan R <dharanitharan725@...il.com>,
syzbot+5dd615f890ddada54057@...kaller.appspotmail.com
Cc: oe-kbuild-all@...ts.linux.dev, netdev@...r.kernel.org,
linux-kernel@...r.kernel.org,
Dharanitharan R <dharanitharan725@...il.com>
Subject: Re: [PATCH] net: atm: lec: add pre_send validation to avoid
uninitialized
Hi Dharanitharan,
kernel test robot noticed the following build warnings:
[auto build test WARNING on net/main]
[also build test WARNING on net-next/main linus/master v6.18 next-20251208]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Dharanitharan-R/net-atm-lec-add-pre_send-validation-to-avoid-uninitialized/20251207-121647
base: net/main
patch link: https://lore.kernel.org/r/20251207041453.8302-1-dharanitharan725%40gmail.com
patch subject: [PATCH] net: atm: lec: add pre_send validation to avoid uninitialized
config: x86_64-randconfig-r072-20251208 (https://download.01.org/0day-ci/archive/20251209/202512090202.P59kzmhm-lkp@intel.com/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
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/202512090202.P59kzmhm-lkp@intel.com/
New smatch warnings:
net/atm/lec.c:503 lec_atm_pre_send() warn: inconsistent indenting
net/atm/lec.c:506 lec_atm_pre_send() warn: curly braces intended?
Old smatch warnings:
net/atm/lec.c:507 lec_atm_pre_send() warn: inconsistent indenting
vim +503 net/atm/lec.c
491
492 static int lec_atm_pre_send(struct atm_vcc *vcc, struct sk_buff *skb)
493 {
494 struct atmlec_msg *mesg;
495 u32 sizeoftlvs;
496 unsigned int msg_size = sizeof(struct atmlec_msg);
497
498 /* Must contain the base message */
499 if (skb->len < msg_size)
500 return -EINVAL;
501
502 /* Must have at least msg_size bytes in linear data */
> 503 if (!pskb_may_pull(skb, msg_size))
504 return -EINVAL;
505
> 506 mesg = (struct atmlec_msg *)skb->data;
507 sizeoftlvs = mesg->sizeoftlvs;
508
509 /* Validate TLVs if present */
510 if (sizeoftlvs && !pskb_may_pull(skb, msg_size + sizeoftlvs))
511 return -EINVAL;
512
513 return 0;
514 }
515
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists