New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

fixed a memory leak and contract violation #575

Merged

saudet merged 1 commit into bytedeco:master from ennerf:master

Dec 6, 2016

Contributor

ennerf commented Dec 5, 2016

OpenCVFrameGrabber .grab() leaks memory and fails the identity test. Fixed by pre-allocating the target Mat.

FrameGrabber grabber = new OpenCVFrameGrabber(0);
grabber.start();
while (true) {
     if (grabber.grab() != grabber.grab())
         throw new IllegalStateException();
}


          fixed a memory leak and contract violation (grabber.grab() != grabber…

e0442c5

….grab()) in OpenCVFrameGrabber

ennerf mentioned this pull request

OpenCVFrameGrabber leaks memory #576

Closed

Member

saudet commented Dec 6, 2016

Nice catch! Thanks

saudet merged commit 9476b19 into bytedeco:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment