Aaryaman Vasishta : d3drm/tests: Fix leaks while creating device in tests (Valgrind).

Alexandre Julliard julliard at wine.codeweavers.com
Fri Jul 17 08:03:41 CDT 2015


Module: wine
Branch: master
Commit: 9939ac8fe0def693107cf4e470d8f6ac4a052d59
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=9939ac8fe0def693107cf4e470d8f6ac4a052d59

Author: Aaryaman Vasishta <jem456.vasishta at gmail.com>
Date:   Thu Jul 16 21:56:07 2015 +0530

d3drm/tests: Fix leaks while creating device in tests (Valgrind).

---

 dlls/d3drm/tests/d3drm.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/dlls/d3drm/tests/d3drm.c b/dlls/d3drm/tests/d3drm.c
index 1816214..1c80536 100644
--- a/dlls/d3drm/tests/d3drm.c
+++ b/dlls/d3drm/tests/d3drm.c
@@ -1999,10 +1999,14 @@ static void test_create_device_from_clipper(void)
 
     hr = IDirect3DRM2_CreateDeviceFromClipper(d3drm2, clipper, &driver, 0, 0, &device2);
     todo_wine ok(hr == D3DRMERR_BADVALUE, "Expected hr == D3DRMERR_BADVALUE, got %x.\n", hr);
+    if (SUCCEEDED(hr))
+        IDirect3DRMDevice2_Release(device2);
 
     /* If NULL is passed for clipper, CreateDeviceFromClipper returns D3DRMERR_BADVALUE */
     hr = IDirect3DRM2_CreateDeviceFromClipper(d3drm2, NULL, &driver, 0, 0, &device2);
     todo_wine ok(hr == D3DRMERR_BADVALUE, "Expected hr == D3DRMERR_BADVALUE, got %x.\n", hr);
+    if (SUCCEEDED(hr))
+        IDirect3DRMDevice2_Release(device2);
 
     hr = IDirect3DRM2_CreateDeviceFromClipper(d3drm2, clipper, &driver, 300, 200, &device2);
     ok(hr == D3DRM_OK, "Cannot create IDirect3DRMDevice2 interface (hr = %x).\n", hr);
@@ -2197,6 +2201,8 @@ static void test_create_device_from_surface(void)
     hr = IDirect3DRM2_CreateDeviceFromSurface(d3drm2, &driver, ddraw, surface, &device2);
     todo_wine ok(hr == DDERR_INVALIDCAPS, "Expected hr == DDERR_INVALIDCAPS, got %x.\n", hr);
     IDirectDrawSurface_Release(surface);
+    if (SUCCEEDED(hr))
+        IDirect3DRMDevice2_Release(device2);
 
     desc.ddsCaps.dwCaps |= DDSCAPS_3DDEVICE;
     hr = IDirectDraw_CreateSurface(ddraw, &desc, &surface, NULL);




More information about the wine-cvs mailing list