Jeff Latimer lats@yless4u.com.au wrote:
+HRESULT WINAPI WMCreateEditor_close(IWMMetadataEditor *iface) +{
- FIXME("iface=%p\n", iface);
- HeapFree(GetProcessHeap(), 0, iface);
- return S_OK;
+}
WMCreateEditor_close() should free the implementation not an interface pointer (although they accidently happen to be the same because of implementation structure layout, but once somebody moves/adds fields that may change and lead to unpredictable results).