diff options
author | James.cf Lin <jamescflin@google.com> | 2021-05-18 09:14:40 +0800 |
---|---|---|
committer | James.cf Lin <jamescflin@google.com> | 2021-05-18 09:14:40 +0800 |
commit | 0a9ad5e0dfd4ebf144f180ad1598019d7d6ff1a3 (patch) | |
tree | aee40750dbc9b1c1939257f829eaf3326175884d | |
parent | 7df39d79379f872429ec43bf2ea5ebce15372cde (diff) | |
download | ims-0a9ad5e0dfd4ebf144f180ad1598019d7d6ff1a3.tar.gz |
Add log to check whetner the IMPU is null or not
Bug: 188367356
Test: atest ImsServiceTest
Merged-In: I227a8c8c0986c422c7d99f6d472d1e1adf97eeaa
Change-Id: I2118c6d98d7b98e49ff06e19cbba7319e5ee27a1
-rw-r--r-- | src/java/com/android/ims/rcs/uce/presence/publish/PublishUtils.java | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/src/java/com/android/ims/rcs/uce/presence/publish/PublishUtils.java b/src/java/com/android/ims/rcs/uce/presence/publish/PublishUtils.java index a7bd2e34..de7305b0 100644 --- a/src/java/com/android/ims/rcs/uce/presence/publish/PublishUtils.java +++ b/src/java/com/android/ims/rcs/uce/presence/publish/PublishUtils.java @@ -59,17 +59,23 @@ class PublishUtils { String[] impus = telephonyManager.getIsimImpu(); if (TextUtils.isEmpty(domain) || impus == null) { + Log.d(LOG_TAG, "getContactUriFromIsim: domain is null=" + TextUtils.isEmpty(domain)); + Log.d(LOG_TAG, "getContactUriFromIsim: impu is null=" + + ((impus == null || impus.length == 0) ? "true" : "false")); return null; } for (String impu : impus) { if (TextUtils.isEmpty(impu)) continue; Uri impuUri = Uri.parse(impu); - if (SCHEME_SIP.equals(impuUri.getScheme()) && - impuUri.getSchemeSpecificPart().endsWith(domain)) { + String scheme = impuUri.getScheme(); + String schemeSpecificPart = impuUri.getSchemeSpecificPart(); + if (SCHEME_SIP.equals(scheme) && !TextUtils.isEmpty(schemeSpecificPart) && + schemeSpecificPart.endsWith(domain)) { return impuUri; } } + Log.d(LOG_TAG, "getContactUriFromIsim: there is no impu matching the domain"); return null; } |