diff mbox series

[1/2] crypto/dpaa2_sec: fix to check queue pair array for null

Message ID 20200903171745.13985-1-hemant.agrawal@nxp.com
State New
Headers show
Series [1/2] crypto/dpaa2_sec: fix to check queue pair array for null | expand

Commit Message

Hemant Agrawal Sept. 3, 2020, 5:17 p.m. UTC 
dpdk-procinfo calls the crypto stats API, which results segmentation
fault on DPAA2_SEC.
The queue pair array will be NULL, when it is used without
configuring the SEC device.

Fixes: 02f35eee264b ("crypto/dpaa2_sec: support statistics")
Cc: stable@dpdk.org

Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>

---
 drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

-- 
2.17.1

Comments

Akhil Goyal Oct. 9, 2020, 7:09 p.m. UTC | #1 
> dpdk-procinfo calls the crypto stats API, which results segmentation

> fault on DPAA2_SEC.

> The queue pair array will be NULL, when it is used without

> configuring the SEC device.

> 

> Fixes: 02f35eee264b ("crypto/dpaa2_sec: support statistics")

> Cc: stable@dpdk.org

> 

> Signed-off-by: Hemant Agrawal <hemant.agrawal@nxp.com>

Series
Acked-by: Akhil Goyal <akhil.goyal@nxp.com>


Applied to dpdk-next-crypto

Thanks.
diff mbox series

Patch

diff --git a/drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c b/drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c
index 109e61ae1..f480aafd0 100644
--- a/drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c
+++ b/drivers/crypto/dpaa2_sec/dpaa2_sec_dpseci.c
@@ -3501,7 +3501,7 @@  void dpaa2_sec_stats_get(struct rte_cryptodev *dev,
 		return;
 	}
 	for (i = 0; i < dev->data->nb_queue_pairs; i++) {
-		if (qp[i] == NULL) {
+		if (qp == NULL || qp[i] == NULL) {
 			DPAA2_SEC_DEBUG("Uninitialised queue pair");
 			continue;
 		}