fix(ui): Shutdown the child tasks if the channel got closed in the su…

…pervisor

crates/matrix-sdk-ui/src/sync_service.rs

@@ -109,9 +109,13 @@ impl SyncTaskSupervisor {
         let state = inner.state.clone();
 
         async move {
-            let Some(report) = receiver.recv().await else {
+            let report = if let Some(report) = receiver.recv().await {
+                report
+            } else {
                 info!("internal channel has been closed?");
-                return;
+                // We should still stop the child tasks in the unlikely scenario that our
+                // receiver died.
+                TerminationReport::supervisor_error()
             };
 
             // If one service failed, make sure to request stopping the other one.
@@ -417,6 +421,16 @@ struct TerminationReport {
     origin: TerminationOrigin,
 }
 
+impl TerminationReport {
+    fn supervisor_error() -> Self {
+        TerminationReport {
+            is_error: true,
+            has_expired: false,
+            origin: TerminationOrigin::Supervisor,
+        }
+    }
+}
+
 // Testing helpers, mostly.
 #[doc(hidden)]
 impl SyncService {