Submitted by 0xNazgul, also found by Deivitto
Across the VaultController.sol there are many external calls to the AdminProxy.sol via execute(). Looking at the execute() function in AdminProxy.sol:
As one can see it does a call to the target contract with the provided callData. Going back to the VaultController.sol the success of the call is check and reverts if unsuccessful. However, in one instance this check is missed and could cause issues.
Looking at that specific instance:
It is clear that the last call to AdminProxy.sols execute is not checked.
Consider adding a check similar to how it is done in the rest of the contract.
RedVeil (Popcorn) confirmed, but disagreed with severity 
