Skip to content

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Aug 26, 2025

Backport of #110379 to release/9.0-staging

/cc @steveisok @mikem8361

Customer Impact

  • Customer reported
  • Found internally

As reported in #118049, when running an application on Linux as PID=1 (the primary process), the process can hang when there's an unhandled exception with crash dumps enabled. The primary process ends up sending a different signal (SIGSEGV) during abort() where we fail to terminate the process and end up spinning indefinitely.

The fix is to break out of PROCCreateCrashDump if we detect the process re-enters the function.

This was originally fixed in #110379, but did not get backported to 9.

Regression

  • Yes
  • No

Testing

Manual - triggering an unhandled exception as the primary process.

Risk

Low

IMPORTANT: If this backport is for a servicing release, please verify that:

  • The PR target branch is release/X.0-staging, not release/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.

@steveisok steveisok added the Servicing-consider Issue for next servicing release review label Sep 3, 2025
@steveisok steveisok changed the title [release/9.0-staging] Fix recursion issue found in issue #109779 [release/9.0-staging] Fix recursion issue found in PROCCreateCrashDump Sep 3, 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 9.0.x

Copy link
Contributor

Tagging subscribers to this area: @steveisok, @dotnet/dotnet-diag
See info in area-owners.md if you want to be subscribed.

@steveisok steveisok requested a review from tommcdon September 3, 2025 17:21
@rbhanda rbhanda modified the milestones: 9.0.x, 9.0.10 Sep 9, 2025
@rbhanda rbhanda added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Sep 9, 2025
@hoyosjs hoyosjs merged commit e357192 into release/9.0-staging Sep 15, 2025
98 of 102 checks passed
@hoyosjs hoyosjs deleted the backport/pr-110379-to-release/9.0-staging branch September 15, 2025 20:39
This was referenced Oct 16, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Oct 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants