Review feedback: add !isTesSuccess(ter)

Author: Bronek

src/xrpld/app/tx/detail/VaultClawback.cpp

@@ -255,7 +255,8 @@ VaultClawback::doApply()
             vaultAccount,
             sharesDestroyed,
             j_,
-            WaiveTransferFee::Yes))
+            WaiveTransferFee::Yes);
+        !isTesSuccess(ter))
         return ter;
 
     // Try to remove MPToken for shares, if the holder balance is zero. Vault
@@ -280,7 +281,8 @@ VaultClawback::doApply()
             account_,
             assetsRecovered,
             j_,
-            WaiveTransferFee::Yes))
+            WaiveTransferFee::Yes);
+        !isTesSuccess(ter))
         return ter;
 
     // Sanity check

src/xrpld/app/tx/detail/VaultDeposit.cpp

@@ -292,7 +292,8 @@ VaultDeposit::doApply()
             vaultAccount,
             assetsDeposited,
             j_,
-            WaiveTransferFee::Yes))
+            WaiveTransferFee::Yes);
+        !isTesSuccess(ter))
         return ter;
 
     // Sanity check
@@ -317,7 +318,8 @@ VaultDeposit::doApply()
             account_,
             sharesCreated,
             j_,
-            WaiveTransferFee::Yes))
+            WaiveTransferFee::Yes);
+        !isTesSuccess(ter))
         return ter;
 
     return tesSUCCESS;

src/xrpld/app/tx/detail/VaultWithdraw.cpp

@@ -270,7 +270,8 @@ VaultWithdraw::doApply()
             vaultAccount,
             sharesRedeemed,
             j_,
-            WaiveTransferFee::Yes))
+            WaiveTransferFee::Yes);
+        !isTesSuccess(ter))
         return ter;
 
     // Try to remove MPToken for shares, if the account balance is zero. Vault
@@ -301,7 +302,8 @@ VaultWithdraw::doApply()
             dstAcct,
             assetsWithdrawn,
             j_,
-            WaiveTransferFee::Yes))
+            WaiveTransferFee::Yes);
+        !isTesSuccess(ter))
         return ter;
 
     // Sanity check