From: Jeffrey Altman <jaltman@your-file-system.com>
Date: Thu, 9 Jan 2014 14:57:33 +0000 (-0500)
Subject: Windows: Mark Irp Pending before Deferring
X-Git-Tag: upstream/1.8.0_pre1^2~858
X-Git-Url: https://git.michaelhowe.org/gitweb/?a=commitdiff_plain;h=d5bcff7429dd9972b17055a9abd98f3a590dee68;p=packages%2Fo%2Fopenafs.git

Windows: Mark Irp Pending before Deferring

After CcDeferWrite() is called we no longer have access to the
current Irp.  If we mark it deferred after calling CcDeferWrite()
we might mark the wrong thing.

Change-Id: Id4b1bbd241b5e2acafc8d015e85966cb80518dde
Reviewed-on: http://gerrit.openafs.org/10684
Tested-by: BuildBot <buildbot@rampaginggeek.com>
Reviewed-by: Jeffrey Altman <jaltman@your-file-system.com>
---

diff --git a/src/WINNT/afsrdr/kernel/lib/AFSWorker.cpp b/src/WINNT/afsrdr/kernel/lib/AFSWorker.cpp
index ed912c394..53354138a 100644
--- a/src/WINNT/afsrdr/kernel/lib/AFSWorker.cpp
+++ b/src/WINNT/afsrdr/kernel/lib/AFSWorker.cpp
@@ -2750,10 +2750,10 @@ AFSDeferWrite( IN PDEVICE_OBJECT DeviceObject,
                       __FUNCTION__,
                       pWorkItem));
 
-        CcDeferWrite( FileObject, AFSPostedDeferredWrite, pWorkItem, NULL, BytesToWrite, bRetrying);
-
         IoMarkIrpPending(Irp);
 
+	CcDeferWrite( FileObject, AFSPostedDeferredWrite, pWorkItem, NULL, BytesToWrite, bRetrying);
+
         ntStatus = STATUS_PENDING;
     }
     __except( AFSExceptionFilter( __FUNCTION__, GetExceptionCode(), GetExceptionInformation()) )