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

Anchor outputs are enabled by dual-funding, so this was reported by

Browse Source 
users there (and reliably detected now).

Reported-by: @larsschenk
Reported-by: @nickfarrow
Fixes: #4728
Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Changelog-Fixed: EXPERIMENTAL: crash for some users while requesting dual funding leases.
This commit is contained in:
Rusty Russell
2021-09-01 12:23:31 +09:30
committed by Christian Decker
parent 2e01d26588
commit 372fffac69
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
common/initial_commit_tx.c
View File

@@ -1,5 +1,6 @@
#include <bitcoin/script.h> #include <bitcoin/script.h>
#include <bitcoin/tx.h> #include <bitcoin/tx.h>
#include <ccan/array_size/array_size.h>
#include <ccan/endian/endian.h> #include <ccan/endian/endian.h>
#include <common/initial_commit_tx.h> #include <common/initial_commit_tx.h>
#include <common/keyset.h> #include <common/keyset.h>
@@ -101,7 +102,8 @@ struct bitcoin_tx *initial_commit_tx(const tal_t *ctx,
enum side lessor = !opener; enum side lessor = !opener;
u32 sequence; u32 sequence;
void *dummy_local = (void *)LOCAL, *dummy_remote = (void *)REMOTE; void *dummy_local = (void *)LOCAL, *dummy_remote = (void *)REMOTE;
const void *output_order[NUM_SIDES]; /* There is a direct, and possibly an anchor output for each side. */
const void *output_order[2 * NUM_SIDES];
const u8 *funding_wscript = bitcoin_redeem_2of2(tmpctx, const u8 *funding_wscript = bitcoin_redeem_2of2(tmpctx,
&funding_key[LOCAL], &funding_key[LOCAL],
&funding_key[REMOTE]); &funding_key[REMOTE]);
@@ -286,6 +288,7 @@ struct bitcoin_tx *initial_commit_tx(const tal_t *ctx,
} }
assert(n <= tx->wtx->num_outputs); assert(n <= tx->wtx->num_outputs);
assert(n <= ARRAY_SIZE(output_order));
/* BOLT #3: /* BOLT #3:
* *