BlockState.AddTransactions: fix revert in the case where tx is the fi…

…rst in the sequence

miner/collator.go

@@ -320,7 +320,12 @@ func (bs *collatorBlockState) AddTransactions(txs types.Transactions) (error, ty
 
 	if retErr != nil {
 		bs.logs = bs.logs[:len(bs.logs)-tcount]
-		bs.state.RevertToSnapshot(bs.snapshots[len(bs.snapshots)-(tcount+1)])
+		if tcount != 0 {
+			// first transaction in the sequence failed so the state will already be reverted
+			// to what it was before the sequence was applied
+			bs.state.RevertToSnapshot(bs.snapshots[len(bs.snapshots)-(tcount+1)])
+		}
+
 		bs.snapshots = bs.snapshots[:len(bs.snapshots)-(tcount+1)]
 
 		return retErr, nil

miner/worker.go

@@ -609,7 +609,7 @@ func (w *worker) makeEnv(parent *types.Block, header *types.Header, coinbase com
 		state:     state,
 		gasPool:   new(core.GasPool).AddGas(header.GasLimit),
 		tcount:    0,
-		snapshots: []int{state.Snapshot()},
+		snapshots: []int{},
 	}
 
 	ctx, cancel := context.WithCancel(context.Background())