Skip to content

Commit

Permalink
Remove NetAdapterStop() call
Browse files Browse the repository at this point in the history
This breaks on Windows 10 when called from EvtCleanupCallback
and looks like it doesn't even required - device got deleted
and verifier doesn't bark.

Bump version to 0.7.3

Signed-off-by: Lev Stipakov <[email protected]>
  • Loading branch information
lstipakov committed May 20, 2022
1 parent 6d32753 commit 8400e9e
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion PropertySheet.props
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
<PropertyGroup Label="UserMacros">
<OVPN_DCO_VERSION_MAJOR>0</OVPN_DCO_VERSION_MAJOR>
<OVPN_DCO_VERSION_MINOR>7</OVPN_DCO_VERSION_MINOR>
<OVPN_DCO_VERSION_PATCH>2</OVPN_DCO_VERSION_PATCH>
<OVPN_DCO_VERSION_PATCH>3</OVPN_DCO_VERSION_PATCH>
</PropertyGroup>
<PropertyGroup />
<ItemDefinitionGroup>
Expand Down
4 changes: 3 additions & 1 deletion adapter.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -119,7 +119,9 @@ OvpnAdapterDestroy(NETADAPTER netAdapter)
device->Adapter = WDF_NO_HANDLE;
ExReleaseSpinLockExclusive(&device->SpinLock, irql);

NetAdapterStop(netAdapter);
// TODO: this breaks on Windows 10 when called from EvtCleanupCallback, is this call even needed?
// NetAdapterStop(netAdapter);

WdfObjectDelete(netAdapter);
}

Expand Down

0 comments on commit 8400e9e

Please sign in to comment.