From fd14525a52284d54dc331409e91c794e04d804e6 Mon Sep 17 00:00:00 2001
From: Jeffy Chen <jeffy.chen@rock-chips.com>
Date: Tue, 7 Apr 2020 15:30:48 +0800
Subject: [PATCH] [vp8e]: fix double-free when deinit vp8e

The ctx should be freed in enc_impl_deinit().

Change-Id: I65bac3a0211f7e470f625fcf5d2b9c1efde1d1c6
Signed-off-by: Jeffy Chen <jeffy.chen@rock-chips.com>
---
 mpp/codec/enc/vp8/vp8e_api.c | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/mpp/codec/enc/vp8/vp8e_api.c b/mpp/codec/enc/vp8/vp8e_api.c
index 940c8808..5cace50c 100644
--- a/mpp/codec/enc/vp8/vp8e_api.c
+++ b/mpp/codec/enc/vp8/vp8e_api.c
@@ -129,11 +129,9 @@ MPP_RET vp8e_deinit(void *ctx)
     Vp8eCtx *p = (Vp8eCtx *)ctx;
 
     vp8e_rc_dbg_func("enter\n");
-    if (p) {
-        if (p->rc)
-            mpp_free(p->rc);
-        mpp_free(p);
-    }
+
+    if (p->rc)
+        mpp_free(p->rc);
 
     vp8e_rc_dbg_func("leave\n");
     return MPP_OK;