Skip to content
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

[mono] Ensure exception_message points to a string #66991

Merged
merged 1 commit into from
Mar 24, 2022

Conversation

stefan-sf-ibm
Copy link
Contributor

If obj is an exception exception->message might be NULL and therefore render mono_string_chars_internal (exception->message) into a NULL deref. Also ensure that exception_message is a valid string in case the else branch is taken.

@ghost ghost added the community-contribution Indicates that the PR has been added by a community member label Mar 22, 2022
@dotnet-issue-labeler dotnet-issue-labeler bot added area-VM-meta-mono and removed community-contribution Indicates that the PR has been added by a community member labels Mar 22, 2022
@lambdageek lambdageek merged commit 0e35b8d into dotnet:main Mar 24, 2022
@stefan-sf-ibm stefan-sf-ibm deleted the fix-exception-message branch March 25, 2022 07:26
radekdoulik pushed a commit to radekdoulik/runtime that referenced this pull request Mar 30, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Apr 24, 2022
@lateralusX
Copy link
Member

Looks like this has been hit on iOS (device), #71492. I think we should consider this PR as a backport for .net6.

@dotnet dotnet unlocked this conversation Jul 12, 2022
@lateralusX
Copy link
Member

/backport to release/6.0

@github-actions
Copy link
Contributor

Started backporting to release/6.0: https://github.com/dotnet/runtime/actions/runs/2655223657

@github-actions
Copy link
Contributor

@lateralusX an error occurred while backporting to release/6.0, please check the run log for details!

Error: @lateralusX is not a repo collaborator, backporting is not allowed. If you're a collaborator please make sure your Microsoft team membership visibility is set to Public on https://github.com/orgs/microsoft/people?query=lateralusX

@github-actions
Copy link
Contributor

Started backporting to release/6.0: https://github.com/dotnet/runtime/actions/runs/2655223657

@dotnet dotnet locked as resolved and limited conversation to collaborators Jul 12, 2022
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.

3 participants