VideoWriter.open Leak (Bug #3206)

Added by Jonathan Herriott over 11 years ago. Updated over 11 years ago.

Status:Done Start date:2013-08-08
Priority:High Due date:
Assignee:- % Done:

100%
Category:highgui-video
Target version:2.4.7 Estimated time:1.00 hour
Affected version:branch 'master' (2.9) Operating System:Any
Difficulty:Easy HW Platform:Any
Pull request:https://github.com/Itseez/opencv/pull/1246

Description

See Ugo Galassi's suggestion in http://code.opencv.org/issues/775 which so far appears to work for me as valgrind reports no leaks after applying this patch. This bug still exists on master source and I have seen reports of this since 2.1 looking back through people's compaints.

Related issues

related to Patch #775: Memory Leak when using ffmpeg Video Writer Cancelled

Associated revisions

Revision d11746e5
Added by Steven Puttemans over 11 years ago

Added fix mentioned in bug #3206
VideoWriter memory leak solved.

Revision e810f305
Added by Vadim Pisarevsky over 10 years ago

Merge pull request #3206 from stonier:parallel_qt

History

#1
Updated by Victor Kocheganov over 11 years ago

Hello Jonathan!

Thanks for perseverance and submitting this issue again. May be you could provide our source with patch from http://code.opencv.org/issues/775 (please see http://code.opencv.org/projects/opencv/wiki/How_to_contribute for details)? It would be really great and appreciated by the community!

Thank you in advance,
Victor.

  • Status changed from New to Open
  • Target version set to 2.4.7
  • Assignee set to Jonathan Herriott

#2
Updated by Steven Puttemans over 11 years ago

Give me a shout if you do not have the time to create a pull request.
I will gladly apply the fix and submit one for you.

#3
Updated by Steven Puttemans over 11 years ago

Since I had 5 minutes left, I created a small pull request for this: https://github.com/Itseez/opencv/pull/1246
It would be nice if you could keep track of bugs popping of after buildbot compilation and suggest changes if they happen.

#4
Updated by Victor Kocheganov over 11 years ago

Steven,

thank you for this Pull Request too!

  • Pull request set to https://github.com/Itseez/opencv/pull/1246

#5
Updated by Steven Puttemans over 11 years ago

Pull request merged. Problem solved!

  • Assignee deleted (Jonathan Herriott)
  • Status changed from Open to Done
  • % Done changed from 0 to 100

Also available in: Atom PDF