CC: [email protected] BCC: [email protected] CC: [email protected] TO: Baowen Zheng <[email protected]> CC: Louis Peens <[email protected]> CC: Simon Horman <[email protected]>
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 <[email protected]> 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 _______________________________________________ kbuild mailing list -- [email protected] To unsubscribe send an email to [email protected]
