Changeset 8638 for trunk/src/kernel32/overlappedio.cpp

Timestamp:

Jun 10, 2002, 7:46:53 PM (23 years ago)

Author:

sandervl

Message:

Don't use user supplied pointers in ReadFile/WriteFile for overlapped IO

File:

• trunk/src/kernel32/overlappedio.cpp (modified) (5 diffs)

trunk/src/kernel32/overlappedio.cpp

@@ -1 @@
-/* $Id: overlappedio.cpp,v 1.14 2001-12-14 13:45:19 sandervl Exp $ */
+/* $Id: overlappedio.cpp,v 1.15 2002-06-10 17:46:53 sandervl Exp $ */
 
 /*
@@ -243 +243 @@
                     lpOverlapped->Internal     = lpRequest->dwLastError;
                     lpOverlapped->InternalHigh = dwResult;
-                    if(lpRequest->lpdwResult) {
-                        *lpRequest->lpdwResult = dwResult;
-                    }
+
+                    //must NOT store result in specified location!!! (stack corruption)
+                    //if(lpRequest->lpdwResult) {
+                    //    *lpRequest->lpdwResult = dwResult;
+                    //}
 #ifdef DEBUG
                     if(lpRequest->dwAsyncType == ASYNCIO_READ) {
@@ -330 +332 @@
     lpRequest->lpBuffer            = lpBuffer;
     lpRequest->nNumberOfBytes      = nNumberOfBytesToWrite;
-    lpRequest->lpdwResult          = lpNumberOfBytesWritten;
+    //must NOT store result in specified location!!! (stack corruption)
+////    lpRequest->lpdwResult          = lpNumberOfBytesWritten;
+    lpRequest->lpdwResult          = NULL;
     lpRequest->lpOverlapped        = lpOverlapped;
     lpRequest->lpCompletionRoutine = lpCompletionRoutine;
@@ -391 +395 @@
     lpRequest->lpBuffer            = lpBuffer;
     lpRequest->nNumberOfBytes      = nNumberOfBytesToRead;
-    lpRequest->lpdwResult          = lpNumberOfBytesRead;
+    //must NOT store result in specified location!!! (stack corruption)
+////    lpRequest->lpdwResult          = lpNumberOfBytesRead;
+    lpRequest->lpdwResult          = NULL;
     lpRequest->lpOverlapped        = lpOverlapped;
     lpRequest->lpCompletionRoutine = lpCompletionRoutine;
@@ -451 +457 @@
     lpRequest->lpBuffer            = NULL;
     lpRequest->nNumberOfBytes      = 0;
+    //must store result also in specified location
     lpRequest->lpdwResult          = lpfdwEvtMask;
     lpRequest->lpOverlapped        = lpOverlapped;