[PATCH v2 7/8] crypto/ipsec_mb: do not free linear_sgl always

[Ciara Power]

From: Pablo de Lara <pablo.de.lara.gua...@intel.com>

linear_sgl buffer only needs to be freed
if it was allocated previously.

Signed-off-by: Pablo de Lara <pablo.de.lara.gua...@intel.com>
---
 drivers/crypto/ipsec_mb/pmd_aesni_mb.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/drivers/crypto/ipsec_mb/pmd_aesni_mb.c 
b/drivers/crypto/ipsec_mb/pmd_aesni_mb.c
index 58faf3502c..f83738a5eb 100644
--- a/drivers/crypto/ipsec_mb/pmd_aesni_mb.c
+++ b/drivers/crypto/ipsec_mb/pmd_aesni_mb.c
@@ -1898,6 +1898,7 @@ post_process_mb_job(struct ipsec_mb_qp *qp, IMB_JOB *job)
        struct rte_crypto_op *op = (struct rte_crypto_op *)job->user_data;
        struct aesni_mb_session *sess = NULL;
        uint8_t *linear_buf = NULL;
+       int sgl = 0;
 
 #ifdef AESNI_MB_DOCSIS_SEC_ENABLED
        uint8_t is_docsis_sec = 0;
@@ -1923,6 +1924,8 @@ post_process_mb_job(struct ipsec_mb_qp *qp, IMB_JOB *job)
                                        op->sym->m_dst->nb_segs > 1)) &&
                                        
!imb_lib_support_sgl_algo(job->cipher_mode)) {
                                linear_buf = (uint8_t *) job->user_data2;
+                               sgl = 1;
+
                                post_process_sgl_linear(op, job, sess, 
linear_buf);
                        }
 
@@ -1952,7 +1955,8 @@ post_process_mb_job(struct ipsec_mb_qp *qp, IMB_JOB *job)
                default:
                        op->status = RTE_CRYPTO_OP_STATUS_ERROR;
                }
-               rte_free(linear_buf);
+               if (sgl)
+                       rte_free(linear_buf);
        }
 
        /* Free session if a session-less crypto op */
-- 
2.25.1