Skip to content

Commit 9fb10b8

Browse files
ddustinsangbida
authored andcommitted
fixup! reestablish: Send announcement sigs when asked
1 parent 3acbaf1 commit 9fb10b8

1 file changed

Lines changed: 8 additions & 14 deletions

File tree

lightningd/channel_gossip.c

Lines changed: 8 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -687,7 +687,7 @@ static void stash_remote_announce_sigs(struct channel *channel,
687687
* - MUST NOT send the `announcement_signatures` message.
688688
*/
689689

690-
static bool send_channel_announce_sigs(struct channel *channel)
690+
static void send_channel_announce_sigs(struct channel *channel)
691691
{
692692
/* First 2 + 256 byte are the signatures and msg type, skip them */
693693
const size_t offset = 258;
@@ -698,10 +698,8 @@ static bool send_channel_announce_sigs(struct channel *channel)
698698
const u8 *ca, *msg;
699699

700700
/* Wait until we've exchanged reestablish messages */
701-
if (!channel->reestablished) {
701+
if (!channel->reestablished)
702702
log_debug(channel->log, "channel_gossip: not sending channel_announcement_sigs until reestablished");
703-
return false;
704-
}
705703

706704
ca = create_channel_announcement(tmpctx, channel, *channel->scid,
707705
NULL, NULL, NULL, NULL);
@@ -716,33 +714,29 @@ static bool send_channel_announce_sigs(struct channel *channel)
716714

717715
/* Double-check that HSM gave valid signatures. */
718716
sha256_double(&hash, ca + offset, tal_count(ca) - offset);
719-
if (!check_signed_hash(&hash, &local_node_sig, &ld->our_pubkey)) {
717+
if (!check_signed_hash(&hash, &local_node_sig, &ld->our_pubkey))
720718
channel_internal_error(channel,
721719
"HSM returned an invalid node signature");
722-
return false;
723-
}
724720

725-
if (!check_signed_hash(&hash, &local_bitcoin_sig, &channel->local_funding_pubkey)) {
721+
if (!check_signed_hash(&hash, &local_bitcoin_sig, &channel->local_funding_pubkey))
726722
channel_internal_error(channel,
727723
"HSM returned an invalid bitcoin signature");
728-
return false;
729-
}
730724

731725
msg = towire_announcement_signatures(NULL,
732726
&channel->cid, *channel->scid,
733727
&local_node_sig, &local_bitcoin_sig);
734728
msg_to_peer(channel->peer, take(msg));
735-
return cg->sent_sigs = true;
729+
cg->sent_sigs = true;
736730
}
737731

738-
static bool send_channel_announce_sigs_once(struct channel *channel)
732+
static void send_channel_announce_sigs_once(struct channel *channel)
739733
{
740734
struct channel_gossip *cg = channel->channel_gossip;
741735

742736
if (cg->sent_sigs)
743-
return false;
737+
return;
744738

745-
return send_channel_announce_sigs(channel);
739+
send_channel_announce_sigs(channel);
746740
}
747741

748742
/* Sends channel_announcement */

0 commit comments

Comments
 (0)