[MPlayer-dev-eng] [PATCH] memory leak in vo_macosx
Alexander Strange
astrange at ithinksw.com
Thu Apr 9 23:38:15 CEST 2009
On Apr 9, 2009, at 5:30 PM, Guillaume POIRIER wrote:
> Hello,
>
> 2009/4/9 Alexander Strange <astrange at ithinksw.com>:
>> vo_macosx leaks ~80 bytes with each call to flip_page, because this
>> line:
>> error = CVOpenGLTextureCacheCreateTextureFromImage(NULL,
>> textureCache, frameBuffers[image_page], 0, &texture);
>>
>> overwrites 'texture' without freeing the old one first (checked with
>> 'leaks'). I'm not sure when this started happening. Attached patch
>> fixes it
>> by releasing the old texture.
>
> Thanks for the patch, but have you tested it before submitting it? I
> doesn't compile! Attached patch has the changes your meant.
> I've tested it on several videos, and confirmed that it didn't cause
> any regression, but I'd like to use it more one real world usage to
> confirm it does work as expected.
How didn't it compile? I did accidentally introduce a whitespace
change, so I should've read the diff output again, but aside from that
they're the same change (and both work for me). Maybe you got a patch
conflict applying it?
More information about the MPlayer-dev-eng
mailing list