theredguild / damn-vulnerable-defi

The smart contract security training ground for developers, security researchers and educators.

Home Page:https://damnvulnerabledefi.xyz

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

[Wallet Mining] Changing DEPOSIT_ADDRESS to the correct checksummed address?

nzmpi opened this issue · comments

commented

Maybe it will be better to change DEPOSIT_ADDRESS

https://github.com/tinchoabbate/damn-vulnerable-defi/blob/a5d47759c2132175ed8b5b42a6ba28c1e436032d/test/wallet-mining/wallet-mining.challenge.js#L9

to the correct checksummed address: const DEPOSIT_ADDRESS = '0x9B6fb606A9f5789444c17768c6dFCF2f83563801';?
Otherwise if (proxyAddress === DEPOSIT_ADDRESS) {...} doesn't work, because Solidity returns proxyAddress as a checksummed address.