Thanks for spotting. I will send an additional patch for that or do an update.<br><br><div class="gmail_quote">2012/11/6 Nikolay Sivov <span dir="ltr"><<a href="mailto:bunglehead@gmail.com" target="_blank">bunglehead@gmail.com</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On 11/6/2012 14:09, Alistair Leslie-Hughes wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
On 6/11/2012 7:05 PM, Christian Costa wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
� static HRESULT WINAPI IDirectMusicLoaderImpl_<u></u>IDirectMusicLoader_GetObject(<u></u>LPDIRECTMUSICLOADER8 iface, LPDMUS_OBJECTDESC pDesc, REFIID riid, LPVOID* ppv)<br>
@@ -880,7 +884,7 @@ HRESULT WINAPI DMUSIC_<u></u>CreateDirectMusicLoaderImpl (LPCGUID lpcGUID, LPVOID *ppob<br>
� � � � � return E_OUTOFMEMORY;<br>
� � � }<br>
� � � obj->IDirectMusicLoader8_<u></u>iface.lpVtbl = &DirectMusicLoader_Loader_<u></u>Vtbl;<br>
- � �obj->dwRef = 0; /* will be inited with QueryInterface */<br>
+ � �obj->ref = 0; /* Will be inited with QueryInterface */<br>
</blockquote>
<br>
Shouldn't obj->ref start at 1 or else it will never be freed.<br>
</blockquote></div></div>
It's supposed to be incremented later with QI call, see comment. This module is still dusty and COM cleanup is necessary, this is ugly:<br>
<br>
---<br>
� � IDirectMusicLoader_SetObject ((LPDIRECTMUSICLOADER8)obj, &Desc);<br>
---<br>
� � return IDirectMusicLoaderImpl_<u></u>IDirectMusicLoader_<u></u>QueryInterface ((LPDIRECTMUSICLOADER8)obj, lpcGUID, ppobj);<br>
---<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Best Regards<br>
�Alistair Leslie-Hughes<br>
<br>
<br>
<br>
<br>
</blockquote>
<br>
<br>
<br>
</blockquote></div><br>