summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorWink Saville <wink@google.com>2010-12-09 13:50:28 -0800
committerAndroid Code Review <code-review@android.com>2010-12-09 13:50:28 -0800
commit19b23afadf1053a8e06cb3444d9cdae3405ad9a1 (patch)
tree599b7aa3f4115638ae59442a220cae7b887760de
parent7f7474ddd6170b68b8b58cc03b75df85c96f08f2 (diff)
parentcc7605ce61c27e3688e33dbf2bcafd1c12788784 (diff)
downloadbase-19b23afadf1053a8e06cb3444d9cdae3405ad9a1.tar.gz
Merge "Fix for phone app crash in Icc Card."
-rw-r--r--telephony/java/com/android/internal/telephony/IccCard.java14
1 files changed, 13 insertions, 1 deletions
diff --git a/telephony/java/com/android/internal/telephony/IccCard.java b/telephony/java/com/android/internal/telephony/IccCard.java
index 90f9e8cad111..e270ce918904 100644
--- a/telephony/java/com/android/internal/telephony/IccCard.java
+++ b/telephony/java/com/android/internal/telephony/IccCard.java
@@ -487,6 +487,12 @@ public abstract class IccCard {
CommandsInterface.SERVICE_CLASS_DATA +
CommandsInterface.SERVICE_CLASS_FAX;
+ if (!mPhone.mIsTheCurrentActivePhone) {
+ Log.e(mLogTag, "Received message " + msg + "[" + msg.what
+ + "] while being destroyed. Ignoring.");
+ return;
+ }
+
switch (msg.what) {
case EVENT_RADIO_OFF_OR_NOT_AVAILABLE:
mState = null;
@@ -626,7 +632,13 @@ public abstract class IccCard {
index = mIccCardStatus.getGsmUmtsSubscriptionAppIndex();
}
- IccCardApplication app = mIccCardStatus.getApplication(index);
+ IccCardApplication app;
+ if (index >= 0 && index < IccCardStatus.CARD_MAX_APPS) {
+ app = mIccCardStatus.getApplication(index);
+ } else {
+ Log.e(mLogTag, "[IccCard] Invalid Subscription Application index:" + index);
+ return IccCard.State.ABSENT;
+ }
if (app == null) {
Log.e(mLogTag, "[IccCard] Subscription Application in not present");