summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBabis Triantafyllou <babis.triantafyllou.x@sonyericsson.com>2010-10-08 16:02:11 +0200
committerPatrick Jacques <kernelzilla@kinetic-computing.com>2010-11-30 01:39:12 -0700
commitf93d3f268afaa75b8b13282b130d258bae4ddcb0 (patch)
tree8356f6b885dd8db65516db9b98157dfcb7f6919c
parentbe1da028a13ee95e094e9cbae8cc09ea3ef239fe (diff)
downloadframeworks_base-f93d3f268afaa75b8b13282b130d258bae4ddcb0.zip
frameworks_base-f93d3f268afaa75b8b13282b130d258bae4ddcb0.tar.gz
frameworks_base-f93d3f268afaa75b8b13282b130d258bae4ddcb0.tar.bz2
NullPointerException at handleMessage(GSMPhone.java)
NullPointerException at com.android.internal.telephony.gsm.GSMPhone.handleMessage(GSMPhone.java) Failing to retrieved the IMSI number from SIM card could lead to an exception. A null pointer check will prevent this. Change-Id: I26760543484504c8d35215bfb1e8f1ae664aeade
-rwxr-xr-xtelephony/java/com/android/internal/telephony/gsm/GSMPhone.java3
1 files changed, 2 insertions, 1 deletions
diff --git a/telephony/java/com/android/internal/telephony/gsm/GSMPhone.java b/telephony/java/com/android/internal/telephony/gsm/GSMPhone.java
index 2bb7968..3d22441 100755
--- a/telephony/java/com/android/internal/telephony/gsm/GSMPhone.java
+++ b/telephony/java/com/android/internal/telephony/gsm/GSMPhone.java
@@ -1219,7 +1219,8 @@ public class GSMPhone extends PhoneBase {
// Check if this is a different SIM than the previous one. If so unset the
// voice mail number.
String imsi = getVmSimImsi();
- if (imsi != null && !getSubscriberId().equals(imsi)) {
+ String imsiFromSIM = getSubscriberId();
+ if (imsi != null && imsiFromSIM != null && !imsiFromSIM.equals(imsi)) {
storeVoiceMailNumber(null);
setVmSimImsi(null);
}