Submission Details
Severity:
For Loops Reading from storage results in high gas cost for owner
Summary
The for loops in MartenitsaVoting::announceWinner and MartenitsaEvent::stopEvent
read from storage _tokenIds.length and participants.length respectively which consumes a lot of gas
Impact
The owner has to pay a lot of gasfee when he calls these functions
Recommendations
Instead of Reading from storage, creating Local Variables for array Lengths and using those Local Variables in the for loops will mitigate this issue.
Rewards breakdown
nSLOC
239This 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.