diff options
author | Babis Triantafyllou <babis.triantafyllou.x@sonyericsson.com> | 2010-10-08 16:02:11 +0200 |
---|---|---|
committer | Patrick Jacques <kernelzilla@kinetic-computing.com> | 2010-11-30 01:39:12 -0700 |
commit | f93d3f268afaa75b8b13282b130d258bae4ddcb0 (patch) | |
tree | 8356f6b885dd8db65516db9b98157dfcb7f6919c | |
parent | be1da028a13ee95e094e9cbae8cc09ea3ef239fe (diff) | |
download | frameworks_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-x | telephony/java/com/android/internal/telephony/gsm/GSMPhone.java | 3 |
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); } |