Checking transaction recepient validity (#780)

src/status_im/transactions/handlers.cljs

@@ -4,6 +4,7 @@
             [status-im.navigation.handlers :as nav]
             [status-im.utils.handlers :as u]
             [status-im.utils.types :as t]
+            [status-im.utils.hex :refer [valid-hex?]]
             [status-im.components.status :as status]
             [clojure.string :as s]
             [taoensso.timbre :as log]))
@@ -120,13 +121,15 @@
 (register-handler ::transaction-queued
   (after #(dispatch [:navigate-to-modal :confirm]))
   (fn [db [_ {:keys [id message_id args]}]]
-    (let [{:keys [from to value]} args
+    (let [{:keys [from to value]} args]
-          transaction {:id         id
+      (if (valid-hex? to)
-                       :from       from
+        (let [transaction {:id         id
-                       :to         to
+                           :from       from
-                       :value      (.toDecimal js/Web3.prototype value)
+                           :to         to
-                       :message-id message_id}]
+                           :value      (.toDecimal js/Web3.prototype value)
-      (assoc-in db [:transactions-queue id] transaction))))
+                           :message-id message_id}]
+          (assoc-in db [:transactions-queue id] transaction))
+        db))))
 
 (register-handler :transaction-completed
   (u/side-effect!

src/status_im/utils/hex.cljs

@@ -5,3 +5,8 @@
   (if (and hex (s/starts-with? hex "0x"))
     (subs hex 2)
     hex))
+
+(defn valid-hex? [hex]
+  (let [hex (normalize-hex hex)]
+    (and (re-matches #"^[0-9a-fA-F]+$" hex)
+         (not= (js/parseInt hex 16) 0))))