[release/9.0-staging] [mono] Fix crash while debugging AOT'ed app. #117891
+1
−1
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport of #117627 to release/9.0-staging
/cc @thaystg
Customer Impact
We initially backported this change to 10, but held off because there were no reported issues with .NET 9. That has changed with https://devdiv.visualstudio.com/DevDiv/_workitems/edit/2604939 where we were able to reproduce the issue.
The change fixes a crash in debugger by adding a null check around TLS data before dereferencing a pointer.
Regression
[If yes, specify when the regression was introduced. Provide the PR or commit if known.]
Testing
Manually tested a default iOS SDK application
Risk
Low
IMPORTANT: If this backport is for a servicing release, please verify that:
release/X.0-staging, notrelease/X.0.Package authoring no longer needed in .NET 9
IMPORTANT: Starting with .NET 9, you no longer need to edit a NuGet package's csproj to enable building and bump the version.
Keep in mind that we still need package authoring in .NET 8 and older versions.