search

code-423n4 / 2021-05-nftx-findings

1 stars 0 forks source link

struct with 1 field #35

Closed code423n4 closed 3 years ago

code423n4 commented 3 years ago

Handle

paulius.eth

Vulnerability details

Impact

No need to use a struct with 1 field here, consumes extra gas: struct EligibilityModule { address impl; } EligibilityModule[] public modules;

Recommended Mitigation Steps

can be refactored to: address[] public modules;