test/crypto: fix return value from session
Checks
Commit Message
Fixing return value if session failure occurs
due to unsupported feature.
Fixes: eec136f3c54f ("aesni_gcm: add driver for AES-GCM crypto operations")
Cc: stable@dpdk.org
Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
---
app/test/test_cryptodev.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
Comments
>
> Fixing return value if session failure occurs due to unsupported feature.
>
> Fixes: eec136f3c54f ("aesni_gcm: add driver for AES-GCM crypto
> operations")
> Cc: stable@dpdk.org
>
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
Acked-by: Anoob Joseph <anoobj@marvell.com>
> -----Original Message-----
> From: Tejasree Kondoj <ktejasree@marvell.com>
> Sent: Friday 5 May 2023 17:54
> To: Akhil Goyal <gakhil@marvell.com>; Fan Zhang
> <fanzhang.oss@gmail.com>
> Cc: Anoob Joseph <anoobj@marvell.com>; Power, Ciara
> <ciara.power@intel.com>; Jerin Jacob <jerinj@marvell.com>; Aakash
> Sasidharan <asasidharan@marvell.com>; Gowrishankar Muthukrishnan
> <gmuthukrishn@marvell.com>; Vidya Sagar Velumuri
> <vvelumuri@marvell.com>; dev@dpdk.org; stable@dpdk.org
> Subject: [PATCH] test/crypto: fix return value from session
>
> Fixing return value if session failure occurs due to unsupported feature.
>
> Fixes: eec136f3c54f ("aesni_gcm: add driver for AES-GCM crypto
> operations")
> Cc: stable@dpdk.org
>
> Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> ---
> app/test/test_cryptodev.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
Acked-by: Ciara Power <ciara.power@intel.com>
> > Subject: [PATCH] test/crypto: fix return value from session
> >
> > Fixing return value if session failure occurs due to unsupported feature.
> >
> > Fixes: eec136f3c54f ("aesni_gcm: add driver for AES-GCM crypto
> > operations")
> > Cc: stable@dpdk.org
> >
> > Signed-off-by: Tejasree Kondoj <ktejasree@marvell.com>
> > ---
> > app/test/test_cryptodev.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
>
> Acked-by: Ciara Power <ciara.power@intel.com>
Applied to dpdk-next-crypto
Thanks.
@@ -8778,7 +8778,7 @@ test_authenticated_encryption(const struct aead_test_data *tdata)
tdata->key.data, tdata->key.len,
tdata->aad.len, tdata->auth_tag.len,
tdata->iv.len);
- if (retval < 0)
+ if (retval != TEST_SUCCESS)
return retval;
if (tdata->aad.len > MBUF_SIZE) {
@@ -11482,7 +11482,7 @@ test_authenticated_decryption(const struct aead_test_data *tdata)
tdata->key.data, tdata->key.len,
tdata->aad.len, tdata->auth_tag.len,
tdata->iv.len);
- if (retval < 0)
+ if (retval != TEST_SUCCESS)
return retval;
/* alloc mbuf and set payload */