VideoProperties: fix memory leak when analyseGopStructure

Clement Champetier · Clement Champetier · commit 2ccf4b698549 · 2016-11-03T16:24:59.000+01:00
The data buffer of the VideoFrame will be allocated by the decoder.
diff --git a/src/AvTranscoder/properties/VideoProperties.cpp b/src/AvTranscoder/properties/VideoProperties.cpp
@@ -515,7 +515,7 @@ void VideoProperties::analyseGopStructure(IProgress& progress)
     size_t count = 0;
     int positionOfFirstKeyFrame = -1;
     int positionOfLastKeyFrame = -1;
-    VideoFrame frame(VideoFrameDesc(getWidth(), getHeight(), getPixelFormatName(getPixelProperties().getAVPixelFormat())));
+    VideoFrame frame(VideoFrameDesc(getWidth(), getHeight(), getPixelFormatName(getPixelProperties().getAVPixelFormat())), false);
     while(decoder.decodeNextFrame(frame))
     {
         AVFrame& avFrame = frame.getAVFrame();

Original file line number	Diff line number	Diff line change
`@@ -515,7 +515,7 @@ void VideoProperties::analyseGopStructure(IProgress& progress)`
`515`	`515`	`size_t count = 0;`
`516`	`516`	`int positionOfFirstKeyFrame = -1;`
`517`	`517`	`int positionOfLastKeyFrame = -1;`
`518`		`- VideoFrame frame(VideoFrameDesc(getWidth(), getHeight(), getPixelFormatName(getPixelProperties().getAVPixelFormat())));`
	`518`	`+ VideoFrame frame(VideoFrameDesc(getWidth(), getHeight(), getPixelFormatName(getPixelProperties().getAVPixelFormat())), false);`
`519`	`519`	`while(decoder.decodeNextFrame(frame))`
`520`	`520`	`{`
`521`	`521`	`AVFrame& avFrame = frame.getAVFrame();`