ILSpy

.NET Decompiler with support for PDB generation, ReadyToRun, Metadata (&more) - cross-platform!

c-sharp dotnet mono decompile decompiler decompiler-engine dotnetcore ilspy pdb unity

Siegfried Pammer 15719eedd6 Reject negative char index in the length-and-char string switch MatchSwitchOnCharBlock's case 2 and default paths guarded against a negative character index, but case 1 (a bare switch on get_Chars) did not. Crafted IL whose get_Chars/get_Item index is negative - a value no compiler emits, but valid IL - therefore reached the pattern reconstruction unchecked. For a length-1 group this silently miscompiled the switch (it rebuilds the string switch from the char labels without using the index), turning IL that reads s[-1] into `switch (s)`; for longer strings it threw IndexOutOfRangeException and aborted the method. Move the check into MatchGetChars so all three call sites reject a negative index by construction, and drop the two now-redundant guards. Same class of unvalidated-integer robustness issue as #3878, in a different switch-on-string pattern. Assisted-by: Claude:claude-opus-4-8:Claude Code		2 months ago
..
Correctness	Merge pull request #3828 from sailro/fix-unchecked-constant-overflow	2 months ago
Disassembler/Pretty	dev: Strip BOM mark from text files	1 year ago
ILPretty	Reject negative char index in the length-and-char string switch	2 months ago
IsolatedDecompilation	Fix #3774 : keep field initializers when decompiling a static ctor alone	3 months ago
PdbGen	Emit member-initializer sequence points in every constructor	3 months ago
Pretty	Add Issue3877 test and handle negative dict capacity	2 months ago
Ugly	Remove IL file from ugly tests	5 months ago
VBPretty	Add CompilerOptions.UseRoslyn4_14_0	5 months ago