Submission Details
Severity:
Use hardcode address instead address(this)
Summary
Use hardcode address instead address(this)
Vulnerability Details
Instead of using address(this), it is more gas-efficient to pre-calculate and use the hardcoded address. Foundry’s script.sol and solmate’s LibRlp.sol contracts can help achieve this.
44 : if (tokenContract.balanceOf(address(this)) < price) revert Escrow__MustDeployWithTokenBalance();
98 : i_tokenContract.safeTransfer(i_seller, i_tokenContract.balanceOf(address(this)));
110: uint256 tokenBalance = i_tokenContract.balanceOf(address(this));
125 : tokenBalance = i_tokenContract.balanceOf(address(this));
References :
https://book.getfoundry.sh/reference/forge-std/compute-create-address
https://twitter.com/transmissions11/status/1518507047943245824
Recommendations
Use hardcoded address.
Prize pool breakdown
Total prize
40,000 USDC
nSLOC
186215 USDC / LOC
37,000 USDC
3,000 USDC
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.