FilterGraph: fixed memory deallocation

Clement Champetier · Clement Champetier · commit 72e6d5b22db6 · 2016-07-20T16:48:30.000+02:00
refFrame method allocates new data that will be freed by calling clear
method or the destructor of the referenced frame.
diff --git a/src/AvTranscoder/data/decoded/Frame.hpp b/src/AvTranscoder/data/decoded/Frame.hpp
@@ -64,7 +64,8 @@ class AvExport Frame
 
     /**
      * @brief Copy frame properties and create a new reference to data of the given frame.
-     * @warning This method allocates new data that will be freed only by calling the destructor of the referenced frame.
+     * @warning This method allocates new data that will be freed by calling clear method or the destructor of the referenced frame.
+     * @see clear
      */
     void refFrame(const Frame& otherFrame);
 
diff --git a/src/AvTranscoder/filter/FilterGraph.cpp b/src/AvTranscoder/filter/FilterGraph.cpp
@@ -40,6 +40,7 @@ void FilterGraph::process(const std::vector<Frame*>& inputs, Frame& output)
     if(!hasFilters())
     {
         LOG_DEBUG("No filter to process: reference first input frame to the given output.")
+        output.clear();
         output.refFrame(*inputs.at(0));
         return;
     }

Original file line number	Diff line number	Diff line change
`@@ -40,6 +40,7 @@ void FilterGraph::process(const std::vector<Frame*>& inputs, Frame& output)`
`40`	`40`	`if(!hasFilters())`
`41`	`41`	`{`
`42`	`42`	`LOG_DEBUG("No filter to process: reference first input frame to the given output.")`
	`43`	`+ output.clear();`
`43`	`44`	`output.refFrame(*inputs.at(0));`
`44`	`45`	`return;`
`45`	`46`	`}`