From d00aa1e77a5d1b42361d34865bfaca4548bd95ba Mon Sep 17 00:00:00 2001 From: Jon Chambers Date: Wed, 27 Jul 2022 09:56:08 -0400 Subject: [PATCH] Fix an inconsistent check for the presence/absence of "change number" arguments --- .../textsecuregcm/storage/ChangeNumberManager.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/service/src/main/java/org/whispersystems/textsecuregcm/storage/ChangeNumberManager.java b/service/src/main/java/org/whispersystems/textsecuregcm/storage/ChangeNumberManager.java index 796da06aa..b87168cad 100644 --- a/service/src/main/java/org/whispersystems/textsecuregcm/storage/ChangeNumberManager.java +++ b/service/src/main/java/org/whispersystems/textsecuregcm/storage/ChangeNumberManager.java @@ -64,8 +64,8 @@ public class ChangeNumberManager { IncomingMessage::getDestinationDeviceId, IncomingMessage::getDestinationRegistrationId)), false); - } else if (!ObjectUtils.allNull(deviceSignedPreKeys, deviceMessages, pniRegistrationIds)) { - throw new IllegalArgumentException("Signed pre-keys, device messages, and registration IDs must be all null or all non-null"); + } else if (!ObjectUtils.allNull(pniIdentityKey, deviceSignedPreKeys, deviceMessages, pniRegistrationIds)) { + throw new IllegalArgumentException("PNI identity key, signed pre-keys, device messages, and registration IDs must be all null or all non-null"); } final Account updatedAccount;