From 4a32ccecfa26303f218167ed38edd3dc43380b57 Mon Sep 17 00:00:00 2001
From: Florian Renaud <florianr@element.io>
Date: Tue, 18 Oct 2022 00:37:38 +0200
Subject: [PATCH] Voice Broadcast Player - Add missing try catch

---
 .../app/features/voicebroadcast/VoiceBroadcastPlayer.kt  | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/vector/src/main/java/im/vector/app/features/voicebroadcast/VoiceBroadcastPlayer.kt b/vector/src/main/java/im/vector/app/features/voicebroadcast/VoiceBroadcastPlayer.kt
index 8970a65990..41ed1ff84d 100644
--- a/vector/src/main/java/im/vector/app/features/voicebroadcast/VoiceBroadcastPlayer.kt
+++ b/vector/src/main/java/im/vector/app/features/voicebroadcast/VoiceBroadcastPlayer.kt
@@ -120,7 +120,14 @@ class VoiceBroadcastPlayer @Inject constructor(
     }
 
     private suspend fun prepareMediaPlayer(messageAudioContent: MessageAudioContent): MediaPlayer {
-        val audioFile = session.fileService().downloadFile(messageAudioContent)
+        // Download can fail
+        val audioFile = try {
+            session.fileService().downloadFile(messageAudioContent)
+        } catch (failure: Throwable) {
+            Timber.e(failure, "Unable to start playback")
+            throw VoiceFailure.UnableToPlay(failure)
+        }
+
         return audioFile.inputStream().use { fis ->
             MediaPlayer().apply {
                 setAudioAttributes(