Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

msquic crashing on Windows (built from current main) #3484

Closed
1 of 4 tasks
wfurt opened this issue Mar 7, 2023 · 1 comment
Closed
1 of 4 tasks

msquic crashing on Windows (built from current main) #3484

wfurt opened this issue Mar 7, 2023 · 1 comment
Labels
Bug: Platform A code bug in platform/TLS specific code. OS: Windows (User) Partner: .NET By or For the .NET team
Milestone

Comments

@wfurt
Copy link
Member

wfurt commented Mar 7, 2023

Describe the bug

I recently submit change to use msquic main as our base for .NET test runs.
We start seeing occasional crashes on Windows platform.

quic

Affected OS

  • Windows
  • Linux
  • macOS
  • Other (specify below)

Additional OS information

Windos 11 in CI

MsQuic version

main

Steps taken to reproduce bug

no clear repro. I tried to run our test suite locally on Windows and I did not get crash.
It may be timing. IT seems like this is overlapped completion.

Expected behavior

msquic should not crash

Actual outcome

crash on AV.

Additional details

https://dev.azure.com/dnceng-public/public/_build/results?buildId=194019&view=ms.vss-test-web.build-test-results-tab&runId=3682560&resultId=209584&paneView=dotnet-dnceng.dnceng-anon-build-release-tasks.helix-anon-test-information-tab

has link to core dump. I tried to peek at the SocketProc but it is optimized out. Somebody with better Windows debugging skills may have better luck. My guess is that the SockProc->Parent is null.

I can also share the core dump if easier.

cc: @ManickaP @CarnaViire

@nibanks
Copy link
Member

nibanks commented Mar 13, 2023

Fixed by #3489.

@nibanks nibanks closed this as completed Mar 13, 2023
@nibanks nibanks added the Partner: .NET By or For the .NET team label Mar 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug: Platform A code bug in platform/TLS specific code. OS: Windows (User) Partner: .NET By or For the .NET team
Projects
None yet
Development

No branches or pull requests

2 participants