Submission Details
Severity:
Missing checks in constructor/initialize
Summary
Missing checks in constructor/initialize
Vulnerability Details
There are some missing checks in these functions, and this could lead to unexpected scenarios. Consider always adding a sanity check for state variables.
File: /src/Escrow.sol
//@audit `arbiter` is not checked
32: constructor(
uint256 price,
IERC20 tokenContract,
address buyer,
address seller,
address arbiter,
uint256 arbiterFee
) {
Impact
Tools Used
Manual
Recommendations
check the value of arbiter variable before assigning it
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.