quartz: Destroy BaseFilter after child class.

Austin English austinenglish at gmail.com
Mon Jul 21 14:07:17 CDT 2014


On Mon, Jul 21, 2014 at 12:04 PM, Thomas Faber <thomas.faber at reactos.org>
wrote:

> For the AsyncReader, this is a use after free on the critical section
> again.
>
> The NullRenderer doesn't have any pins, so that change is mostly
> cosmetical to align it with the other classes.
>
> The Parser change fixes a memory leak in addition to the use after free,
> because AVISplitter_Release manually decrements the refCount and calls
> Parser_Destroy instead of using Parser_Release -- so the filter's
> members were never released in this code path.
>
>
>
You forgot the patch.

Cheers,
-Austin
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.winehq.org/pipermail/wine-devel/attachments/20140721/5ffba4bb/attachment.html>


More information about the wine-devel mailing list