-
Couldn't load subscription status.
- Fork 5.2k
Fix CORDBG_E_MODULE_NOT_LOADED error in func-eval #121057
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR fixes a CORDBG_E_MODULE_NOT_LOADED error in the debugger's function evaluation by ensuring the vmModule field is properly initialized to NULL in TypeHandleToBasicTypeInfo. The fix addresses issue #120848 where uninitialized vmModule pointers could cause errors during debugging operations.
Key changes:
- Added initialization of
res->vmModule.SetRawPtr(NULL)in three code paths within the switch statement
|
Tagging subscribers to this area: @steveisok, @dotnet/dotnet-diag |
|
/backport to release/10.0 |
|
Started backporting to |
|
@tommcdon Was this a long standing issue or was this introduced with a new feature/PR? |
@AaronRobinsonMSFT #118414 exposed a latent issue where we didn't zero init the debugger reponse for func-eval. t's access non-initialized memory and so the value is random. On x86 this seems to happen relatively consistently in the test case we used to investigate. While the issue was found on x86 func-eval testing, the issue could happen on any architecture. |
Correctly initialize vmModule in FuncEvalComplete in the debugger
Fixes #120848