* [PATCH] test/cryptodev: remove unnecessary null check before free
@ 2023-07-10 20:22 Stephen Hemminger
2023-07-11 4:20 ` [EXT] " Gowrishankar Muthukrishnan
2023-07-11 8:51 ` Akhil Goyal
0 siblings, 2 replies; 3+ messages in thread
From: Stephen Hemminger @ 2023-07-10 20:22 UTC (permalink / raw)
To: dev; +Cc: Stephen Hemminger, gmuthukrishn
Found by devtools/cocci/nullfree.cocci
Fixes: 2d970c663314 ("cryptodev: add asymmetric SM2 algorithm")
Cc: gmuthukrishn@marvell.com
Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
---
app/test/test_cryptodev_asym.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/app/test/test_cryptodev_asym.c b/app/test/test_cryptodev_asym.c
index 15a1f3d6385e..391d122f530d 100644
--- a/app/test/test_cryptodev_asym.c
+++ b/app/test/test_cryptodev_asym.c
@@ -2654,8 +2654,7 @@ _test_sm2_enc(bool rnd_secret)
}
}
exit:
- if (pbuf != NULL)
- rte_free(pbuf);
+ rte_free(pbuf);
if (sess != NULL)
rte_cryptodev_asym_session_free(dev_id, sess);
--
2.39.2
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [EXT] [PATCH] test/cryptodev: remove unnecessary null check before free
2023-07-10 20:22 [PATCH] test/cryptodev: remove unnecessary null check before free Stephen Hemminger
@ 2023-07-11 4:20 ` Gowrishankar Muthukrishnan
2023-07-11 8:51 ` Akhil Goyal
1 sibling, 0 replies; 3+ messages in thread
From: Gowrishankar Muthukrishnan @ 2023-07-11 4:20 UTC (permalink / raw)
To: Stephen Hemminger, dev
Acked-by: Gowrishankar Muthukrishnan <gmuthukrishn@marvell.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [EXT] [PATCH] test/cryptodev: remove unnecessary null check before free
2023-07-10 20:22 [PATCH] test/cryptodev: remove unnecessary null check before free Stephen Hemminger
2023-07-11 4:20 ` [EXT] " Gowrishankar Muthukrishnan
@ 2023-07-11 8:51 ` Akhil Goyal
1 sibling, 0 replies; 3+ messages in thread
From: Akhil Goyal @ 2023-07-11 8:51 UTC (permalink / raw)
To: Stephen Hemminger, dev; +Cc: Gowrishankar Muthukrishnan
> Found by devtools/cocci/nullfree.cocci
>
> Fixes: 2d970c663314 ("cryptodev: add asymmetric SM2 algorithm")
> Cc: gmuthukrishn@marvell.com
> Signed-off-by: Stephen Hemminger <stephen@networkplumber.org>
> ---
Applied to dpdk-next-crypto
Thanks.
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-07-11 8:51 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-07-10 20:22 [PATCH] test/cryptodev: remove unnecessary null check before free Stephen Hemminger
2023-07-11 4:20 ` [EXT] " Gowrishankar Muthukrishnan
2023-07-11 8:51 ` 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).