search

code-423n4 / 2022-01-yield-findings

1 stars 0 forks source link

removeVault function can be external #109

Closed code423n4 closed 2 years ago

code423n4 commented 2 years ago

Handle

sirhashalot

Vulnerability details

Impact

The removeVault() function of ConvexYieldWrapper.sol can be declared external for gas savings

Proof of Concept

Recommended Mitigation Steps

Declare functions as external instead of public when possible for gas savings

devtooligan commented 2 years ago

dup of #12