channel: don't sent announcement_signatures if we're already shutting…

… down. This fixes an occasional test failure, too: lightningd(2339): 0296d4e77857ab47b20ef43c055d4770617405df02d79b6c934ab8c27a49207cfb chan #1: Peer permanent failure in CLOSINGD_SIGEXCHANGE: lightning_closingd: sent ERROR Expected closing_signed: 0103231454a085a9d186abd6affe845bdea9fd2f0a88e4af8b68e3a90a5dc1e11c080001b2000001000137fa480dccdf1b9e46fe5c0939ffedb2a719bd5a4d4672ba00d11f8e61062ce8721c32f93bd849df27306b46e6212fb9eae9c1b22f852229f9f1ccdaa39641ab9433ddb1ca27de818dd21a4c499b9336b6c6e91c476b69617deb019bd1c636771a5b1ecc09ef0902e7226d6e86f8a3ce747178d010b744cf6271359a4bc77899 Signed-off-by: Rusty Russell <[email protected]>
rustyrussell · Feb 23, 2018 · e1fc8ec · e1fc8ec
1 parent 0a20bd4
commit e1fc8ec
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/channeld/channel.c b/channeld/channel.c
@@ -322,6 +322,10 @@ static void send_announcement_signatures(struct peer *peer)
 	    !peer->funding_locked[REMOTE])
 		return;
 
+	/* FIXME: Put in spec! */
+	if (peer->shutdown_sent[LOCAL])
+		return;
+
 	tmpctx = tal_tmpctx(peer);
 	status_trace("Exchanging announcement signatures.");
 	ca = create_channel_announcement(tmpctx, peer);