From bb601a1eebf0ed57eec33506d51816c1ee3b5463 Mon Sep 17 00:00:00 2001 From: Rusty Russell Date: Wed, 20 Dec 2017 19:56:54 +1030 Subject: [PATCH] gossipd/test/run-bench-find_route: don't abort if we try to route to ourselves This would fail, and we'd free an uninitialized pointer. Also, add us to .gitignore and clear up a comment. Signed-off-by: Rusty Russell --- gossipd/test/.gitignore | 1 + gossipd/test/run-bench-find_route.c | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/gossipd/test/.gitignore b/gossipd/test/.gitignore index 306731586..fc88c5323 100644 --- a/gossipd/test/.gitignore +++ b/gossipd/test/.gitignore @@ -1 +1,2 @@ run-find_route +run-bench-find_route diff --git a/gossipd/test/run-bench-find_route.c b/gossipd/test/run-bench-find_route.c index 9407a1943..6f40e8fa7 100644 --- a/gossipd/test/run-bench-find_route.c +++ b/gossipd/test/run-bench-find_route.c @@ -113,7 +113,7 @@ static void populate_random_node(struct routing_state *rstate, u64 n) { struct pubkey id = nodeid(n); - /* Create up to 2 random channels. */ + /* Create 2 random channels. */ if (n < 1) return; @@ -189,7 +189,7 @@ int main(int argc, char *argv[]) struct pubkey from = nodeid(pseudorand(num_nodes)); struct pubkey to = nodeid(pseudorand(num_nodes)); u64 fee; - struct node_connection **route, *nc; + struct node_connection **route = NULL, *nc; nc = find_route(ctx, rstate, &from, &to, pseudorand(100000),