commit 77c3997e905d4e9abfc9b4767294aa8eca309e8a
author Alexandre Julliard <julliard@winehq.org> Fri Sep 25 15:57:41 2009 +0200
committer Alexandre Julliard <julliard@winehq.org> Fri Sep 25 15:58:55 2009 +0200
tree 9cf025f31bfd52d418a6208a3536d9dc3ecd91db
parent 33bfcdd04452accf40256e1693b6a934e817dec9

windowscodecs: Fix incorrect overflow check caused by test mistake.

dlls/windowscodecs/stream.c

diff --git a/dlls/windowscodecs/stream.c b/dlls/windowscodecs/stream.c
index 5672ff5..c59a9d2 100644
--- a/dlls/windowscodecs/stream.c
+++ b/dlls/windowscodecs/stream.c
@@ -129,13 +129,12 @@
     LARGE_INTEGER NewPosition;
     TRACE("(%p)\n", This);
 
-    if (dlibMove.QuadPart > 0xFFFFFFFF) return HRESULT_FROM_WIN32(ERROR_ARITHMETIC_OVERFLOW);
-
     if (dwOrigin == STREAM_SEEK_SET) NewPosition.QuadPart = dlibMove.QuadPart;
     else if (dwOrigin == STREAM_SEEK_CUR) NewPosition.QuadPart = This->dwCurPos + dlibMove.QuadPart;
     else if (dwOrigin == STREAM_SEEK_END) NewPosition.QuadPart = This->dwMemsize + dlibMove.QuadPart;
     else return E_INVALIDARG;
 
+    if (NewPosition.u.HighPart) return HRESULT_FROM_WIN32(ERROR_ARITHMETIC_OVERFLOW);
     if (NewPosition.QuadPart > This->dwMemsize) return E_INVALIDARG;
     if (NewPosition.QuadPart < 0) return E_INVALIDARG;
     This->dwCurPos = NewPosition.u.LowPart;