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

nonReentrant modifier should be used first before other modifiers

Summary

Good practice is to place the nonReentrant modifier first before other modifiers

Vulnerability Details

In Escrow.resolveDispute() the nonReentrant modifier is placed second after onlyArbiter(). In order for the nonReentrant modifier to check if the _status is not _ENTERED before any code is executed in the function , It should be placed first

Impact

None impact at all. Just following Solidity best practices

Tools Used

Manual Review

Recommendations

Rearrange the modifiers of Escrow.resolveDispute() and place the nonReentrant modifier first

Support

FAQs

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