[PATCH] quartz: use FreeLibrary instead of CLoseHandle (Coverity)

Marcus Meissner marcus at jet.franken.de
Fri Nov 8 16:00:52 CST 2013


1127290 Incorrect deallocator used
---
 dlls/quartz/vmr9.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/quartz/vmr9.c b/dlls/quartz/vmr9.c
index 2dc12a6..cd1e139 100644
--- a/dlls/quartz/vmr9.c
+++ b/dlls/quartz/vmr9.c
@@ -838,7 +838,7 @@ static ULONG WINAPI VMR9Inner_Release(IUnknown * iface)
     {
         TRACE("Destroying\n");
         BaseControlWindow_Destroy(&This->baseControlWindow);
-        CloseHandle(This->hD3d9);
+        FreeLibrary(This->hD3d9);
 
         if (This->allocator)
             IVMRSurfaceAllocatorEx9_Release(This->allocator);
@@ -2308,7 +2308,7 @@ static HRESULT vmr_create(IUnknown *outer_unk, LPVOID *ppv, const CLSID *clsid)
 
 fail:
     BaseRendererImpl_Release(&pVMR->renderer.filter.IBaseFilter_iface);
-    CloseHandle(pVMR->hD3d9);
+    FreeLibrary(pVMR->hD3d9);
     CoTaskMemFree(pVMR);
     return hr;
 }
-- 
1.7.10.4




More information about the wine-patches mailing list