Skip to content

[release/8.0-staging] Remove invalid GCPROTECT for duplicate slot #117030

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

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jun 25, 2025

Backport of #100787 to release/8.0-staging

/cc @elinor-fung @AaronRobinsonMSFT @dotnet/interop-contrib

Customer Impact

  • Customer reported
  • Found internally

This is an intermittent issue that can result in a crash when a GC occurs during marshalling of a struct. This was found via tests for 9.0 and reported by an external customer for 8.0.

Regression

  • Yes
  • No

dotnet/coreclr#26340

Testing

This issue was hit in GC stress tests. It was fixed in 9.0 and has been working since.

Risk

Low. Fix has been in since 9.0.

@teo-tsirpanis teo-tsirpanis added this to the 8.0.x milestone Jun 25, 2025
Copy link
Contributor

Tagging subscribers to this area: @mangod9
See info in area-owners.md if you want to be subscribed.

@elinor-fung elinor-fung added the Servicing-consider Issue for next servicing release review label Jun 25, 2025
Copy link
Member

@jeffschwMSFT jeffschwMSFT left a comment

Choose a reason for hiding this comment

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

lgtm. please get a code review. we will take for consideration in 8.0.x

@rbhanda rbhanda modified the milestones: 8.0.x, 8.0.19 Jun 26, 2025
@rbhanda rbhanda added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Jun 26, 2025
@jeffschwMSFT jeffschwMSFT merged commit 7fea3b3 into release/8.0-staging Jun 27, 2025
123 of 130 checks passed
@jkotas jkotas deleted the backport/pr-100787-to-release/8.0-staging branch July 20, 2025 00:07
This was referenced Aug 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-VM-coreclr Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants