The Wayback Machine - https://web.archive.org/web/20210601225620/https://github.com/dotnet/aspnetcore/pull/33207
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

Less static Sockets #33207

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Less static Sockets #33207

wants to merge 1 commit into from

Conversation

@halter73
Copy link
Member

@halter73 halter73 commented Jun 1, 2021

This way we don't risk _fileHandleSocket getting finalized while the derived socket is still in use just because the member data was re-enumerated.

Addresses #33206.

@halter73 halter73 requested review from BrennanConroy and wtgodbe Jun 1, 2021
@halter73 halter73 requested a review from Tratcher as a code owner Jun 1, 2021
@msftbot msftbot bot added the area-runtime label Jun 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant