review use array last for safety

element-hq · Apr 15, 2024 · afd6dee · afd6dee
1 parent a5d4d1c
commit afd6dee
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/Riot/Modules/Analytics/DecryptionFailureTracker.swift b/Riot/Modules/Analytics/DecryptionFailureTracker.swift
@@ -103,12 +103,12 @@ class DecryptionFailureTracker: NSObject {
         failure.eventLocalAgeMillis = Int(exactly: eventRelativeAgeMillis)
         failure.trustOwnIdentityAtTimeOfFailure = isSessionVerified
 
-        let myDomain = userId.components(separatedBy: ":")[1]
+        let myDomain = userId.components(separatedBy: ":").last
         failure.isMatrixOrg = myDomain == "matrix.org"
 
         if MXTools.isMatrixUserIdentifier(event.sender) {
-            let senderDomain = event.sender.components(separatedBy: ":")[1]
-            failure.isFederated = senderDomain != myDomain
+            let senderDomain = event.sender.components(separatedBy: ":").last
+            failure.isFederated = senderDomain != nil && senderDomain != myDomain
         }
 
         /// XXX for future work, as for now only the event formatter reports UTDs. That means that it's only UTD ~visible to users