comctl32: In certain circumstances, native comctl32's animation control accepts…

comctl32: In certain circumstances, native comctl32's animation control accepts negative start positions for the ACM_PLAY message. These circumstances seems to be after a seek is performed, but we play it safe and allow negative start positions to be treated as zero always.

comctl32: In certain circumstances, native comctl32's animation control accepts…
8cda4a8d · Rob Shearman · Alexandre Julliard · 51e2e1cc · 8cda4a8d
Commit 8cda4a8d authored Jan 31, 2008 by Rob Shearman Committed by Alexandre Julliard Feb 04, 2008
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

animate.c dlls/comctl32/animate.c +4 -0

No files found.
--- a/dlls/comctl32/animate.c
+++ b/dlls/comctl32/animate.c
@@ -434,6 +434,10 @@ static LRESULT ANIMATE_Play(ANIMATE_INFO *infoPtr, UINT cRepeat, WORD wFrom, WOR
    TRACE("(repeat=%d from=%d to=%d);\n",
 	  infoPtr->nLoop, infoPtr->nFromFrame, infoPtr->nToFrame);

+    if (infoPtr->nFromFrame >= infoPtr->mah.dwTotalFrames &&
+        (SHORT)infoPtr->nFromFrame < 0)
+        infoPtr->nFromFrame = 0;
+
    if (infoPtr->nFromFrame > infoPtr->nToFrame ||
 	infoPtr->nToFrame >= infoPtr->mah.dwTotalFrames)
 	return FALSE;