| 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: <202103120621.Qd0N4TkW-lkp@intel.com>
Date: Fri, 12 Mar 2021 06:31:28 +0800
From: kernel test robot <lkp@...el.com>
To: Raed Salem <raeds@...lanox.com>
Cc: kbuild-all@...ts.01.org, linux-kernel@...r.kernel.org,
Saeed Mahameed <saeedm@...dia.com>,
Huy Nguyen <huyn@...lanox.com>,
Maxim Mikityanskiy <maximmi@...lanox.com>,
Tariq Toukan <tariqt@...dia.com>
Subject: drivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec_rxtx.c:361
mlx5e_ipsec_handle_tx_skb() error: uninitialized symbol 'mdata'.
tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head: 28806e4d9b97865b450d72156e9ad229f2067f0b
commit: 5be019040cb7bab4caf152cacadffee91a78b506 net/mlx5e: IPsec: Add Connect-X IPsec Tx data path offload
date: 5 months ago
config: arc-randconfig-m031-20210311 (attached as .config)
compiler: arceb-elf-gcc (GCC) 9.3.0
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@...el.com>
smatch warnings:
drivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec_rxtx.c:361 mlx5e_ipsec_handle_tx_skb() error: uninitialized symbol 'mdata'.
vim +/mdata +361 drivers/net/ethernet/mellanox/mlx5/core/en_accel/ipsec_rxtx.c
5be019040cb7ba Raed Salem 2020-06-16 313
5be019040cb7ba Raed Salem 2020-06-16 314 bool mlx5e_ipsec_handle_tx_skb(struct net_device *netdev,
5be019040cb7ba Raed Salem 2020-06-16 315 struct sk_buff *skb,
5be019040cb7ba Raed Salem 2020-06-16 316 struct mlx5e_accel_tx_ipsec_state *ipsec_st)
5be019040cb7ba Raed Salem 2020-06-16 317 {
5be019040cb7ba Raed Salem 2020-06-16 318 struct mlx5e_priv *priv = netdev_priv(netdev);
5be019040cb7ba Raed Salem 2020-06-16 319 struct xfrm_offload *xo = xfrm_offload(skb);
5be019040cb7ba Raed Salem 2020-06-16 320 struct mlx5e_ipsec_sa_entry *sa_entry;
5be019040cb7ba Raed Salem 2020-06-16 321 struct mlx5e_ipsec_metadata *mdata;
5be019040cb7ba Raed Salem 2020-06-16 322 struct xfrm_state *x;
5be019040cb7ba Raed Salem 2020-06-16 323 struct sec_path *sp;
2ac9cfe78223bb Ilan Tayari 2017-04-18 324
6362a6a04031ca Florian Westphal 2018-12-18 325 sp = skb_sec_path(skb);
6362a6a04031ca Florian Westphal 2018-12-18 326 if (unlikely(sp->len != 1)) {
2ac9cfe78223bb Ilan Tayari 2017-04-18 327 atomic64_inc(&priv->ipsec->sw_stats.ipsec_tx_drop_bundle);
2ac9cfe78223bb Ilan Tayari 2017-04-18 328 goto drop;
2ac9cfe78223bb Ilan Tayari 2017-04-18 329 }
2ac9cfe78223bb Ilan Tayari 2017-04-18 330
2ac9cfe78223bb Ilan Tayari 2017-04-18 331 x = xfrm_input_state(skb);
2ac9cfe78223bb Ilan Tayari 2017-04-18 332 if (unlikely(!x)) {
2ac9cfe78223bb Ilan Tayari 2017-04-18 333 atomic64_inc(&priv->ipsec->sw_stats.ipsec_tx_drop_no_state);
2ac9cfe78223bb Ilan Tayari 2017-04-18 334 goto drop;
2ac9cfe78223bb Ilan Tayari 2017-04-18 335 }
2ac9cfe78223bb Ilan Tayari 2017-04-18 336
2ac9cfe78223bb Ilan Tayari 2017-04-18 337 if (unlikely(!x->xso.offload_handle ||
2ac9cfe78223bb Ilan Tayari 2017-04-18 338 (skb->protocol != htons(ETH_P_IP) &&
2ac9cfe78223bb Ilan Tayari 2017-04-18 339 skb->protocol != htons(ETH_P_IPV6)))) {
2ac9cfe78223bb Ilan Tayari 2017-04-18 340 atomic64_inc(&priv->ipsec->sw_stats.ipsec_tx_drop_not_ip);
2ac9cfe78223bb Ilan Tayari 2017-04-18 341 goto drop;
2ac9cfe78223bb Ilan Tayari 2017-04-18 342 }
2ac9cfe78223bb Ilan Tayari 2017-04-18 343
2ac9cfe78223bb Ilan Tayari 2017-04-18 344 if (!skb_is_gso(skb))
2ac9cfe78223bb Ilan Tayari 2017-04-18 345 if (unlikely(mlx5e_ipsec_remove_trailer(skb, x))) {
2ac9cfe78223bb Ilan Tayari 2017-04-18 346 atomic64_inc(&priv->ipsec->sw_stats.ipsec_tx_drop_trailer);
2ac9cfe78223bb Ilan Tayari 2017-04-18 347 goto drop;
2ac9cfe78223bb Ilan Tayari 2017-04-18 348 }
5be019040cb7ba Raed Salem 2020-06-16 349
5be019040cb7ba Raed Salem 2020-06-16 350 if (MLX5_CAP_GEN(priv->mdev, fpga)) {
2ac9cfe78223bb Ilan Tayari 2017-04-18 351 mdata = mlx5e_ipsec_add_metadata(skb);
92978ee801844b Tobias Klauser 2017-09-26 352 if (IS_ERR(mdata)) {
2ac9cfe78223bb Ilan Tayari 2017-04-18 353 atomic64_inc(&priv->ipsec->sw_stats.ipsec_tx_drop_metadata);
2ac9cfe78223bb Ilan Tayari 2017-04-18 354 goto drop;
2ac9cfe78223bb Ilan Tayari 2017-04-18 355 }
5be019040cb7ba Raed Salem 2020-06-16 356 }
5be019040cb7ba Raed Salem 2020-06-16 357
cb01008390bb06 Aviad Yehezkel 2018-01-18 358 sa_entry = (struct mlx5e_ipsec_sa_entry *)x->xso.offload_handle;
cb01008390bb06 Aviad Yehezkel 2018-01-18 359 sa_entry->set_iv_op(skb, x, xo);
5be019040cb7ba Raed Salem 2020-06-16 360 if (MLX5_CAP_GEN(priv->mdev, fpga))
2ac9cfe78223bb Ilan Tayari 2017-04-18 @361 mlx5e_ipsec_set_metadata(skb, mdata, xo);
2ac9cfe78223bb Ilan Tayari 2017-04-18 362
5be019040cb7ba Raed Salem 2020-06-16 363 mlx5e_ipsec_set_state(priv, skb, x, xo, ipsec_st);
5be019040cb7ba Raed Salem 2020-06-16 364
f02bac9ad6415e Maxim Mikityanskiy 2019-12-05 365 return true;
2ac9cfe78223bb Ilan Tayari 2017-04-18 366
2ac9cfe78223bb Ilan Tayari 2017-04-18 367 drop:
2ac9cfe78223bb Ilan Tayari 2017-04-18 368 kfree_skb(skb);
f02bac9ad6415e Maxim Mikityanskiy 2019-12-05 369 return false;
2ac9cfe78223bb Ilan Tayari 2017-04-18 370 }
2ac9cfe78223bb Ilan Tayari 2017-04-18 371
:::::: The code at line 361 was first introduced by commit
:::::: 2ac9cfe78223bb88be8cff3b59e0e13551b4e29c net/mlx5e: IPSec, Add Innova IPSec offload TX data path
:::::: TO: Ilan Tayari <ilant@...lanox.com>
:::::: CC: Saeed Mahameed <saeedm@...lanox.com>
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
Download attachment ".config.gz" of type "application/gzip" (28239 bytes)
Powered by blists - more mailing lists