diff options
| author | Bojie Wu <bojie@dexon.org> | 2018-10-09 13:28:45 +0800 |
|---|---|---|
| committer | Wei-Ning Huang <w@dexon.org> | 2019-04-09 21:32:51 +0800 |
| commit | f5fa30b21ea123d66cf5fdaf527f546ef5a2f46b (patch) | |
| tree | 8172379c6938219a864f887e3eec46005e4a4e29 /dex | |
| parent | 8c03d52e2e7c99ca615c7f67312b584af6eab637 (diff) | |
| download | dexon-f5fa30b21ea123d66cf5fdaf527f546ef5a2f46b.tar.gz dexon-f5fa30b21ea123d66cf5fdaf527f546ef5a2f46b.tar.zst dexon-f5fa30b21ea123d66cf5fdaf527f546ef5a2f46b.zip | |
app: fix bug when prepare transaction
Diffstat (limited to 'dex')
| -rw-r--r-- | dex/app.go | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/dex/app.go b/dex/app.go index 577135dc0..d5b7d3556 100644 --- a/dex/app.go +++ b/dex/app.go @@ -160,6 +160,8 @@ func (d *DexconApp) PreparePayload(position coreTypes.Position) (payload []byte, blockGasLimit := new(big.Int).SetUint64(core.CalcGasLimit(d.blockchain.CurrentBlock(), d.config.GasFloor, d.config.GasCeil)) blockGasUsed := new(big.Int) var allTxs types.Transactions + +addressMap: for address, txs := range txsMap { // every address's transactions will appear in fixed chain if !d.checkChain(address, chainNums, chainID) { @@ -217,8 +219,7 @@ func (d *DexconApp) PreparePayload(position coreTypes.Position) (payload []byte, blockGasUsed = new(big.Int).Add(blockGasUsed, new(big.Int).SetUint64(tx.Gas())) if blockGasLimit.Cmp(blockGasUsed) < 0 { - log.Error("Reach block gas limit", "limit", blockGasLimit, "gasUsed", blockGasUsed) - return nil, fmt.Errorf("reach block gas limit %v", blockGasLimit) + break addressMap } allTxs = append(allTxs, tx) @@ -238,9 +239,13 @@ func (d *DexconApp) PrepareWitness(consensusHeight uint64) (witness coreTypes.Wi if d.lastPendingHeight == 0 && consensusHeight == 0 { witnessBlock = d.blockchain.CurrentBlock() } else if d.lastPendingHeight >= consensusHeight { + d.insertMu.Lock() witnessBlock = d.blockchain.GetPendingBlockByHeight(d.lastPendingHeight) + d.insertMu.Unlock() } else if h := <-d.addNotify(consensusHeight); h >= consensusHeight { + d.insertMu.Lock() witnessBlock = d.blockchain.GetPendingBlockByHeight(h) + d.insertMu.Unlock() } else { log.Error("need pending block") return witness, fmt.Errorf("need pending block") |