Fix NPE when calling unicodeWrap DO NOT MERGE

unicodeWrap isn't null safe so check before calling

Bug:18086401
Change-Id: I74537fff226276733b0475029b4a4a57a0055244
(cherry picked from commit b2215766433478ac1c53d61c08f4844326dfe4f8)

2 files changed, 7 insertions, 5 deletions

diff --git a/src/com/android/contacts/interactions/CallLogInteraction.java b/src/com/android/contacts/interactions/CallLogInteraction.java
index f751ae7d2..89ca3fb61 100644
--- a/src/com/android/contacts/interactions/CallLogInteraction.java
+++ b/src/com/android/contacts/interactions/CallLogInteraction.java
@@ -161,8 +161,9 @@ public class CallLogInteraction implements ContactInteraction {
     }
 
     public String getNumber() {
-        return sBidiFormatter.unicodeWrap(
-                mValues.getAsString(Calls.NUMBER), TextDirectionHeuristics.LTR);
+        final String number = mValues.getAsString(Calls.NUMBER);
+        return number == null ? null :
+            sBidiFormatter.unicodeWrap(number, TextDirectionHeuristics.LTR);
     }
 
     public Integer getNumberPresentation() {
@@ -210,4 +211,4 @@ public class CallLogInteraction implements ContactInteraction {
     public int getIconResourceId() {
         return CALL_LOG_ICON_RES;
     }
-}
\ No newline at end of file
+}
diff --git a/src/com/android/contacts/interactions/SmsInteraction.java b/src/com/android/contacts/interactions/SmsInteraction.java
index 8b26023b2..8e61213e9 100644
--- a/src/com/android/contacts/interactions/SmsInteraction.java
+++ b/src/com/android/contacts/interactions/SmsInteraction.java
@@ -92,8 +92,9 @@ public class SmsInteraction implements ContactInteraction {
     }
 
     public String getAddress() {
-        return sBidiFormatter.unicodeWrap(
-                mValues.getAsString(Sms.ADDRESS), TextDirectionHeuristics.LTR);
+        final String address = mValues.getAsString(Sms.ADDRESS);
+        return address == null ? null :
+            sBidiFormatter.unicodeWrap(address, TextDirectionHeuristics.LTR);
     }
 
     public String getBody() {