[FFmpeg-devel] [PATCH 2/2] lavd/sdl: reset context variables after destroy
george at nsup.org
Fri Feb 7 17:01:01 CET 2014
Le nonidi 19 pluviôse, an CCXXII, Lukasz Marek a écrit :
> SDL device reports double free or hangs when quiting.
> Reseting variables solves the issue.
I am a bit unsure about this: it actually fixes the actual double frees, but
it still means someone is trying to free something they should not be
freeing; the issue is only hidden, and other symptoms may be present.
Did you investigate to find what part of the code does the second freeing?
Not that I object to the patch: not leaving loaded pointers is usually good
practice anyway. But maybe add a comment somewhere:
/* XXX someone is trying to free this twice */
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 819 bytes
Desc: Digital signature
More information about the ffmpeg-devel