<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Mon, Jul 21, 2014 at 12:04 PM, Thomas Faber <span dir="ltr"><<a href="mailto:thomas.faber@reactos.org" target="_blank">thomas.faber@reactos.org</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">For the AsyncReader, this is a use after free on the critical section<br>
again.<br>
<br>
The NullRenderer doesn't have any pins, so that change is mostly<br>
cosmetical to align it with the other classes.<br>
<br>
The Parser change fixes a memory leak in addition to the use after free,<br>
because AVISplitter_Release manually decrements the refCount and calls<br>
Parser_Destroy instead of using Parser_Release -- so the filter's<br>
members were never released in this code path.<br>
<br>
<br>
</blockquote></div><br>You forgot the patch.<br><br></div><div class="gmail_extra">Cheers,<br></div><div class="gmail_extra">-Austin
</div></div>