diff options
author | Wink Saville <wink@google.com> | 2010-12-09 13:50:28 -0800 |
---|---|---|
committer | Android Code Review <code-review@android.com> | 2010-12-09 13:50:28 -0800 |
commit | 19b23afadf1053a8e06cb3444d9cdae3405ad9a1 (patch) | |
tree | 599b7aa3f4115638ae59442a220cae7b887760de /telephony/java | |
parent | 7f7474ddd6170b68b8b58cc03b75df85c96f08f2 (diff) | |
parent | cc7605ce61c27e3688e33dbf2bcafd1c12788784 (diff) | |
download | frameworks_base-19b23afadf1053a8e06cb3444d9cdae3405ad9a1.zip frameworks_base-19b23afadf1053a8e06cb3444d9cdae3405ad9a1.tar.gz frameworks_base-19b23afadf1053a8e06cb3444d9cdae3405ad9a1.tar.bz2 |
Merge "Fix for phone app crash in Icc Card."
Diffstat (limited to 'telephony/java')
-rw-r--r-- | telephony/java/com/android/internal/telephony/IccCard.java | 14 |
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 90f9e8c..e270ce9 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"); |