DPDK patches and discussions
 help / color / mirror / Atom feed
* [dpdk-dev] [PATCH] crypto/mvsam: fix memset of mrvl sess private data
@ 2021-07-01  6:59 danat
  2021-07-07 15:41 ` Akhil Goyal
  0 siblings, 1 reply; 2+ messages in thread
From: danat @ 2021-07-01  6:59 UTC (permalink / raw)
  To: gakhil; +Cc: dev, danat, lironh, michaelsh, stable

From: Dana Vardi <danat@marvell.com>

This patch fix wrong clear memory session pointer.

When call mrvl_crypto_pmd_sym_session_clear function,
it might cause the following error:
CRYPTODEV: set_sym_session_private_data() line 497:
Set private data for driver 1 not allowed

This fix set zeroes to mrvl_crypto_session pointer instead to
rte_cryptodev_sym_session pointer, and will verify that
rte_cryptodev_sym_session pointer will not lose data such as
nb_drivers.

Bugzilla ID: 646
Fixes: 8a61c83af2fa ("crypto/mrvl: add mrvl crypto driver")
Cc: stable@dpdk.org

Signed-off-by: Dana Vardi <danat@marvell.com>
Reviewed-by: Michael Shamis <michaelsh@marvell.com>
Reviewed-by: Liron Himi <lironh@marvell.com>
---
 drivers/crypto/mvsam/rte_mrvl_pmd_ops.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/crypto/mvsam/rte_mrvl_pmd_ops.c b/drivers/crypto/mvsam/rte_mrvl_pmd_ops.c
index 3cb7e3b9c..1a0a9fc14 100644
--- a/drivers/crypto/mvsam/rte_mrvl_pmd_ops.c
+++ b/drivers/crypto/mvsam/rte_mrvl_pmd_ops.c
@@ -794,7 +794,7 @@ mrvl_crypto_pmd_sym_session_clear(struct rte_cryptodev *dev,
 			MRVL_LOG(ERR, "Error while destroying session!");
 		}
 
-		memset(sess, 0, sizeof(struct mrvl_crypto_session));
+		memset(mrvl_sess, 0, sizeof(struct mrvl_crypto_session));
 		struct rte_mempool *sess_mp = rte_mempool_from_obj(sess_priv);
 		set_sym_session_private_data(sess, index, NULL);
 		rte_mempool_put(sess_mp, sess_priv);
-- 
2.29.0


^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [dpdk-dev] [PATCH] crypto/mvsam: fix memset of mrvl sess private data
  2021-07-01  6:59 [dpdk-dev] [PATCH] crypto/mvsam: fix memset of mrvl sess private data danat
@ 2021-07-07 15:41 ` Akhil Goyal
  0 siblings, 0 replies; 2+ messages in thread
From: Akhil Goyal @ 2021-07-07 15:41 UTC (permalink / raw)
  To: Dana Vardi; +Cc: dev, Dana Vardi, Liron Himi, Michael Shamis, stable

> From: Dana Vardi <danat@marvell.com>
> 
> This patch fix wrong clear memory session pointer.
> 
> When call mrvl_crypto_pmd_sym_session_clear function,
> it might cause the following error:
> CRYPTODEV: set_sym_session_private_data() line 497:
> Set private data for driver 1 not allowed
> 
> This fix set zeroes to mrvl_crypto_session pointer instead to
> rte_cryptodev_sym_session pointer, and will verify that
> rte_cryptodev_sym_session pointer will not lose data such as
> nb_drivers.
> 
> Bugzilla ID: 646
> Fixes: 8a61c83af2fa ("crypto/mrvl: add mrvl crypto driver")
> Cc: stable@dpdk.org
> 
> Signed-off-by: Dana Vardi <danat@marvell.com>
> Reviewed-by: Michael Shamis <michaelsh@marvell.com>
> Reviewed-by: Liron Himi <lironh@marvell.com>
> ---
Applied to dpdk-next-crypto

Thanks.

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2021-07-07 15:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-07-01  6:59 [dpdk-dev] [PATCH] crypto/mvsam: fix memset of mrvl sess private data danat
2021-07-07 15:41 ` Akhil Goyal

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).