-
Notifications
You must be signed in to change notification settings - Fork 47
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
Ensure ClrInstrumentationEngine handles ref fields #464
Comments
See the additional ECMA-335 changes too - dotnet/runtime#63659 |
@wiktork Any updates on this investigation? |
We haven' started it yet. We have talked about this a bit since there is a risk that our IL decoding logic might just crash if it comes across something like this. Hopefully it won't be impactful in the short term because ref fields will be rare in production code for the time being. |
Thanks @delmyers. I agree this is a potential concern. Something to note is that |
@AaronRobinsonMSFT I hope we will still have a bit of time before this becomes a big issue, since someone would have to decode a method that references the type first. There are some scenarios where that is more likely. |
@AaronRobinsonMSFT Is there an easy way for our partners to install and test these new features? I think that the easiest way for us to find issues is to get partners to write tests that use them, so we can work together to fix issues. We, of course, should be adding tests to this codebase, too. |
@delmyers The C# language support doesn't presently exist. I wrote a library in IL that could be used to try out things and sent that to @wiktork. I can share with you as well. The support for |
dotnet/runtime#63768
May be particularly impactful with regards to signature parsing.
The text was updated successfully, but these errors were encountered: