From 6e980cb316154551c04cf4d9a87c914d75704428 Mon Sep 17 00:00:00 2001
From: "B. Petersen" <r10s@b44t.com>
Date: Sat, 6 Jul 2019 00:36:54 +0200
Subject: [PATCH] check against null-pointer return from
 RingtoneManager.getRingtone(), tackles #984

---
 .../securesms/notifications/MessageNotifier.java | 16 +++++++++++-----
 1 file changed, 11 insertions(+), 5 deletions(-)

diff --git a/src/org/thoughtcrime/securesms/notifications/MessageNotifier.java b/src/org/thoughtcrime/securesms/notifications/MessageNotifier.java
index 03aafe341..c5acc1ad8 100644
--- a/src/org/thoughtcrime/securesms/notifications/MessageNotifier.java
+++ b/src/org/thoughtcrime/securesms/notifications/MessageNotifier.java
@@ -3,6 +3,7 @@ package org.thoughtcrime.securesms.notifications;
 import android.app.NotificationManager;
 import android.content.Context;
 import android.media.AudioManager;
+import android.media.Ringtone;
 import android.media.RingtoneManager;
 import android.media.SoundPool;
 import android.net.Uri;
@@ -279,13 +280,18 @@ abstract class MessageNotifier {
 
     }
 
-    void playNotificationSound(Uri ringtone, boolean vibrate) {
-        RingtoneManager.getRingtone(appContext, ringtone).play();
-        if (vibrate) {
+    private void playNotificationSound(Uri uri, boolean vibrate) {
+        Ringtone ringtone = RingtoneManager.getRingtone(appContext, uri);
+        if (ringtone!=null) {
+            ringtone.play();
+        }
 
+        if (vibrate) {
             Vibrator v = (Vibrator) appContext.getSystemService(Context.VIBRATOR_SERVICE);
-            v.vibrate(100);
-            v.vibrate(200);
+            if (v!=null) {
+                v.vibrate(100);
+                v.vibrate(200);
+            }
         }
     }