code-423n4 / 2021-09-bvecvx-findings

0 stars 0 forks source link

veCVXStrategy: Extra functions can be external instead of public #36

Open code423n4 opened 3 years ago

code423n4 commented 3 years ago

Handle

hickuphh3

Vulnerability details

Impact

The setWithdrawalSafetyCheck(), setHarvestOnRebalance(), setProcessLocksOnReinvest() and setProcessLocksOnRebalance() functions are unused internally but have public visibility. Their visibility can be changed to external.

GalloDaSballo commented 3 years ago

Sure