From 538f763fdc304329d6231e6cb31efab4cba5153e Mon Sep 17 00:00:00 2001
From: Piotr Dyraga <piotr.dyraga@keep.network>
Date: Mon, 21 Oct 2019 10:13:41 +0200
Subject: [PATCH] accounts/abi/bind: take into account gas price during gas
estimation (#20189)
The gas price was not passed to the `EstimateGas` function. As a result,
conditional execution paths depending on `tx.gasprice` could be not
correctly processed and we could get invalid gas estimates for contract
function calls.
---
accounts/abi/bind/base.go | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/accounts/abi/bind/base.go b/accounts/abi/bind/base.go
index f74a0af21..499b4bda0 100644
--- a/accounts/abi/bind/base.go
+++ b/accounts/abi/bind/base.go
@@ -218,7 +218,7 @@ func (c *BoundContract) transact(opts *TransactOpts, contract *common.Address, i
}
}
// If the contract surely has code (or code is not needed), estimate the transaction
- msg := ethereum.CallMsg{From: opts.From, To: contract, Value: value, Data: input}
+ msg := ethereum.CallMsg{From: opts.From, To: contract, GasPrice: gasPrice, Value: value, Data: input}
gasLimit, err = c.transactor.EstimateGas(ensureContext(opts.Context), msg)
if err != nil {
return nil, fmt.Errorf("failed to estimate gas needed: %v", err)
--
GitLab