* net/sched/act_api.c:875:15: warning: Uninitialized variable: id_ptr->id [uninitvar]
@ 2022-04-16 19:53 kernel test robot
0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2022-04-16 19:53 UTC (permalink / raw)
To: kbuild
[-- Attachment #1: Type: text/plain, Size: 4798 bytes --]
CC: kbuild-all(a)lists.01.org
BCC: lkp(a)intel.com
CC: linux-kernel(a)vger.kernel.org
TO: Baowen Zheng <baowen.zheng@corigine.com>
CC: Louis Peens <louis.peens@corigine.com>
CC: Simon Horman <simon.horman@corigine.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 59250f8a7f3a60a2661b84cbafc1e0eb5d05ec9b
commit: 13926d19a11e303f12571df61b7bb64f17cb4561 flow_offload: add reoffload process to update hw_count
date: 4 months ago
:::::: branch date: 21 hours ago
:::::: commit date: 4 months ago
compiler: csky-linux-gcc (GCC) 11.2.0
reproduce (cppcheck warning):
# apt-get install cppcheck
git checkout 13926d19a11e303f12571df61b7bb64f17cb4561
cppcheck --quiet --enable=style,performance,portability --template=gcc FILE
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
net/sched/act_api.c:1166:60: warning: Parameter 'actions' can be declared with const [constParameter]
int tcf_action_dump(struct sk_buff *skb, struct tc_action *actions[],
^
net/sched/act_api.c:2005:54: warning: Parameter 'nla' can be declared with const [constParameter]
static struct nlattr *find_dump_kind(struct nlattr **nla)
^
>> net/sched/act_api.c:875:15: warning: Uninitialized variable: id_ptr->id [uninitvar]
if (id_ptr->id == id) {
^
net/sched/act_api.c:901:15: warning: Uninitialized variable: id_ptr->id [uninitvar]
if (id_ptr->id == id) {
^
--
>> net/sched/cls_flower.c:1954:33: warning: Parameter 'fold' can be declared with const [constParameter]
struct cls_fl_filter *fold,
^
>> net/sched/cls_flower.c:268:30: warning: Uninitialized variables: filter.mask, filter.ht_node, filter.mkey, filter.exts, filter.res, filter.key, filter.list, filter.hw_list, filter.handle, filter.flags, filter.in_hw_count, filter.rwork, filter.hw_dev, filter.refcnt, filter.deleted [uninitvar]
if (!fl_range_port_dst_cmp(filter, key, mkey))
^
>> net/sched/cls_flower.c:319:35: warning: Uninitialized variables: mask.key, mask.range, mask.flags, mask.ht_node, mask.ht, mask.filter_ht_params, mask.dissector, mask.filters, mask.rwork, mask.list, mask.refcnt [uninitvar]
fl_clear_masked_range(&skb_key, mask);
^
>> net/sched/cls_flower.c:587:20: warning: Uninitialized variables: f.mask, f.ht_node, f.mkey, f.exts, f.res, f.key, f.list, f.hw_list, f.handle, f.flags, f.in_hw_count, f.rwork, f.hw_dev, f.refcnt, f.deleted [uninitvar]
__fl_delete(tp, f, &last, rtnl_held, extack);
^
vim +875 net/sched/act_api.c
13926d19a11e30 Baowen Zheng 2021-12-17 867
13926d19a11e30 Baowen Zheng 2021-12-17 868 static int tcf_pernet_add_id_list(unsigned int id)
13926d19a11e30 Baowen Zheng 2021-12-17 869 {
13926d19a11e30 Baowen Zheng 2021-12-17 870 struct tc_act_pernet_id *id_ptr;
13926d19a11e30 Baowen Zheng 2021-12-17 871 int ret = 0;
13926d19a11e30 Baowen Zheng 2021-12-17 872
13926d19a11e30 Baowen Zheng 2021-12-17 873 mutex_lock(&act_id_mutex);
13926d19a11e30 Baowen Zheng 2021-12-17 874 list_for_each_entry(id_ptr, &act_pernet_id_list, list) {
13926d19a11e30 Baowen Zheng 2021-12-17 @875 if (id_ptr->id == id) {
13926d19a11e30 Baowen Zheng 2021-12-17 876 ret = -EEXIST;
13926d19a11e30 Baowen Zheng 2021-12-17 877 goto err_out;
13926d19a11e30 Baowen Zheng 2021-12-17 878 }
13926d19a11e30 Baowen Zheng 2021-12-17 879 }
13926d19a11e30 Baowen Zheng 2021-12-17 880
13926d19a11e30 Baowen Zheng 2021-12-17 881 id_ptr = kzalloc(sizeof(*id_ptr), GFP_KERNEL);
13926d19a11e30 Baowen Zheng 2021-12-17 882 if (!id_ptr) {
13926d19a11e30 Baowen Zheng 2021-12-17 883 ret = -ENOMEM;
13926d19a11e30 Baowen Zheng 2021-12-17 884 goto err_out;
13926d19a11e30 Baowen Zheng 2021-12-17 885 }
13926d19a11e30 Baowen Zheng 2021-12-17 886 id_ptr->id = id;
13926d19a11e30 Baowen Zheng 2021-12-17 887
13926d19a11e30 Baowen Zheng 2021-12-17 888 list_add_tail(&id_ptr->list, &act_pernet_id_list);
13926d19a11e30 Baowen Zheng 2021-12-17 889
13926d19a11e30 Baowen Zheng 2021-12-17 890 err_out:
13926d19a11e30 Baowen Zheng 2021-12-17 891 mutex_unlock(&act_id_mutex);
13926d19a11e30 Baowen Zheng 2021-12-17 892 return ret;
13926d19a11e30 Baowen Zheng 2021-12-17 893 }
13926d19a11e30 Baowen Zheng 2021-12-17 894
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] 2+ messages in thread
* net/sched/act_api.c:875:15: warning: Uninitialized variable: id_ptr->id [uninitvar]
@ 2022-04-21 6:40 kernel test robot
0 siblings, 0 replies; 2+ messages in thread
From: kernel test robot @ 2022-04-21 6:40 UTC (permalink / raw)
To: kbuild
[-- Attachment #1: Type: text/plain, Size: 4234 bytes --]
CC: kbuild-all(a)lists.01.org
BCC: lkp(a)intel.com
CC: linux-kernel(a)vger.kernel.org
TO: Baowen Zheng <baowen.zheng@corigine.com>
CC: Louis Peens <louis.peens@corigine.com>
CC: Simon Horman <simon.horman@corigine.com>
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 559089e0a93d44280ec3ab478830af319c56dbe3
commit: 13926d19a11e303f12571df61b7bb64f17cb4561 flow_offload: add reoffload process to update hw_count
date: 4 months ago
:::::: branch date: 20 hours ago
:::::: commit date: 4 months ago
compiler: sparc64-linux-gcc (GCC) 11.2.0
reproduce (cppcheck warning):
# apt-get install cppcheck
git checkout 13926d19a11e303f12571df61b7bb64f17cb4561
cppcheck --quiet --enable=style,performance,portability --template=gcc FILE
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
cppcheck possible warnings: (new ones prefixed by >>, may not real problems)
>> net/x25/af_x25.c:308:75: warning: Parameter 'nb' can be declared with const [constParameter]
static struct sock *__x25_find_socket(unsigned int lci, struct x25_neigh *nb)
^
>> net/x25/af_x25.c:1628:5: warning: Redundant initialization for 'rc'. The initialized value is overwritten before it is read. [redundantInitialization]
rc = -EFAULT;
^
net/x25/af_x25.c:1626:9: note: rc is initialized
int rc = -EINVAL;
^
net/x25/af_x25.c:1628:5: note: rc is overwritten
rc = -EFAULT;
^
--
net/sched/act_api.c:1166:60: warning: Parameter 'actions' can be declared with const [constParameter]
int tcf_action_dump(struct sk_buff *skb, struct tc_action *actions[],
^
net/sched/act_api.c:2005:54: warning: Parameter 'nla' can be declared with const [constParameter]
static struct nlattr *find_dump_kind(struct nlattr **nla)
^
>> net/sched/act_api.c:875:15: warning: Uninitialized variable: id_ptr->id [uninitvar]
if (id_ptr->id == id) {
^
net/sched/act_api.c:901:15: warning: Uninitialized variable: id_ptr->id [uninitvar]
if (id_ptr->id == id) {
^
vim +875 net/sched/act_api.c
13926d19a11e30 Baowen Zheng 2021-12-17 867
13926d19a11e30 Baowen Zheng 2021-12-17 868 static int tcf_pernet_add_id_list(unsigned int id)
13926d19a11e30 Baowen Zheng 2021-12-17 869 {
13926d19a11e30 Baowen Zheng 2021-12-17 870 struct tc_act_pernet_id *id_ptr;
13926d19a11e30 Baowen Zheng 2021-12-17 871 int ret = 0;
13926d19a11e30 Baowen Zheng 2021-12-17 872
13926d19a11e30 Baowen Zheng 2021-12-17 873 mutex_lock(&act_id_mutex);
13926d19a11e30 Baowen Zheng 2021-12-17 874 list_for_each_entry(id_ptr, &act_pernet_id_list, list) {
13926d19a11e30 Baowen Zheng 2021-12-17 @875 if (id_ptr->id == id) {
13926d19a11e30 Baowen Zheng 2021-12-17 876 ret = -EEXIST;
13926d19a11e30 Baowen Zheng 2021-12-17 877 goto err_out;
13926d19a11e30 Baowen Zheng 2021-12-17 878 }
13926d19a11e30 Baowen Zheng 2021-12-17 879 }
13926d19a11e30 Baowen Zheng 2021-12-17 880
13926d19a11e30 Baowen Zheng 2021-12-17 881 id_ptr = kzalloc(sizeof(*id_ptr), GFP_KERNEL);
13926d19a11e30 Baowen Zheng 2021-12-17 882 if (!id_ptr) {
13926d19a11e30 Baowen Zheng 2021-12-17 883 ret = -ENOMEM;
13926d19a11e30 Baowen Zheng 2021-12-17 884 goto err_out;
13926d19a11e30 Baowen Zheng 2021-12-17 885 }
13926d19a11e30 Baowen Zheng 2021-12-17 886 id_ptr->id = id;
13926d19a11e30 Baowen Zheng 2021-12-17 887
13926d19a11e30 Baowen Zheng 2021-12-17 888 list_add_tail(&id_ptr->list, &act_pernet_id_list);
13926d19a11e30 Baowen Zheng 2021-12-17 889
13926d19a11e30 Baowen Zheng 2021-12-17 890 err_out:
13926d19a11e30 Baowen Zheng 2021-12-17 891 mutex_unlock(&act_id_mutex);
13926d19a11e30 Baowen Zheng 2021-12-17 892 return ret;
13926d19a11e30 Baowen Zheng 2021-12-17 893 }
13926d19a11e30 Baowen Zheng 2021-12-17 894
--
0-DAY CI Kernel Test Service
https://01.org/lkp
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2022-04-21 6:40 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-04-16 19:53 net/sched/act_api.c:875:15: warning: Uninitialized variable: id_ptr->id [uninitvar] kernel test robot
2022-04-21 6:40 kernel test robot
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.