Good practice is to place the nonReentrant modifier first before other modifiers
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
None impact at all. Just following Solidity best practices
Manual Review
Rearrange the modifiers of Escrow.resolveDispute() and place the nonReentrant modifier first
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.