Skip to content
This repository has been archived by the owner on Feb 24, 2023. It is now read-only.

Commit

Permalink
Merge pull request #37 from tayvano/patch-1
Browse files Browse the repository at this point in the history
add more examples
  • Loading branch information
montyly committed Feb 25, 2020
2 parents e568d39 + f41afd4 commit b150070
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions unprotected_function/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,5 +12,7 @@ Always specify a modifier for functions.

## Examples
- An `onlyOwner` modifier is [defined but not used](Unprotected.sol), allowing anyone to become the `owner`
- [Parity Wallet](https://blog.zeppelin.solutions/on-the-parity-wallet-multisig-hack-405a8c12e8f7). For code, see [initWallet](WalletLibrary_source_code/WalletLibrary.sol)

- April 2016: [Rubixi allows anyone to become owner](https://etherscan.io/address/0xe82719202e5965Cf5D9B6673B7503a3b92DE20be#code)
- July 2017: [Parity Wallet](https://blog.zeppelin.solutions/on-the-parity-wallet-multisig-hack-405a8c12e8f7). For code, see [initWallet](WalletLibrary_source_code/WalletLibrary.sol)
- BitGo Wallet v2 allows anyone to call tryInsertSequenceId. If you try close to MAXINT, no further transactions would be allowed. [Fix: make tryInsertSequenceId private.](https://github.com/BitGo/eth-multisig-v2/commit/8042188f08c879e06f097ae55c140e0aa7baaff8#diff-b498cc6fd64f83803c260abd8de0a8f5)
- Feb 2020: [Nexus Mutual's Oraclize callback was unprotected—allowing anyone to call it.](https://medium.com/nexus-mutual/responsible-vulnerability-disclosure-ece3fe3bcefa) Oraclize triggers a rebalance to occur via Uniswap.

0 comments on commit b150070

Please sign in to comment.