code-423n4 / 2021-04-vader-findings

1 stars 0 forks source link

excludedCount #300

Open code423n4 opened 3 years ago

code423n4 commented 3 years ago

Handle

paulius.eth

Vulnerability details

Impact

Tracking excludedCount separately is consuming more gas and is error-prone as you need to remember to always update it whenever excludedArray changes. Better refactor it to a view that returns excludedArray.length.