[PATCH] winex11.drv: Fix a memory leak (Coverity).

Sven Baars sven.wine at gmail.com
Fri Sep 6 15:24:11 CDT 2019


Signed-off-by: Sven Baars <sven.wine at gmail.com>
---
 dlls/winex11.drv/clipboard.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/dlls/winex11.drv/clipboard.c b/dlls/winex11.drv/clipboard.c
index a37530fdbf..c51954f1bf 100644
--- a/dlls/winex11.drv/clipboard.c
+++ b/dlls/winex11.drv/clipboard.c
@@ -1849,13 +1849,12 @@ static BOOL request_selection_contents( Display *display, BOOL changed )
                last_size != size ||
                memcmp( last_data, data, size ));
 
-    if (!changed)
+    if (!changed || !OpenClipboard( clipboard_hwnd ))
     {
         HeapFree( GetProcessHeap(), 0, data );
         return FALSE;
     }
 
-    if (!OpenClipboard( clipboard_hwnd )) return FALSE;
     TRACE( "selection changed, importing\n" );
     EmptyClipboard();
     is_clipboard_owner = TRUE;
-- 
2.17.1




More information about the wine-devel mailing list