From 7c4568644038fd2b99856f549bc2f247e98995ca Mon Sep 17 00:00:00 2001
From: Greyson Parrelli <greyson@signal.org>
Date: Wed, 1 Apr 2026 11:10:03 -0400
Subject: [PATCH] Fix potential missing ACI crash in verify screen.

---
 .../securesms/verify/VerifySafetyNumberViewModel.kt       | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/app/src/main/java/org/thoughtcrime/securesms/verify/VerifySafetyNumberViewModel.kt b/app/src/main/java/org/thoughtcrime/securesms/verify/VerifySafetyNumberViewModel.kt
index c47d1c81c0..aacfac66ba 100644
--- a/app/src/main/java/org/thoughtcrime/securesms/verify/VerifySafetyNumberViewModel.kt
+++ b/app/src/main/java/org/thoughtcrime/securesms/verify/VerifySafetyNumberViewModel.kt
@@ -134,13 +134,19 @@ class VerifySafetyNumberViewModel(
     val context: Context = AppDependencies.application
 
     SignalExecutors.BOUNDED.execute {
+      val resolved = recipient.resolve()
+      if (resolved.aci.isEmpty) {
+        Log.w(TAG, "Cannot update safety number verification -- recipient has no ACI")
+        return@execute
+      }
+
       ReentrantSessionLock.INSTANCE.acquire().use { _ ->
         if (verified) {
           Log.i(TAG, "Saving identity: $recipientId")
           AppDependencies.protocolStore.aci().identities()
             .saveIdentityWithoutSideEffects(
               recipientId,
-              recipient.resolve().requireAci(),
+              resolved.requireAci(),
               remoteIdentity,
               IdentityTable.VerifiedStatus.VERIFIED,
               false,