From 30933d792b347260994f5f425021a2ae923c4252 Mon Sep 17 00:00:00 2001 From: Moxie Marlinspike Date: Wed, 3 Dec 2014 11:33:34 -0800 Subject: [PATCH] Timestamp comparison should be the other way. // FREEBIE --- .../whispersystems/textsecuregcm/push/FeedbackHandler.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/whispersystems/textsecuregcm/push/FeedbackHandler.java b/src/main/java/org/whispersystems/textsecuregcm/push/FeedbackHandler.java index 5ebe4801d..f40d58725 100644 --- a/src/main/java/org/whispersystems/textsecuregcm/push/FeedbackHandler.java +++ b/src/main/java/org/whispersystems/textsecuregcm/push/FeedbackHandler.java @@ -73,7 +73,7 @@ public class FeedbackHandler implements Managed, Runnable { if (event.getRegistrationId().equals(device.get().getGcmId())) { logger.warn("GCM Unregister GCM ID matches!"); if (device.get().getPushTimestamp() == 0 || - event.getTimestamp() < device.get().getPushTimestamp()) + event.getTimestamp() > device.get().getPushTimestamp()) { logger.warn("GCM Unregister Timestamp matches!"); device.get().setGcmId(null); @@ -96,7 +96,7 @@ public class FeedbackHandler implements Managed, Runnable { if (event.getRegistrationId().equals(device.get().getApnId())) { logger.warn("APN Unregister APN ID matches!"); if (device.get().getPushTimestamp() == 0 || - event.getTimestamp() < device.get().getPushTimestamp()) + event.getTimestamp() > device.get().getPushTimestamp()) { logger.warn("APN Unregister timestamp matches!"); device.get().setApnId(null);