Submission Details
Severity:
Empty revert() statements
Summary
In the FjordPoints contract, there is an empty revert() statement without providing reason strings or custom errors in Solidity. Using reason strings or custom errors in revert() statements helps in debugging and understanding why a transaction was reverted, thereby improving the overall readability and maintainability of the code.
File: FjordPoints.sol
186: revert();
Tools Used
Aderyn
Recommendations
+ error NoPoints();
function setPointsPerEpoch(uint256 _points) external onlyOwner checkDistribution {
if (_points == 0) {
- revert();
+ revert NoPoints();
}
pointsPerEpoch = _points;
}
Prize pool breakdown
Total prize
20,000 USDC
nSLOC
66230 USDC / LOC
16,000 USDC
2,500 USDC
1,500 USDC
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.