Skip to content

release/6.0.1: Preserve last error in call-counting stubs - #60887

Merged
Anipik merged 1 commit into
release/6.0from
backport/pr-60886-to-release/6.0
Nov 11, 2021
Merged

release/6.0.1: Preserve last error in call-counting stubs#60887
Anipik merged 1 commit into
release/6.0from
backport/pr-60886-to-release/6.0

Conversation

@github-actions

@github-actionsgithub-actionsBot commented Oct 26, 2021

Copy link
Copy Markdown
Contributor

Backport of #60886 to release/6.0

/cc @jkotas

Customer Impact

Intermittent corruption of error code returned by Marshal.GetLastSystemError/GetLastPInvokeError APIs . This bug can manifest itself as a wrong exception being thrown, or as a bogus exception being thrown when PInvoke succeed. The bug manifested itself at least 3 times in dotnet/runtime CI in last month.

Testing

No reliable repro, fix verified by stepping through the code in debugger.

Risk

Low.

@ghostghost added the area-VM-coreclr label Oct 26, 2021
@jkotasjkotas changed the title [release/6.0] Preserve last error in call-counting stubsrelease/6.0.1: Preserve last error in call-counting stubsOct 26, 2021
@jeffschwMSFTjeffschwMSFT added the Servicing-consider Issue for next servicing release review label Oct 26, 2021
@jeffschwMSFTjeffschwMSFT modified the milestones: 6.0.1, 6.0.0Oct 26, 2021

@jeffschwMSFTjeffschwMSFT left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. We will take for consideration in .NET 6 servicing

@leecowleecow added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Oct 28, 2021
@leecowleecow modified the milestones: 6.0.0, 6.0.1Oct 28, 2021
@leecowleecow added Servicing-consider Issue for next servicing release review Servicing-approved Approved for servicing release and removed Servicing-approved Approved for servicing release Servicing-consider Issue for next servicing release review labels Oct 28, 2021
@Anipik

Copy link
Copy Markdown
Contributor

/azp run runtime, runtime-dev-innerloop, dotnet-linker-test

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 2 pipeline(s).

@Anipik

Copy link
Copy Markdown
Contributor

/azp run runtime, runtime-dev-innerloop, dotnet-linker-tests

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 3 pipeline(s).

@Anipik
Anipik merged commit 9b58e44 into release/6.0Nov 11, 2021
@Anipik
Anipik deleted the backport/pr-60886-to-release/6.0 branch November 11, 2021 23:36
@ghostghost locked as resolved and limited conversation to collaborators Dec 12, 2021
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-VM-coreclrServicing-approvedApproved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@Anipik@jeffschwMSFT@kouvel@leecow@jkotas