From 962e7acb2e50123e89cf8327ee3e2e52aa9b8f41 Mon Sep 17 00:00:00 2001
From: "pingkai.pk" <pingkai010@gmail.com>
Date: Wed, 8 Apr 2020 14:06:39 +0800
Subject: [PATCH] fix(cacheModule): fix memleak

Signed-off-by: pingkai.pk <pingkai010@gmail.com>
---
 framework/cacheModule/CacheModule.cpp            |  1 +
 framework/cacheModule/cache/CacheFileRemuxer.cpp | 15 +--------------
 framework/cacheModule/cache/CacheFileRemuxer.h   |  2 --
 3 files changed, 2 insertions(+), 16 deletions(-)

diff --git a/framework/cacheModule/CacheModule.cpp b/framework/cacheModule/CacheModule.cpp
index 7791ff8a5..6053d0218 100644
--- a/framework/cacheModule/CacheModule.cpp
+++ b/framework/cacheModule/CacheModule.cpp
@@ -24,6 +24,7 @@ CacheModule::~CacheModule()
 {
     stop();
     clearStreamMetas();
+    delete mCacheFileRemuxer;
 }
 
 void CacheModule::setCacheConfig(const CacheConfig &config)
diff --git a/framework/cacheModule/cache/CacheFileRemuxer.cpp b/framework/cacheModule/cache/CacheFileRemuxer.cpp
index 740f86787..cc89ef381 100644
--- a/framework/cacheModule/cache/CacheFileRemuxer.cpp
+++ b/framework/cacheModule/cache/CacheFileRemuxer.cpp
@@ -39,7 +39,6 @@ CacheFileRemuxer::~CacheFileRemuxer()
     }
 
     mFrameInfoQueue.clear();
-    clearStreamMetas();
 }
 
 void CacheFileRemuxer::addFrame(const unique_ptr<IAFPacket> &frame, StreamType type)
@@ -273,8 +272,7 @@ void CacheFileRemuxer::setResultCallback(function<void(bool)> callback)
 
 void CacheFileRemuxer::setStreamMeta(const vector<Stream_meta *> &streamMetas)
 {
-    clearStreamMetas();
-
+    mStreamMetas.clear();
     if (streamMetas.empty()) {
         return;
     }
@@ -284,17 +282,6 @@ void CacheFileRemuxer::setStreamMeta(const vector<Stream_meta *> &streamMetas)
     }
 }
 
-void CacheFileRemuxer::clearStreamMetas()
-{
-    if (!mStreamMetas.empty()) {
-        for (auto &item : mStreamMetas) {
-            releaseMeta(item);
-        }
-
-        mStreamMetas.clear();
-    }
-}
-
 void CacheFileRemuxer::sendError(const CacheRet &ret)
 {
     mRemuxSuc = false;
diff --git a/framework/cacheModule/cache/CacheFileRemuxer.h b/framework/cacheModule/cache/CacheFileRemuxer.h
index 318e7cb20..2d9b290a2 100644
--- a/framework/cacheModule/cache/CacheFileRemuxer.h
+++ b/framework/cacheModule/cache/CacheFileRemuxer.h
@@ -52,8 +52,6 @@ class CacheFileRemuxer {
 
     void setStreamMeta(const vector<Stream_meta *> &streamMetas);
 
-    void clearStreamMetas();
-
 private :
 
     void sendError(const CacheRet& ret);