Merge pull request ethereum#125 from spencer-tb/len-blob-errors

Add blob length errors for engine api and transition tool.

beacon/engine/types.go

@@ -192,6 +192,12 @@ func ExecutableDataToBlock(params ExecutableData) (*types.Block, error) {
 		h := types.DeriveSha(types.Withdrawals(params.Withdrawals), trie.NewStackTrie(nil))
 		withdrawalsRoot = &h
 	}
+	// Check that number of blobs are valid
+	for _, tx := range txs {
+		if tx.Type() == types.BlobTxType && len(tx.DataHashes()) == 0 {
+			return nil, fmt.Errorf("zero blobs within a blob transaction")
+		}
+	}
 	header := &types.Header{
 		ParentHash:      params.ParentHash,
 		UncleHash:       types.EmptyUncleHash,

cmd/evm/internal/t8ntool/execution.go

@@ -177,6 +177,9 @@ func (pre *Prestate) Apply(vmConfig vm.Config, chainConfig *params.ChainConfig,
 
 	for i, tx := range txs {
 		msg, err := core.TransactionToMessage(tx, signer, pre.Env.BaseFee)
+		if tx.Type() == types.BlobTxType && len(tx.DataHashes()) == 0 {
+			err = fmt.Errorf("blob transaction with zero blobs")
+		}
 		if err != nil {
 			log.Warn("rejected tx", "index", i, "hash", tx.Hash(), "error", err)
 			rejectedTxs = append(rejectedTxs, &rejectedTx{i, err.Error()})