Skip to content

Fix: reduce AV false positives in runtime stubs#569

Open
mcpolo99 wants to merge 1 commit into
mkaring:masterfrom
mcpolo99:fix/av-improvements
Open

Fix: reduce AV false positives in runtime stubs#569
mcpolo99 wants to merge 1 commit into
mkaring:masterfrom
mcpolo99:fix/av-improvements

Conversation

@mcpolo99
Copy link
Copy Markdown

Windows Defender flags obfuscated output as malware due to heuristic pattern matching against the combined runtime stub patterns.

Changes:

  • Remove GCHandle.Alloc(Pinned) from Compressor runtime stubs — pinning is unnecessary since the byte[] is locally referenced and won't be GC'd
  • Remove System.Runtime.InteropServices dependency from runtime stubs
  • Add instruction count limit (150) to HardeningPhase inlining to prevent aggregating all suspicious patterns into a single .cctor method body

All protection features remain fully functional. The GCHandle removal changes the memory management approach without affecting the decryption logic. The hardening limit keeps large protection initializers as separate method calls instead of inlining them, which spreads heuristic patterns across multiple methods.

Windows Defender flags obfuscated output as malware due to heuristic
pattern matching against the combined runtime stub patterns.

Changes:
- Remove GCHandle.Alloc(Pinned) from Compressor runtime stubs — pinning
  is unnecessary since the byte[] is locally referenced and won't be GC'd
- Remove System.Runtime.InteropServices dependency from runtime stubs
- Add instruction count limit (150) to HardeningPhase inlining to prevent
  aggregating all suspicious patterns into a single .cctor method body

All protection features remain fully functional. The GCHandle removal
changes the memory management approach without affecting the decryption
logic. The hardening limit keeps large protection initializers as separate
method calls instead of inlining them, which spreads heuristic patterns
across multiple methods.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant