From 02865f99a9b8e84611c9d8db7ee31bff2ceceb63 Mon Sep 17 00:00:00 2001 From: Greyson Parrelli Date: Wed, 29 Jan 2020 18:28:59 -0500 Subject: [PATCH] Limit impact of crash on unexpected SMS receive. --- .../org/thoughtcrime/securesms/jobs/SmsReceiveJob.java | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/app/src/main/java/org/thoughtcrime/securesms/jobs/SmsReceiveJob.java b/app/src/main/java/org/thoughtcrime/securesms/jobs/SmsReceiveJob.java index bf96e51ec4..4b3afcdee7 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/jobs/SmsReceiveJob.java +++ b/app/src/main/java/org/thoughtcrime/securesms/jobs/SmsReceiveJob.java @@ -71,8 +71,10 @@ public class SmsReceiveJob extends BaseJob { @Override public void onRun() throws MigrationPendingException { - Log.i(TAG, "onRun()"); - + if (TextSecurePreferences.getLocalUuid(context) == null && TextSecurePreferences.getLocalNumber(context) == null) { + throw new NotReadyException(); + } + Optional message = assembleMessageFragments(pdus, subscriptionId); if (message.isPresent() && !isBlocked(message.get())) { @@ -167,4 +169,7 @@ public class SmsReceiveJob extends BaseJob { } } } + + private class NotReadyException extends RuntimeException { + } }