From 5ab06940cabd9aaced3c9b61fab5fd419937b876 Mon Sep 17 00:00:00 2001 From: nielsandriesse Date: Fri, 8 Jan 2021 13:31:46 +1100 Subject: [PATCH] Add safety check --- .../Sending & Receiving/MessageReceiver+Handling.swift | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/SessionMessagingKit/Sending & Receiving/MessageReceiver+Handling.swift b/SessionMessagingKit/Sending & Receiving/MessageReceiver+Handling.swift index 72a4b3a7e..ff3cb175f 100644 --- a/SessionMessagingKit/Sending & Receiving/MessageReceiver+Handling.swift +++ b/SessionMessagingKit/Sending & Receiving/MessageReceiver+Handling.swift @@ -286,6 +286,10 @@ extension MessageReceiver { guard Set(group.groupMemberIds).contains(message.sender!) else { return SNLog("Ignoring closed group update message from non-member.") } + // Check that the admin wasn't removed + guard members.contains(group.groupAdminIds.first!) else { + return SNLog("Ignoring invalid closed group update message.") + } // Remove the group from the user's set of public keys to poll for if the current user was removed let userPublicKey = getUserHexEncodedPublicKey() let wasCurrentUserRemoved = !members.contains(userPublicKey)