40,000 USDC
View results
Submission Details
Severity: medium
Valid

Lack of Input Validation - Arbiter Addresses

Summary

lack of validation for whether the arbiter address is the zero address

Vulnerability Details

In the EscrowFactory.newEscrow() and the Escrow.constructor() function, there is a lack of validation for whether the arbiter address is the zero address. Given that the zero address cannot initiate transactions, this could lead to the permanent locking of funds in the escrow contract if either the seller or arbiter is set to the zero address.

Impact

The impact can be significant if high-value transactions are involved. If the arbiter addresses is set to the zero address, funds could be permanently locked in the escrow contract.

Tools Used

Manual Review

Recommendations

Ensure that the arbiter addresses provided to the EscrowFactory.newEscrow() and or Escrow.constructor() function are not the zero address.

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.