Summary

In StakingPool, _totalFeesBasisPoints() is required to be `<= 4000` whereas in VaultControllerStrategy it is required to be `<= 3000`. That's an inconsistency that has to be fixed.

Impact

There can be a max difference of 10% between the StakingPool cotnract and VaultControllerStrategy

Tools Used

Manual analysis

Recommendations

Choose a unique fee threshold and stick to it across the whole codebase