Skip to content

Commit

Permalink
chore: lint
Browse files Browse the repository at this point in the history
  • Loading branch information
@leovct
leovct committed Sep 24, 2024
1 parent f4008a2 commit cf8a712
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
14 changes: 7 additions & 7 deletions doc/EthernautCTF.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -20,15 +20,15 @@
| 16 | [Preservation](../src/EthernautCTF/Preservation.sol) | ❌ | | |
| 17 | [Recovery](../src/EthernautCTF/Recovery.sol) | ❌ | | |
| 18 | [MagicNumber](../src/EthernautCTF/MagicNumber.sol) | ❌ | | |
| 19 | AlienCode | ❌ | | |
| 20 | Denial | ❌ | | |
| 19 | AlienCode | ❌ | | |
| 20 | Denial | ❌ | | |
| 21 | [Shop](../src/EthernautCTF/Shop.sol) | ❌ | | |
| 22 | Dex | ❌ | | |
| 23 | DexTwo | ❌ | | |
| 22 | Dex | ❌ | | |
| 23 | DexTwo | ❌ | | |
| 24 | PuzzleWallet | ❌ | | |
| 25 | Motorbike | ❌ | | |
| 26 | DoubleEntry | ❌ | | |
| 27 | GoodSamaritan | ❌ | | |
| 25 | Motorbike | ❌ | | |
| 26 | DoubleEntry | ❌ | | |
| 27 | GoodSamaritan | ❌ | | |
| 28 | [GatekeeperThree](../src/EthernautCTF/GatekeeperThree.sol) | ❌ | | |
| 29 | [Switch](../src/EthernautCTF/Switch.sol) | ❌ | | |
| 30 | [HigherOrder](../src/EthernautCTF/HigherOrder.sol) | ❌ | | |
Expand Down
2 changes: 1 addition & 1 deletion doc/QuillCTF.md
View file
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
| 03 | [VIPBank](../src/QuillCTF/VIPBank.sol) | ✅ | [VIPBankExploit](../test/QuillCTF/VIPBankExploit.t.sol) | - Wrong check of parameter to prevent users from withdrawing too many ethers at a time which leads to funds locked forever in the contract. |
| 04 | [SafeNFT](../src/QuillCTF/SafeNFT.sol) | ✅ | [SafeNFTExploit](../test/QuillCTF/SafeNFTExploit.t.sol) | - OpenZeppelin's ERC721 implementation of `safeMint` is not safe and performs an external call to the receiver address. |
| 05 | [Delegate](../src/QuillCTF/Delegate.sol) | ✅ | [DelegateExploit](../test/QuillCTF/SafeNFTExploit.t.sol) | - `delegatecall` can override variables of the calling contract. |
| 06 | [CollatzPuzzle](../src/QuillCTF/CollatzPuzzle.sol) | ✅ | [CollatzPuzzleExploit](../test/QuillCTF/CollatzPuzzleExploit.t.sol) | - Use huff to heavily optimize the contract by relying on opcodes directly. |
| 06 | [CollatzPuzzle](../src/QuillCTF/CollatzPuzzle.sol) | ❌ | [CollatzPuzzleExploit](../test/QuillCTF/CollatzPuzzleExploit.t.sol) | - Use huff to heavily optimize the contract by relying on opcodes directly. |
| 07 | [TrueXOR](../src/QuillCTF/TrueXOR.sol) | ✅ | [TrueXORExploit](../test/QuillCTF/TrueXORExploit.t.sol) | - Use the amount of gas left (with `gasleft()`) to return different values using a view function that takes no parameters<br>- Use `delegatecall` to keep the context of the main contract (especially, `msg.sender`, `msg.value` and `address(this)`). |
| 20 | [VoteToken](../src/QuillCTF/VoteToken.sol) | ✅ | [VoteTokenExploit](../test/QuillCTF/VoteTokenExploit.t.sol) (\*) | - The contract doesn't update its state when users transfer tokens |
| 21 | [PrivateClub](../src/QuillCTF/PrivateClub.sol) | ✅ | [PrivateClubExploit](../test/QuillCTF/PrivateClubExploit.t.sol) (\*) | - Not enough check of the parameters (e.g. `becomeMember`)<br>- Too much power given to the contract owner |
Expand Down

0 comments on commit cf8a712

Please sign in to comment.