<div dir="ltr"><div><div>Right, it's actually for the different return values, sorry to not mention it earlier.<br><br></div>Cheers,<br></div>Aaryaman<br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 11, 2016 at 10:16 PM, Henri Verbeet <span dir="ltr"><<a href="mailto:hverbeet@gmail.com" target="_blank">hverbeet@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>On 11 May 2016 at 17:50, Aaryaman Vasishta <<a href="mailto:jem456.vasishta@gmail.com" target="_blank">jem456.vasishta@gmail.com</a>> wrote:<br>
> If CreateTexture fails because of an invalid image, it doesn't leak a<br>
> reference to d3drm, unlike the InitFrom* calls. So I had to either do this,<br>
> or explicitly release texture->d3drm if InitFromImage fails.<br>
><br>
</span>That's not what the implementation in patch 2/6 does. It will leak if<br>
the texture is already initialised, but not when<br>
d3drm_validate_image() fails.<br>
</blockquote></div><br></div></div>