diff options
author | Simon Rozman <simon@rozman.si> | 2019-06-12 22:10:37 +0200 |
---|---|---|
committer | Simon Rozman <simon@rozman.si> | 2019-06-20 11:54:57 +0200 |
commit | 70fcfdc85d2480e7f2dea4fce364a19548e0f12f (patch) | |
tree | fc69adfc5741dc4a911b7882fdd930476396a12a /wintun.c | |
parent | Revise adapter state checking (diff) | |
download | wintun-70fcfdc85d2480e7f2dea4fce364a19548e0f12f.tar.xz wintun-70fcfdc85d2480e7f2dea4fce364a19548e0f12f.zip |
Add missing state transition lock
When we are transitioning to a state that suspends some operations, we
must get an exclusive transition lock to wait for all operations that
have a shared transition lock. And TunHaltEx() is no exception.
Signed-off-by: Simon Rozman <simon@rozman.si>
Diffstat (limited to '')
-rw-r--r-- | wintun.c | 2 |
1 files changed, 2 insertions, 0 deletions
@@ -1332,7 +1332,9 @@ static void TunHaltEx(NDIS_HANDLE MiniportAdapterContext, NDIS_HALT_ACTION HaltA ASSERT(!ctx->PnPNotifications.Handle); ASSERT(!InterlockedGet64(&ctx->ActiveTransactionCount)); /* Adapter should not be halted if it wasn't fully paused first. */ + KIRQL irql = ExAcquireSpinLockExclusive(&ctx->TransitionLock); InterlockedAnd(&ctx->Flags, ~TUN_FLAGS_PRESENT); + ExReleaseSpinLockExclusive(&ctx->TransitionLock, irql); for (IRP *pending_irp; (pending_irp = IoCsqRemoveNextIrp(&ctx->Device.ReadQueue.Csq, NULL)) != NULL;) TunCompleteRequest(ctx, pending_irp, STATUS_FILE_FORCED_CLOSED, IO_NO_INCREMENT); |