Core Contracts
Regnum Aurum Acquisition Corp
HardhatReal World AssetsNFT
77,280 USDC
View results
Previous Next
Submission Details
Severity: low
Invalid

G-1 Gas Optimization in FeeCollector::collectedFees

radcipher

Summary

Reading from collectedFees 8 times could increase gas costs

function _calculateTotalFees() internal view returns (uint256) {
return collectedFees.protocolFees +
collectedFees.lendingFees +
collectedFees.performanceFees +
collectedFees.insuranceFees +
collectedFees.mintRedeemFees +
collectedFees.vaultFees +
collectedFees.swapTaxes +
collectedFees.nftRoyalties;
}

Recommendations

Optimize for fewer storage reads and faster execution

function _calculateTotalFees() internal view returns (uint256 total) {
CollectedFees memory fees = collectedFees; // Load into memory once
total = fees.protocolFees +
fees.lendingFees +
fees.performanceFees +
fees.insuranceFees +
fees.mintRedeemFees +
fees.vaultFees +
fees.swapTaxes +
fees.nftRoyalties;
}
Updates

Lead Judging Commences

inallhonesty Lead Judge 3 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
inallhonesty Lead Judge 3 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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