[PATCH] d3d8/tests: Fix resource leak.

[Andrey Gusev]

Signed-off-by: Andrey Gusev <andrey.goosev(a)gmail.com>
---
 dlls/d3d8/tests/device.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/dlls/d3d8/tests/device.c b/dlls/d3d8/tests/device.c
index a9188f0ee4..e372c25dc8 100644
--- a/dlls/d3d8/tests/device.c
+++ b/dlls/d3d8/tests/device.c
@@ -8514,6 +8514,7 @@ START_TEST(device)
             || current_mode.dmPelsHeight != registry_mode.dmPelsHeight)
     {
         skip("Current mode does not match registry mode, skipping test.\n");
+        FreeLibrary(d3d8_handle);
         return;
     }
 
@@ -8527,6 +8528,7 @@ START_TEST(device)
     if (!(d3d8 = Direct3DCreate8(D3D_SDK_VERSION)))
     {
         skip("could not create D3D8\n");
+        FreeLibrary(d3d8_handle);
         return;
     }
     IDirect3D8_Release(d3d8);
@@ -8602,4 +8604,5 @@ START_TEST(device)
     test_clip_planes_limits();
 
     UnregisterClassA("d3d8_test_wc", GetModuleHandleA(NULL));
+    FreeLibrary(d3d8_handle);
 }
-- 
2.13.6

[Józef Kucia]

On Mon, Jan 15, 2018 at 6:16 PM, Andrey Gusev <andrey.goosev(a)gmail.com> wrote:
> Signed-off-by: Andrey Gusev <andrey.goosev(a)gmail.com>
> ---
>  dlls/d3d8/tests/device.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/dlls/d3d8/tests/device.c b/dlls/d3d8/tests/device.c
> index a9188f0ee4..e372c25dc8 100644
> --- a/dlls/d3d8/tests/device.c
> +++ b/dlls/d3d8/tests/device.c
> @@ -8514,6 +8514,7 @@ START_TEST(device)
>              || current_mode.dmPelsHeight != registry_mode.dmPelsHeight)
>      {
>          skip("Current mode does not match registry mode, skipping test.\n");
> +        FreeLibrary(d3d8_handle);
>          return;
>      }
>
> @@ -8527,6 +8528,7 @@ START_TEST(device)
>      if (!(d3d8 = Direct3DCreate8(D3D_SDK_VERSION)))
>      {
>          skip("could not create D3D8\n");
> +        FreeLibrary(d3d8_handle);
>          return;
>      }
>      IDirect3D8_Release(d3d8);
> @@ -8602,4 +8604,5 @@ START_TEST(device)
>      test_clip_planes_limits();
>
> UnregisterClassA("d3d8_test_wc", GetModuleHandleA(NULL));
> +    FreeLibrary(d3d8_handle);
>  }
> --
> 2.13.6

I think it would be better to use GetModuleHandle() instead of LoadLibrary().