Impact is low/informational.
The inconsistency in validation checks between similar functions can lead to unexpected behavior.
Take a look at updateAlpha():
In the updateAlpha function, there is a check _checkAlphaArguments(_alphaNumerator, _alphaDenominator) which ensures the correctness of the arguments provided. This establishes a certain standard of validation for updating alpha values.
However, in the updateMultipliers function, there is no equivalent validation function like _checkAlphaArguments. This discrepancy implies that the two functions, which are logically do not maintain a consistent standard of input validation.
Consider introducing a validation function (e.g., _checkMultipliersArguments) that will validate the parameters supplyMultiplier and borrowMultiplier.
A diff for the recommended changes might appear as:
