[PATCH 04/20] crypto/qat: fix a memory leak when set encrypt key fail

[Weiguo Li]

We allocated memory for 'in', we don't free it when AES_set_encrypt_key()
fails and it will lead to memory leak.
We can move set_encrypt_key() ahead of the memory allocation to fix it.

Fixes: 1703e94ac5ce ("qat: add driver for QuickAssist devices")

Signed-off-by: Weiguo Li <liw...@foxmail.com>
---
 drivers/crypto/qat/qat_sym_session.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/drivers/crypto/qat/qat_sym_session.c 
b/drivers/crypto/qat/qat_sym_session.c
index 8ca475ca8b..3dc13942cb 100644
--- a/drivers/crypto/qat/qat_sym_session.c
+++ b/drivers/crypto/qat/qat_sym_session.c
@@ -1400,18 +1400,17 @@ static int qat_sym_do_precomputes(enum 
icp_qat_hw_auth_algo hash_alg,
                memset(p_state_buf, 0, ICP_QAT_HW_GALOIS_H_SZ +
                                ICP_QAT_HW_GALOIS_LEN_A_SZ +
                                ICP_QAT_HW_GALOIS_E_CTR0_SZ);
+               if (AES_set_encrypt_key(auth_key, auth_keylen << 3,
+                       &enc_key) != 0) {
+                       return -EFAULT;
+               }
                in = rte_zmalloc("working mem for key",
                                ICP_QAT_HW_GALOIS_H_SZ, 16);
                if (in == NULL) {
                        QAT_LOG(ERR, "Failed to alloc memory");
                        return -ENOMEM;
                }
-
                memset(in, 0, ICP_QAT_HW_GALOIS_H_SZ);
-               if (AES_set_encrypt_key(auth_key, auth_keylen << 3,
-                       &enc_key) != 0) {
-                       return -EFAULT;
-               }
                AES_encrypt(in, out, &enc_key);
                *p_state_len = ICP_QAT_HW_GALOIS_H_SZ +
                                ICP_QAT_HW_GALOIS_LEN_A_SZ +
-- 
2.25.1