Reviewed-by: Liron Himi <lir...@marvell.com>
-----Original Message-----
From: Stephen Hemminger <step...@networkplumber.org>
Sent: Monday, 24 January 2022 02:04
To: dev@dpdk.org
Cc: Stephen Hemminger <step...@networkplumber.org>; Michael Shamis
<michae...@marvell.com>; Liron Himi <lir...@marvell.com>
Subject: [EXT] [PATCH 32/82] crypto/mvsam: remove unnecessary NULL checks
External Email
----------------------------------------------------------------------
Remove redundant NULL pointer checks before free functions found by
nullfree.cocci
Signed-off-by: Stephen Hemminger <step...@networkplumber.org>
---
drivers/crypto/mvsam/rte_mrvl_pmd_ops.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/crypto/mvsam/rte_mrvl_pmd_ops.c
b/drivers/crypto/mvsam/rte_mrvl_pmd_ops.c
index 3064b1f136fe..f828dc9db5f6 100644
--- a/drivers/crypto/mvsam/rte_mrvl_pmd_ops.c
+++ b/drivers/crypto/mvsam/rte_mrvl_pmd_ops.c
@@ -774,10 +774,8 @@ mrvl_crypto_pmd_sym_session_configure(__rte_unused struct
rte_cryptodev *dev,
}
/* free the keys memory allocated for session creation */
- if (mrvl_sess->sam_sess_params.cipher_key != NULL)
- free(mrvl_sess->sam_sess_params.cipher_key);
- if (mrvl_sess->sam_sess_params.auth_key != NULL)
- free(mrvl_sess->sam_sess_params.auth_key);
+ free(mrvl_sess->sam_sess_params.cipher_key);
+ free(mrvl_sess->sam_sess_params.auth_key);
return 0;
}
--
2.30.2
