diff options
| author | houha2 <[email protected]> | 2021-01-22 12:16:49 -0800 |
|---|---|---|
| committer | GitHub <[email protected]> | 2021-01-22 12:16:49 -0800 |
| commit | f9d9c5035a7ddfee600763965d412be5bf6b8059 (patch) | |
| tree | e7c9c65d7184fa5654dfd610816a52546d12f71d /audio/simpleaudiosample/Source/Utilities/ToneGenerator.cpp | |
| parent | ad7d35781451b0718072593408d0d392371952aa (diff) | |
Improvements to WFP Inspect Sample (#516)
Changes a `NT_ASSERT` to match what the comment above is saying, the previous version caused a system crash if the packet was not inbound and had a auth decision made already.
The comment
` // Skip pended connections in the list, for which the auth decision is already taken.`
` // They should not be for inbound connections.`
to my understanding means inbound connections should be ignored and ignore connections for which the auth decision was already made. The latter is checked by:
` if (packet->authConnectDecision == 0)`
which keeps `found` equal to `FALSE` triggering the packet and list entry to be set to `NULL` here:
`if (!found)
{
listEntry = NULL;
packet = NULL;
}`
Change also updates the .sln and .vxproj files so the sample compiles on VS2019 without any changes - addressing #366
Co-authored-by: Elaine Houha <[email protected]>
Diffstat (limited to 'audio/simpleaudiosample/Source/Utilities/ToneGenerator.cpp')
0 files changed, 0 insertions, 0 deletions
