From d195cfab50b42d26f3cf9436845838e075e959de Mon Sep 17 00:00:00 2001
From: frankiebee <frankie.diamond@gmail.com>
Date: Tue, 13 Mar 2018 15:13:05 -0700
Subject: [PATCH] transactions - insure if a to field in tx params has a truthy
 valu that it is a valid addres and if it is falsy that it is not null to fix
 issue #3509

---
 app/scripts/lib/tx-gas-utils.js | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/app/scripts/lib/tx-gas-utils.js b/app/scripts/lib/tx-gas-utils.js
index 6f6ff7852..e61db3332 100644
--- a/app/scripts/lib/tx-gas-utils.js
+++ b/app/scripts/lib/tx-gas-utils.js
@@ -4,7 +4,7 @@ const {
   BnMultiplyByFraction,
   bnToHex,
 } = require('./util')
-const addHexPrefix = require('ethereumjs-util').addHexPrefix
+const {addHexPrefix, isValidAddress} = require('ethereumjs-util')
 const SIMPLE_GAS_COST = '0x5208' // Hex for 21000, cost of a simple send.
 
 /*
@@ -101,6 +101,12 @@ module.exports = class TxGasUtil {
 
   async validateTxParams (txParams) {
     this.validateRecipient(txParams)
+    if ('to' in txParams) {
+      if ( txParams.to === null ) delete txParams.to
+      else if ( txParams.to !== undefined && !isValidAddress(txParams.to) ) {
+        throw new Error(`Invalid transaction value of ${txParams.to} not a valid to address.`)
+      }
+    }
     if ('value' in txParams) {
       const value = txParams.value.toString()
       if (value.includes('-')) {