kernel32: Fix handling of GetOverlappedResult when status remains STATUS_PENDING.

Sebastian Lackner sebastian at fds-team.de
Wed Mar 30 22:19:43 CDT 2016


Spotted by Michael Müller.

Signed-off-by: Sebastian Lackner <sebastian at fds-team.de>
---
 dlls/kernel32/file.c       |    2 ++
 dlls/kernel32/tests/file.c |    5 +++++
 2 files changed, 7 insertions(+)

diff --git a/dlls/kernel32/file.c b/dlls/kernel32/file.c
index b6dba6a..a518e08 100644
--- a/dlls/kernel32/file.c
+++ b/dlls/kernel32/file.c
@@ -625,7 +625,9 @@ BOOL WINAPI GetOverlappedResult(HANDLE hFile, LPOVERLAPPED lpOverlapped,
         if (WaitForSingleObject( lpOverlapped->hEvent ? lpOverlapped->hEvent : hFile,
                                  INFINITE ) == WAIT_FAILED)
             return FALSE;
+
         status = lpOverlapped->Internal;
+        if (status == STATUS_PENDING) status = STATUS_SUCCESS;
     }
 
     *lpTransferred = lpOverlapped->InternalHigh;
diff --git a/dlls/kernel32/tests/file.c b/dlls/kernel32/tests/file.c
index 8353c21..a066692 100644
--- a/dlls/kernel32/tests/file.c
+++ b/dlls/kernel32/tests/file.c
@@ -3361,6 +3361,11 @@ static void test_overlapped(void)
         "wrong error %u\n", GetLastError() );
     ok( r == FALSE, "should return false\n");
 
+    r = GetOverlappedResult( 0, &ov, &result, TRUE );
+    ok( r == TRUE, "should return TRUE\n" );
+    ok( result == 0xabcd, "wrong result %u\n", result );
+    ok( ov.Internal == STATUS_PENDING, "expected STATUS_PENDING, got %08lx\n", ov.Internal );
+
     ResetEvent( ov.hEvent );
 
     SetLastError( 0xb00 );
-- 
2.7.1



More information about the wine-patches mailing list