-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Fix sbyte overflow in TypeName parsing #107261
Fix sbyte overflow in TypeName parsing #107261
Conversation
internal const sbyte ByRef = -3; | ||
internal const int SZArray = -1; | ||
internal const int Pointer = -2; | ||
internal const int ByRef = -3; |
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.
this change was not necessary, but since the cast to sbyte
was the source of the initial bug, I wanted to remove all usages of sbyte
so we use int
everywhere and don't ever run into similar problem
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.
Thanks
/ba-g the failure is unrelated |
* AssemblyNameInfo fuzzer (#107195) * add initial AssemblyNameInfo Fuzzer * fix the first bug that it has discovered * Fix sbyte overflow in TypeName parsing (#107261) * Add TypeNameFuzzer (#107206) Co-authored-by: Miha Zupan <mihazupan.zupan1@gmail.com> Co-authored-by: Adam Sitnik <adam.sitnik@gmail.com> * [TypeName] Nested types should respect MaxNode count (#106334) * Improve AssemblyNameInfo Fuzzer (#107257) --------- Co-authored-by: Buyaa Namnan <bunamnan@microsoft.com> Co-authored-by: Miha Zupan <mihazupan.zupan1@gmail.com>
discovered by the Fuzzer in #107206 (comment)