Skip to content

Conversation

@janvorli
Copy link
Member

@janvorli janvorli commented Oct 3, 2025

There is a case where an interpreted code is called by CallDescrWorker and the code that called the CallDescrWorker was invoked via pinvoke. In that case, the SfiNextWorker was not detecting that the exception propagates to CallDescrWorker due to the fact that the final StackFrameIterator::GetFrameState() was returning SFITER_FRAME_FUNCTION for the next pinvoke and not reporting any intermediate SFITER_NATIVE_MARKER_FRAME.

There is a case where an interpreted code is called by CallDescrWorker
and the code that called the CallDescrWorker was invoked via pinvoke. In
that case, the SfiNextWorker was not detecting that the exception
propagates to CallDescrWorker due to the fact that the final
StackFrameIterator::GetFrameState() was returning SFITER_FRAME_FUNCTION
for the next pinvoke and not reporting any intermediate
SFITER_NATIVE_MARKER_FRAME.
@janvorli janvorli added this to the 11.0.0 milestone Oct 3, 2025
@janvorli janvorli requested a review from davidwrighton October 3, 2025 19:12
@janvorli janvorli self-assigned this Oct 3, 2025
Copilot AI review requested due to automatic review settings October 3, 2025 19:12
@dotnet-policy-service
Copy link
Contributor

Tagging subscribers to this area: @BrzVlad, @janvorli, @kg
See info in area-owners.md if you want to be subscribed.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@janvorli janvorli merged commit 5811908 into dotnet:main Oct 7, 2025
98 checks passed
@janvorli janvorli deleted the fix-eh-transition-from-native-to-interpretd branch October 7, 2025 00:58
@github-actions github-actions bot locked and limited conversation to collaborators Nov 6, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants