Submission Details
Severity:
Single-Use Modifier Implementation
Description: The nonReentrant modifier is only used once in the contract, which may not justify the gas cost of implementing it as a modifier.
modifier nonReentrant() {
Impact:
Increased deployment gas costs
Code complexity without proportional benefit
Reduced code maintainability
Recommended Mitigation: Either:
Incorporate the modifier logic directly into the function:
function refund() external beforeDeadline {
require(!locked, "No re-entrancy");
locked = true;
address payable _to = payable(msg.sender);
_refundERC20(_to);
_refundETH(_to);
emit Refunded(msg.sender);
locked = false;
}
Or justify modifier usage by implementing it in multiple functions where reentrancy protection is needed.
Rewards breakdown
nSLOC
129This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
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.