Summary
VaultV2::submit() overflows here when the timelock of the selector has been set to type(uint256).max, which works as intended but could be more verbose.
Vulnerability Detail
VaultV2::abdicateSubmit() sets the timelock to the maximum, making the submit() function overflow to disable function calls with this selector.
function abdicateSubmit(bytes4 selector) external {
timelocked();
timelock[selector] = type(uint256).max;
emit EventsLib.AbdicateSubmit(selector);
}Impact
Error handling is not very verbose
Code Snippet
Tool Used
Manual Review
Recommendation
Add an error message.