reinvest function should return a uint256 reinvested amount but it returns nothing

[code423n4]

Handle

tabish

Vulnerability details

Impact

Detailed description of the impact of this finding.
reinvesting funds after their lock period is over again in the strategy would not be possible.

Proof of Concept

Provide direct links to all referenced code in GitHub. Add screenshots, logs, or any other relevant proof that illustrates the concept.

link to the function -> https://github.com/code-423n4/2021-09-bvecvx/blob/1d64bd58c7a4224cc330cef283561e90ae6a3cf5/veCVX/contracts/veCVXStrategy.sol#L400

Recommended Mitigation Steps

At the end of the function adding a return toDeposit; would fix this.

[GalloDaSballo]

Highly disagree with risk rating, see #18

We should delete the return values

[ghoul-sol]

duplicate #12 ergo low risk