[tfaber] 71357: [AFD] - Don't overwrite values in a completed IRP in ReceiveActivity CORE-11276 #resolve

Author: tfaber
Date: Thu May 19 21:17:18 2016
New Revision: 71357

URL: http://svn.reactos.org/svn/reactos?rev=71357&view=rev
Log:
[AFD]
- Don't overwrite values in a completed IRP in ReceiveActivity
CORE-11276 #resolve

Modified:
    trunk/reactos/drivers/network/afd/afd/read.c

Modified: trunk/reactos/drivers/network/afd/afd/read.c
URL: http://svn.reactos.org/svn/reactos/trunk/reactos/drivers/network/afd/afd/rea...
==============================================================================
--- trunk/reactos/drivers/network/afd/afd/read.c	[iso-8859-1] (original)
+++ trunk/reactos/drivers/network/afd/afd/read.c	[iso-8859-1] Thu May 19 21:17:18 2016
@@ -153,7 +153,7 @@
     PIRP NextIrp;
     PIO_STACK_LOCATION NextIrpSp;
     PAFD_RECV_INFO RecvReq;
-    UINT TotalBytesCopied = 0, RetBytesCopied = 0;
+    UINT TotalBytesCopied = 0;
     NTSTATUS Status = STATUS_SUCCESS, RetStatus = STATUS_PENDING;

AFD_DbgPrint(MID_TRACE,("%p %p\n", FCB, Irp));
@@ -227,7 +227,6 @@
                 NextIrp->IoStatus.Information = TotalBytesCopied;
                 if( NextIrp == Irp ) {
                     RetStatus = Status;
-                    RetBytesCopied = TotalBytesCopied;
                 }
                 if( NextIrp->MdlAddress ) UnlockRequest( NextIrp, IoGetCurrentIrpStackLocation( NextIrp ) );
                 (void)IoSetCancelRoutine(NextIrp, NULL);
@@ -263,12 +262,6 @@
     }

AFD_DbgPrint(MID_TRACE,("RetStatus for irp %p is %x\n", Irp, RetStatus));
-
-    /* Sometimes we're called with a NULL Irp */
-    if( Irp ) {
-        Irp->IoStatus.Status = RetStatus;
-        Irp->IoStatus.Information = RetBytesCopied;
-    }

return RetStatus;
 }