]> git.droids-corp.org - dpdk.git/commit
crypto/ipsec_mb: fix queue setup null pointer dereference
authorCiara Power <ciara.power@intel.com>
Fri, 10 Dec 2021 14:09:50 +0000 (14:09 +0000)
committerAkhil Goyal <gakhil@marvell.com>
Fri, 21 Jan 2022 09:17:35 +0000 (10:17 +0100)
commitf51e40d1bf5efe7e659110613c63f1beb740d203
treec75f60c30ccf0444d29dd93d42cebd39aa5ab647
parent938d4e796a2217fa25e9dd2e7d897452c2e909ad
crypto/ipsec_mb: fix queue setup null pointer dereference

When setting up a qp in a secondary process, the local qp pointer is set
to the stored device qp, configured by the primary process for that
device, but only if that device qp is not NULL.
If the device qp was not set up correctly by the primary process and has
a NULL value, the local qp variable stays at the default initialised
value, NULL. This causes a NULL pointer dereference later in the
function when using the qp value.

This is fixed by always setting the local qp to the value of the device
qp stored, and then checking if qp is NULL, returning an error if it is.

Coverity issue: 374382
Fixes: 72a169278a56 ("crypto/ipsec_mb: support multi-process")
Cc: stable@dpdk.org
Signed-off-by: Ciara Power <ciara.power@intel.com>
Acked-by: Fan Zhang <roy.fan.zhang@intel.com>
drivers/crypto/ipsec_mb/ipsec_mb_ops.c