From 58b7612987100261a6597d63d851003c1d982fcf Mon Sep 17 00:00:00 2001 From: Curt Brune Date: Tue, 18 Feb 2020 09:02:11 -0800 Subject: [PATCH] Drop requests to deny stale incoming calls. This is not an illegal state, as the remote side could have hung-up a microsecond before the local side tries to deny the call. --- .../org/thoughtcrime/securesms/service/WebRtcCallService.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/src/main/java/org/thoughtcrime/securesms/service/WebRtcCallService.java b/app/src/main/java/org/thoughtcrime/securesms/service/WebRtcCallService.java index 0471675a1e..8bfd126422 100644 --- a/app/src/main/java/org/thoughtcrime/securesms/service/WebRtcCallService.java +++ b/app/src/main/java/org/thoughtcrime/securesms/service/WebRtcCallService.java @@ -423,7 +423,8 @@ public class WebRtcCallService extends Service implements CallManager.Observer, private void handleDenyCall(Intent intent) { if (activePeer == null) { - throw new IllegalStateException("Deny with no active call"); + Log.i(TAG, "handleDenyCall(): Ignoring for inactive call."); + return; } if (activePeer.getState() != CallState.LOCAL_RINGING) {