If there are funds remaining in an old strategy, there is only 1 way to claim those funds which is through Sherlock.updateYieldStrategy() . It is quite an inconvenience to do this.
Recommended Mitigation Steps
Create an additional function to allow anyone to call withdraw / withdraw all from an old strategy. The withdrawn funds should be sent to SherlockCore e.g.
function oldStrategyWithdrawAll(IStrategyManager _oldStrategy) external override {
_oldStrategy.withdrawAll();
}
CloudEllie
commented
2 years ago
Handle
GreyArt
Vulnerability details
Impact
If there are funds remaining in an old strategy, there is only 1 way to claim those funds which is through
Sherlock.updateYieldStrategy(). It is quite an inconvenience to do this.Recommended Mitigation Steps
Create an additional function to allow anyone to call withdraw / withdraw all from an old strategy. The withdrawn funds should be sent to SherlockCore e.g.