fix: use 0 as max token amount if balance is not enough for gas

MetaMask · May 8, 2024 · 05d26b8 · 05d26b8
1 parent e345fed
commit 05d26b8
Showing 1 changed file with 9 additions and 4 deletions.
diff --git a/ui/ducks/send/send.js b/ui/ducks/send/send.js
@@ -1065,9 +1065,14 @@ const slice = createSlice({
           16,
         ).add(new Numeric(state.gasTotalForLayer1 ?? '0x0', 16));
 
-        amount = new Numeric(draftTransaction.sendAsset.balance, 16)
-          .minus(_gasTotal)
-          .toString();
+        amount = new Numeric(
+          draftTransaction.sendAsset.balance,
+          16,
+        ).lessThanOrEqualTo(_gasTotal)
+          ? '0'
+          : new Numeric(draftTransaction.sendAsset.balance, 16)
+              .minus(_gasTotal)
+              .toString();
       }
       slice.caseReducers.updateSendAmount(state, {
         payload: amount,
@@ -2743,7 +2748,7 @@ export function signTransaction() {
         from: txParams.from,
         to: txParams.to,
         value: txParams.value,
-        gas: unapprovedTx.userEditedGasLimit
+        gas: unapprovedTx?.userEditedGasLimit
           ? unapprovedTx.txParams.gas
           : txParams.gas,
       };