Skip to content

Classtest#217

Open
Froshboss wants to merge 3 commits into
BlockheaderWeb3-Community:mainfrom
Froshboss:classtest
Open

Classtest#217
Froshboss wants to merge 3 commits into
BlockheaderWeb3-Community:mainfrom
Froshboss:classtest

Conversation

@Froshboss
Copy link
Copy Markdown
Collaborator

Summary

An attack contract that exploits this vulnerability
Demonstrate the exploit using a test
Fix the contract using:

  • Checks-Effects-Interactions pattern OR
  • Reentrancy guard

Write tests proving:

  • Attack fails after fix
  • Legit users can still withdraw

Testing

  • Unit tests added
  • Tests passing locally

Notes

Any assumptions or trade-offs?

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