From 4e7fc99ae1594403bf3b626739f1780a296a9dd1 Mon Sep 17 00:00:00 2001 From: Christian Decker Date: Fri, 8 Jun 2018 18:34:44 +0200 Subject: [PATCH] gossip: Duplicate removes can result in null pointers in broadcast Signed-off-by: Christian Decker --- gossipd/broadcast.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/gossipd/broadcast.c b/gossipd/broadcast.c index 4cbbb05e4..6bcb7afd1 100644 --- a/gossipd/broadcast.c +++ b/gossipd/broadcast.c @@ -33,8 +33,10 @@ struct broadcast_state *new_broadcast_state(tal_t *ctx) void broadcast_del(struct broadcast_state *bstate, u64 index, const u8 *payload) { const struct queued_message *q = uintmap_del(&bstate->broadcasts, index); - assert(q->payload == payload); - broadcast_state_check(bstate, "broadcast_del"); + if (q != NULL) { + assert(q->payload == payload); + broadcast_state_check(bstate, "broadcast_del"); + } } static void destroy_queued_message(struct queued_message *msg,