From 08c8581461933191fced9e2c8c9b6c8e59a66480 Mon Sep 17 00:00:00 2001 From: niftynei Date: Wed, 9 Dec 2020 13:05:12 -0600 Subject: [PATCH] bcli: allocate temp string off of short-lived cmd `cmd` gets cleaned up almost immediately, and is the right scope for a temporary string allocation --- plugins/bcli.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/bcli.c b/plugins/bcli.c index 140f8ee22..49665eb65 100644 --- a/plugins/bcli.c +++ b/plugins/bcli.c @@ -601,7 +601,7 @@ static struct command_result *process_sendrawtransaction(struct bitcoin_cli *bcl response = jsonrpc_stream_success(bcli->cmd); json_add_bool(response, "success", *bcli->exitstatus == 0); json_add_string(response, "errmsg", - bcli->exitstatus ? tal_strndup(bcli, bcli->output, + bcli->exitstatus ? tal_strndup(bcli->cmd, bcli->output, bcli->output_bytes-1) : ""); return command_finished(bcli->cmd, response);