Merge pull request #274 from valnoel/fix_filtergraph_memory_leak

cchampet · web-flow · commit fde92307cd40 · 2016-07-27T11:31:19.000-04:00
Fix filter graph memory leak
diff --git a/src/AvTranscoder/filter/FilterGraph.hpp b/src/AvTranscoder/filter/FilterGraph.hpp
@@ -52,6 +52,8 @@ class AvExport FilterGraph
      *                               filter 1 -> filter 2 -> output
      *                                 |
      *                      input 2 ---|
+     * @warning the output frame must be cleared once it has been used
+     * @see the av_buffersink_get_frame function documentation
      */
     void process(const std::vector<Frame*>& inputs, Frame& output);
 
diff --git a/src/AvTranscoder/transcoder/StreamTranscoder.cpp b/src/AvTranscoder/transcoder/StreamTranscoder.cpp
@@ -550,6 +550,8 @@ bool StreamTranscoder::processTranscode()
         LOG_DEBUG("Convert")
         _transform->convert(*_filteredData, *_transformedData);
 
+        _filteredData->clear();
+
         LOG_DEBUG("Encode")
         _outputEncoder->encodeFrame(*_transformedData, data);
     }

Original file line number	Diff line number	Diff line change
`@@ -550,6 +550,8 @@ bool StreamTranscoder::processTranscode()`
`550`	`550`	`LOG_DEBUG("Convert")`
`551`	`551`	`_transform->convert(_filteredData, _transformedData);`
`552`	`552`
	`553`	`+ _filteredData->clear();`
	`554`	`+`
`553`	`555`	`LOG_DEBUG("Encode")`
`554`	`556`	`_outputEncoder->encodeFrame(*_transformedData, data);`
`555`	`557`	`}`