listfunds: also list reserved outputs

Currently 'listfunds' lies, a teensy eeinsy bit, in that it doesn't list
all of the funds in a wallet (it omits reserved wallet UTXOs). This
change makes the reserved outputs visible by listing them in the
'outputs' section along with a new field, 'reserved', which denotes the
UTXO's state

Changelog-Changed: JSON-RPC: `listfunds` 'outputs' now includes reserved outputs, designated as 'reserved' = true

doc/lightning-listfunds.7.md

@@ -28,6 +28,7 @@ Each entry in *outputs* will include:
     appended)
 -   *address*
 -   *status* (whether *unconfirmed*, *confirmed*, or *spent*)
+-   *reserved* (whether this is UTXO is currently reserved for an in-flight tx)
 
 Each entry in *channels* will include:
 -   *peer\_id* - the peer with which the channel is opened.

wallet/walletrpc.c

@@ -797,23 +797,13 @@ static const struct json_command listaddrs_command = {
 };
 AUTODATA(json_command, &listaddrs_command);
 
-static struct command_result *json_listfunds(struct command *cmd,
-					     const char *buffer,
-					     const jsmntok_t *obj UNNEEDED,
-					     const jsmntok_t *params)
+static struct command_result *json_outputs(struct command *cmd,
+					   struct json_stream *response,
+					   struct utxo **utxos)
 {
-	struct json_stream *response;
-	struct peer *p;
-	struct utxo **utxos;
 	char* out;
 	struct pubkey funding_pubkey;
 
-	if (!param(cmd, buffer, params, NULL))
-		return command_param_failed();
-
-	utxos = wallet_get_utxos(cmd, cmd->ld->wallet, output_state_available);
-	response = json_stream_success(cmd);
-	json_array_start(response, "outputs");
 	for (size_t i = 0; i < tal_count(utxos); i++) {
 		json_object_start(response, NULL);
 		json_add_txid(response, "txid", &utxos[i]->txid);
@@ -850,8 +840,38 @@ static struct command_result *json_listfunds(struct command *cmd,
 		} else
 			json_add_string(response, "status", "unconfirmed");
 
+		json_add_bool(response, "reserved",
+			      utxos[i]->status == output_state_reserved);
 		json_object_end(response);
 	}
+
+	return NULL;
+}
+
+static struct command_result *json_listfunds(struct command *cmd,
+					     const char *buffer,
+					     const jsmntok_t *obj UNNEEDED,
+					     const jsmntok_t *params)
+{
+	struct json_stream *response;
+	struct peer *p;
+	struct utxo **utxos, **reserved_utxos;
+	struct command_result *ret;
+
+	if (!param(cmd, buffer, params, NULL))
+		return command_param_failed();
+
+	response = json_stream_success(cmd);
+
+	utxos = wallet_get_utxos(cmd, cmd->ld->wallet, output_state_available);
+	reserved_utxos = wallet_get_utxos(cmd, cmd->ld->wallet, output_state_reserved);
+	json_array_start(response, "outputs");
+	ret = json_outputs(cmd, response, utxos);
+	if (ret)
+		return ret;
+	ret = json_outputs(cmd, response, reserved_utxos);
+	if (ret)
+		return ret;
 	json_array_end(response);
 
 	/* Add funds that are allocated to channels */