[PATCH] user32: Fix some leaks on error path (Coverity).

[Sven Baars]

Signed-off-by: Sven Baars <sven.wine(a)gmail.com>
---
 dlls/user32/rawinput.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/dlls/user32/rawinput.c b/dlls/user32/rawinput.c
index 2085fd3f9f..56ea2497d1 100644
--- a/dlls/user32/rawinput.c
+++ b/dlls/user32/rawinput.c
@@ -141,6 +141,8 @@ static void find_hid_devices(void)
             if (!hid_devices)
             {
                 ERR("Failed to allocate memory.\n");
+                CloseHandle(file);
+                heap_free(path);
                 goto done;
             }
         }
-- 
2.17.1

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=53122

Your paranoid android.


=== debian9 (32 bit report) ===

user32:
win.c:10131: Test failed: GetActiveWindow() = 00000000
win.c:10131: Test failed: GetFocus() = 00000000
win.c:10133: Test failed: Expected foreground window 000E0120, got 00E300D4
win.c:10136: Test failed: Received WM_ACTIVATEAPP(0), did not expect it.
win.c:10143: Test failed: Expected foreground window 000E0120, got 00000000
win.c:10145: Test failed: GetActiveWindow() = 00000000
win.c:10145: Test failed: GetFocus() = 00000000
win.c:10150: Test failed: Expected foreground window 000E0120, got 00E300D4
win.c:10153: Test failed: Received WM_ACTIVATEAPP(1), did not expect it.