EstimateGas ignores provided nonce and sets correct nonce from state

dominant-strategies · Oct 30, 2024 · 3025495 · 3025495
1 parent 7246e14
commit 3025495
Showing 1 changed file with 5 additions and 7 deletions.
diff --git a/internal/quaiapi/api.go b/internal/quaiapi/api.go
@@ -573,14 +573,12 @@ func DoCall(ctx context.Context, b Backend, args TransactionArgs, blockNrOrHash
 	// this makes sure resources are cleaned up.
 	defer cancel()
 
-	if args.Nonce == nil {
-		internal, err := args.from(b.NodeLocation()).InternalAndQuaiAddress()
-		if err != nil {
-			return nil, err
-		}
-		nonce := state.GetNonce(internal)
-		args.Nonce = (*hexutil.Uint64)(&nonce)
+	internal, err := args.from(b.NodeLocation()).InternalAndQuaiAddress()
+	if err != nil {
+		return nil, err
 	}
+	nonce := state.GetNonce(internal)
+	args.Nonce = (*hexutil.Uint64)(&nonce) // Ignore provided nonce, reset to correct nonce
 
 	// Get a new instance of the EVM.
 	msg, err := args.ToMessage(globalGasCap, header.BaseFee(), b.NodeLocation())