winmm: Respect return value from joyGetPos (Coverity).

Signed-off-by: André Hentschel <nerv@dawncrow.de> Signed-off-by: Andrew Eikum <aeikum@codeweavers.com> Signed-off-by: Alexandre Julliard <julliard@winehq.org>

winmm: Respect return value from joyGetPos (Coverity).
64f600fe · André Hentschel · Alexandre Julliard · bbc42a23 · 64f600fe
Commit 64f600fe authored Dec 22, 2015 by André Hentschel Committed by Alexandre Julliard Dec 23, 2015
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 1 deletion

joystick.c dlls/winmm/joystick.c +7 -1

No files found.
--- a/dlls/winmm/joystick.c
+++ b/dlls/winmm/joystick.c
@@ -84,6 +84,7 @@ static	void	CALLBACK	JOY_Timer(HWND hWnd, UINT wMsg, UINT_PTR wTimer, DWORD dwTi
 {
    int			i;
    WINE_JOYSTICK*	joy;
+    MMRESULT		res;
    JOYINFO		ji;
    LONG		pos;
    unsigned 		buttonChange;
@@ -93,7 +94,12 @@ static	void	CALLBACK	JOY_Timer(HWND hWnd, UINT wMsg, UINT_PTR wTimer, DWORD dwTi

 	if (joy->hCapture != hWnd) continue;

-	joyGetPos(i, &ji);
+	res = joyGetPos(i, &ji);
+	if (res != JOYERR_NOERROR) {
+	    WARN("joyGetPos failed: %08x\n", res);
+	    continue;
+	}
+
 	pos = MAKELONG(ji.wXpos, ji.wYpos);

 	if (!joy->bChanged ||