diff options
author | Jay Shrauner <shrauner@google.com> | 2013-11-19 09:43:56 -0800 |
---|---|---|
committer | Jay Shrauner <shrauner@google.com> | 2013-11-19 09:43:56 -0800 |
commit | 21d991ee8bab6e2ef7d9df0b268b39053b21cf5b (patch) | |
tree | 67f54d13fb90fb75fede7d9b0f2d908e869ff990 /src | |
parent | f3b71b7d92bd9179dd35fff423063ae9abdcffae (diff) |
Fix crash when number is null
Fix crash in various routines when the number is null.
Bug:11766758
Change-Id: I3bcedbd8ada663a8c47528f32e213f653195fe57
Diffstat (limited to 'src')
-rw-r--r-- | src/com/android/dialer/calllog/PhoneNumberUtilsWrapper.java | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/com/android/dialer/calllog/PhoneNumberUtilsWrapper.java b/src/com/android/dialer/calllog/PhoneNumberUtilsWrapper.java index 32b49b336..2133f0f9d 100644 --- a/src/com/android/dialer/calllog/PhoneNumberUtilsWrapper.java +++ b/src/com/android/dialer/calllog/PhoneNumberUtilsWrapper.java @@ -50,7 +50,7 @@ public class PhoneNumberUtilsWrapper { * mock-out this, it is not a static method. */ public boolean isVoicemailNumber(CharSequence number) { - return PhoneNumberUtils.isVoiceMailNumber(number.toString()); + return number!= null && PhoneNumberUtils.isVoiceMailNumber(number.toString()); } /** @@ -58,7 +58,7 @@ public class PhoneNumberUtilsWrapper { * static method. */ public boolean isSipNumber(CharSequence number) { - return PhoneNumberUtils.isUriNumber(number.toString()); + return number != null && PhoneNumberUtils.isUriNumber(number.toString()); } public static boolean isUnknownNumberThatCanBeLookedUp(CharSequence number, int presentation) { @@ -77,13 +77,13 @@ public class PhoneNumberUtilsWrapper { if (new PhoneNumberUtilsWrapper().isVoicemailNumber(number)) { return false; } - if (isLegacyUnknownNumbers(number.toString())) { + if (isLegacyUnknownNumbers(number)) { return false; } return true; } public static boolean isLegacyUnknownNumbers(CharSequence number) { - return LEGACY_UNKNOWN_NUMBERS.contains(number.toString()); + return number != null && LEGACY_UNKNOWN_NUMBERS.contains(number.toString()); } } |