Skip to content

fix: address findings 3 and 7#1

Merged
philbow61 merged 1 commit into
masterfrom
fix/findings-3-7
Mar 16, 2026
Merged

fix: address findings 3 and 7#1
philbow61 merged 1 commit into
masterfrom
fix/findings-3-7

Conversation

@asker61
Copy link
Copy Markdown
Contributor

@asker61 asker61 commented Mar 16, 2026

Summary

  • make .env.example use a non-zero RECOVERER placeholder with a clear replacement note
  • reject zero-address recipients in recoverETH and recoverERC20
  • reject zero-target and meaningless no-code execute calls while preserving ETH sends to EOAs
  • add coverage for the new RecoveryContract guardrails

Test plan

  • set -a && source .env.example && set +a && forge test

@philbow61 philbow61 merged commit 2113b71 into master Mar 16, 2026
2 checks passed
@philbow61 philbow61 deleted the fix/findings-3-7 branch March 16, 2026 13:38
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.

2 participants