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

The `nonReentrant` `modifier` should occur before all other modifiers

Summary

The nonReentrant modifier should occur before all other modifiers

Vulnerability Details

This is a best-practice to protect against reentrancy in other modifiers

Instances (1):

File: src/Escrow.sol
112: function resolveDispute(uint256 buyerAward) external onlyArbiter nonReentrant inState(State.Disputed) {

Link to code - https://github.com/Cyfrin/2023-07-escrow/tree/main/src/Escrow.sol

Tools Used

Custom analyzer tool

Recommendations

Update the code to add nonReentrant modifier before all other modifiers

Support

FAQs

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