Skip to content

W
wine-winehq
Commit cf1b9fe2 authored by Sven Baars's avatar Sven Baars Committed by Alexandre Julliard
Browse files
Options

dinput: Free the device after a callback (Valgrind).

Signed-off-by: 's avatarSven Baars <sven.wine@gmail.com> Signed-off-by: 's avatarAlexandre Julliard <julliard@winehq.org>
parent 2543295a
Hide whitespace changes
Inline Side-by-side
Showing with 4 additions and 0 deletions
dlls/dinput/dinput_main.c
View file @ cf1b9fe2
...@@ -1104,10 +1104,12 @@ static HRESULT WINAPI IDirectInput8AImpl_EnumDevicesBySemantics( ...@@ -1104,10 +1104,12 @@ static HRESULT WINAPI IDirectInput8AImpl_EnumDevicesBySemantics(
if (lpCallback(&didevis[i], lpdid, callbackFlags, --remain, pvRef) == DIENUM_STOP) if (lpCallback(&didevis[i], lpdid, callbackFlags, --remain, pvRef) == DIENUM_STOP)
{ {
IDirectInputDevice_Release(lpdid);
HeapFree(GetProcessHeap(), 0, didevis); HeapFree(GetProcessHeap(), 0, didevis);
HeapFree(GetProcessHeap(), 0, username_w); HeapFree(GetProcessHeap(), 0, username_w);
return DI_OK; return DI_OK;
} }
IDirectInputDevice_Release(lpdid);
} }
HeapFree(GetProcessHeap(), 0, didevis); HeapFree(GetProcessHeap(), 0, didevis);
...@@ -1130,9 +1132,11 @@ static HRESULT WINAPI IDirectInput8AImpl_EnumDevicesBySemantics( ...@@ -1130,9 +1132,11 @@ static HRESULT WINAPI IDirectInput8AImpl_EnumDevicesBySemantics(
if (lpCallback(&didevi, lpdid, callbackFlags, --remain, pvRef) == DIENUM_STOP) if (lpCallback(&didevi, lpdid, callbackFlags, --remain, pvRef) == DIENUM_STOP)
{ {
IDirectInputDevice_Release(lpdid);
HeapFree(GetProcessHeap(), 0, username_w); HeapFree(GetProcessHeap(), 0, username_w);
return DI_OK; return DI_OK;
} }
IDirectInputDevice_Release(lpdid);
} }
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment