Tested-by: Zhaoyan Chen <zhaoyan.c...@intel.com> - Checked patch: success - Apply patch: success - Compilation: success
Tested basic openssl_pmd tests and check the extra bytes. Passed. Thanks /Joey > -----Original Message----- > From: dev [mailto:dev-boun...@dpdk.org] On Behalf Of Piotr Azarewicz > Sent: Monday, January 9, 2017 10:46 PM > To: De Lara Guarch, Pablo <pablo.de.lara.gua...@intel.com>; dev@dpdk.org > Cc: sta...@dpdk.org > Subject: [dpdk-dev] [PATCH] crypto/openssl: fix that remove unneeded > check > > EVP_CIPHER_CTX_set_padding() function always returns 1, so the check is > unneeded. > > Fixes: d61f70b4c918 ("crypto/libcrypto: add driver for OpenSSL library") > > Signed-off-by: Piotr Azarewicz <piotrx.t.azarew...@intel.com> > --- > drivers/crypto/openssl/rte_openssl_pmd.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/crypto/openssl/rte_openssl_pmd.c > b/drivers/crypto/openssl/rte_openssl_pmd.c > index 832ea1d..312154a 100644 > --- a/drivers/crypto/openssl/rte_openssl_pmd.c > +++ b/drivers/crypto/openssl/rte_openssl_pmd.c > @@ -522,8 +522,7 @@ > if (EVP_DecryptInit_ex(ctx, algo, NULL, key, iv) <= 0) > goto process_cipher_decrypt_err; > > - if (EVP_CIPHER_CTX_set_padding(ctx, 0) <= 0) > - goto process_cipher_decrypt_err; > + EVP_CIPHER_CTX_set_padding(ctx, 0); > > if (EVP_DecryptUpdate(ctx, dst, &dstlen, src, srclen) <= 0) > goto process_cipher_decrypt_err; > -- > 1.7.9.5
