Józef Kucia : dxgi: Avoid possible NULL pointer access in d3d11_swapchain_SetFullscreenState( ).

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 80120d62eb60c9932d2322a08e2da62fb1ae6e15
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=80120d62eb60c9932d2322a08...

Author: Józef Kucia <jkucia(a)codeweavers.com>
Date:   Mon Jul  9 15:25:54 2018 +0200

dxgi: Avoid possible NULL pointer access in d3d11_swapchain_SetFullscreenState().

Signed-off-by: Józef Kucia <jkucia(a)codeweavers.com>
Signed-off-by: Henri Verbeet <hverbeet(a)codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

---

 dlls/dxgi/swapchain.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/dlls/dxgi/swapchain.c b/dlls/dxgi/swapchain.c
index fb91eec..ae7bde7 100644
--- a/dlls/dxgi/swapchain.c
+++ b/dlls/dxgi/swapchain.c
@@ -252,12 +252,11 @@ static HRESULT STDMETHODCALLTYPE DECLSPEC_HOTPATCH d3d11_swapchain_SetFullscreen
         if (swapchain->target)
             IDXGIOutput_Release(swapchain->target);
         swapchain->target = target;
-    }
-    else
-    {
-        IDXGIOutput_Release(target);
+        return S_OK;
     }
 
+    if (target)
+        IDXGIOutput_Release(target);
     return hr;
 }