fix(cacheModule): fix memleak

Signed-off-by: pingkai.pk <pingkai010@gmail.com>
alibaba · Apr 8, 2020 · 962e7ac · 962e7ac
1 parent a13e090
commit 962e7ac
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 16 deletions.
diff --git 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
@@ -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
@@ -52,8 +52,6 @@ class CacheFileRemuxer {
 
     void setStreamMeta(const vector<Stream_meta *> &streamMetas);
 
-    void clearStreamMetas();
-
 private :
 
     void sendError(const CacheRet& ret);