aljaz/lightning
1
0
Fork 0
mirror of https://github.com/aljazceru/lightning.git synced 2025-12-20 15:44:21 +01:00
Code Issues Packages Projects Releases Wiki Activity

bcli: allocate temp string off of short-lived cmd

Browse Source 
`cmd` gets cleaned up almost immediately, and is the right scope for a
temporary string allocation
This commit is contained in:
niftynei
2020-12-09 13:05:12 -06:00
committed by Christian Decker
parent f9a2489424
commit 08c8581461
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
plugins/bcli.c
View File

@@ -601,7 +601,7 @@ static struct command_result *process_sendrawtransaction(struct bitcoin_cli *bcl
response = jsonrpc_stream_success(bcli->cmd); response = jsonrpc_stream_success(bcli->cmd);
json_add_bool(response, "success", *bcli->exitstatus == 0); json_add_bool(response, "success", *bcli->exitstatus == 0);
json_add_string(response, "errmsg", json_add_string(response, "errmsg",
bcli->exitstatus ? tal_strndup(bcli, bcli->output, bcli->exitstatus ? tal_strndup(bcli->cmd, bcli->output,
bcli->output_bytes-1) : ""); bcli->output_bytes-1) : "");
return command_finished(bcli->cmd, response); return command_finished(bcli->cmd, response);