Submission Details
Severity:
Unneccessary changing the state
Summary
Vulnerability Details
msg.sender is allocatorrole which is admin so why we changing the state in each function call if this allocatorRole is always same? NO need of nested mapping.
In Treasury.sol's allocateFunds function:
_allocations[msg.sender][recipient] = amount;
Impact
Unnecessary bytecode and gas is wasted while storing these mappings.
Tools Used
Manual review
Prize pool breakdown
Total prize
77,280 USDC
nSLOC
3,86420
USDC / LOC
74,000
3,280
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.