Adding assert and not using unitialized value even though it wouldn't…

… have mattered.
nanocurrency · Feb 6, 2016 · fe11c29 · fe11c29
1 parent 82cfb5c
commit fe11c29
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/rai/node/node.cpp b/rai/node/node.cpp
@@ -791,7 +791,10 @@ bool rai::node_config::deserialize_json (bool & upgraded_a, boost::property_tree
 			boost::asio::ip::address address;
 			uint16_t port;
 			result |= rai::parse_address_port (work_peer, address, port);
-			work_peers.push_back (std::make_pair (address, port));
+			if (!result)
+			{
+				work_peers.push_back (std::make_pair (address, port));
+			}
 		}
 		auto preconfigured_peers_l (tree_a.get_child ("preconfigured_peers"));
 		preconfigured_peers.clear ();

diff --git a/rai/secure.cpp b/rai/secure.cpp
@@ -671,6 +671,7 @@ std::unique_ptr <rai::block> rai::deserialize_block (rai::stream & stream_a, rai
             break;
         }
         default:
+			assert (false);
             break;
     }
     return result;
-Original file line number
+Diff line change
@@ Expand Up @@
                 break;
             }
             default:
+    			assert (false);
                 break;
         }
         return result;
@@ Expand Down @@