[win-pv-devel] [PATCH] Reorder StorePollLocked and StoreDisable

[Owen Smith <owen.smith@xxxxxxxxxx>]

StoreDisable will close and NULL the event channel, then StorePollLocked
will attempt to call EVTCHN(GetCount...) with the NULLed pointer. This
will lead to a D1 BSOD.

Signed-off-by: Owen Smith <owen.smith@xxxxxxxxxx>
---
 src/xenbus/store.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/xenbus/store.c b/src/xenbus/store.c
index f07e469..cbc442e 100644
--- a/src/xenbus/store.c
+++ b/src/xenbus/store.c
@@ -2611,8 +2611,8 @@ StoreRelease(
 
     XENBUS_SUSPEND(Release, &Context->SuspendInterface);
 
-    StoreDisable(Context);
     (VOID) StorePollLocked(Context);
+    StoreDisable(Context);
     RtlZeroMemory(&Context->Response, sizeof (XENBUS_STORE_RESPONSE));
 
     XENBUS_EVTCHN(Release, &Context->EvtchnInterface);
-- 
2.16.2.windows.1


_______________________________________________
win-pv-devel mailing list
win-pv-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/win-pv-devel