From 4a93658d0f01275189bc56e0ed2bd75cafd50ca3 Mon Sep 17 00:00:00 2001 From: Moxie Marlinspike Date: Thu, 5 Mar 2015 08:36:56 -0800 Subject: [PATCH] Don't consider an empty string to be a possible relay. // FREEBIE --- .../textsecuregcm/controllers/MessageController.java | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/whispersystems/textsecuregcm/controllers/MessageController.java b/src/main/java/org/whispersystems/textsecuregcm/controllers/MessageController.java index bfe4c3562..bd1be52ae 100644 --- a/src/main/java/org/whispersystems/textsecuregcm/controllers/MessageController.java +++ b/src/main/java/org/whispersystems/textsecuregcm/controllers/MessageController.java @@ -39,6 +39,7 @@ import org.whispersystems.textsecuregcm.storage.Account; import org.whispersystems.textsecuregcm.storage.AccountsManager; import org.whispersystems.textsecuregcm.storage.Device; import org.whispersystems.textsecuregcm.util.Base64; +import org.whispersystems.textsecuregcm.util.Util; import javax.validation.Valid; import javax.ws.rs.Consumes; @@ -94,8 +95,8 @@ public class MessageController { try { boolean isSyncMessage = source.getNumber().equals(destinationName); - if (messages.getRelay() == null) sendLocalMessage(source, destinationName, messages, isSyncMessage); - else sendRelayMessage(source, destinationName, messages, isSyncMessage); + if (Util.isEmpty(messages.getRelay())) sendLocalMessage(source, destinationName, messages, isSyncMessage); + else sendRelayMessage(source, destinationName, messages, isSyncMessage); return new SendMessageResponse(!isSyncMessage && source.getActiveDeviceCount() > 1); } catch (NoSuchUserException e) {