test/crypto: fix null dereference of crypto op
authorCiara Power <ciara.power@intel.com>
Fri, 30 Oct 2020 13:11:26 +0000 (13:11 +0000)
committerAkhil Goyal <akhil.goyal@nxp.com>
Thu, 12 Nov 2020 20:25:17 +0000 (21:25 +0100)
commit3231a72a1ddfc19c38aa5909a0ac6c619ca77fbe
treefad5ca3373d6299de0a0828682f6207e6c8a2fa0
parent755d3cfe89ddbf2679d4909033f74fdcc4acc138
test/crypto: fix null dereference of crypto op

In two test cases, the op value is set by the return of the
process_crypto_request function, which may be NULL. The op->status
value was checked afterwards, which was causing a dereference issue.

To fix this, a temporary op variable is used to hold the return
from the process_crypto_request function, so the original op->status
can be checked after the possible NULL return value.
The original op value is then set to hold the temporary op value.

Coverity issue: 363452, 363465
Fixes: 4868f6591c6f ("test/crypto: add cases for raw datapath API")

Signed-off-by: Ciara Power <ciara.power@intel.com>
Acked-by: Akhil Goyal <akhil.goyal@nxp.com>
app/test/test_cryptodev.c