openingd: make sure we take utxos on success.

Otherwise recent additional checks in tal() complain that we're freeing a take() pointer. In this case, we're exiting so it's harmless, but it's still a latent bug. Signed-off-by: Rusty Russell <[email protected]>
vswee · Dec 29, 2018 · b155dbb · b155dbb
1 parent 7314c71
commit b155dbb
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/openingd/openingd.c b/openingd/openingd.c
@@ -778,6 +778,9 @@ static u8 *funder_channel(struct state *state,
 
 	peer_billboard(false, "Funding channel: opening negotiation succeeded");
 
+	if (taken(utxos))
+		tal_free(utxos);
+
 	/* BOLT #2:
 	 *
 	 * The recipient: