Confuser.Protections.HoLLy icon indicating copy to clipboard operation
Confuser.Protections.HoLLy copied to clipboard

Fix type marking in MemoryEditInjectPhase

Open holly-hacker opened this issue 7 years ago • 2 comments

Calling name.Analyze on one of the types returned from the type injection results in a null reference exception somewhere in VTable analyzer code.

holly-hacker avatar Nov 27 '17 23:11 holly-hacker

I just realized that you shouldn't be renaming an override methods, which ToString() is. I'll have to add a check and only rename non-overrides.

While I'm at it, I shouldn't be injecting the ToString() method in the first place on non-debug builds, so I'll have to put it in an #if DEBUG directive.

holly-hacker avatar Dec 01 '17 08:12 holly-hacker

Ignoring all virtual & public methods should fix it, but it reveals another issue: ConfuserEx doesn't seem to update the references to renamed fields/properties.

holly-hacker avatar Dec 03 '17 16:12 holly-hacker