code-423n4 / 2021-10-ambire-findings

0 stars 0 forks source link

Unused imports #59

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

Handle

pauliax

Vulnerability details

Impact

Unused imports:

in Zapper import "../interfaces/IUniV3SwapRouter.sol";

in Identity: import "./libs/SignatureValidatorV2.sol";

Recommended Mitigation Steps

Remove these unused imports to save some deployment gas.

Ivshti commented 2 years ago

those imports are absolutely used, the code cannot compile w/o them

GalloDaSballo commented 2 years ago

Not sure why the warden flagged these up

Agree with the sponsor, the imports are used