From d903014c1f42884531372f18a754e5d6ff7366a7 Mon Sep 17 00:00:00 2001 From: Libin Tang Date: Mon, 27 Jul 2009 16:36:12 -0500 Subject: [PATCH] Fix crash while dialing "*228;1" --- .../internal/telephony/cdma/CDMAPhone.java | 37 ++++++++++--------- 1 file changed, 19 insertions(+), 18 deletions(-) diff --git a/telephony/java/com/android/internal/telephony/cdma/CDMAPhone.java b/telephony/java/com/android/internal/telephony/cdma/CDMAPhone.java index 28ce6a51cea0f..0d46777101e9e 100755 --- a/telephony/java/com/android/internal/telephony/cdma/CDMAPhone.java +++ b/telephony/java/com/android/internal/telephony/cdma/CDMAPhone.java @@ -1338,25 +1338,26 @@ public class CDMAPhone extends PhoneBase { return isOtaSpNum; } - /** - * isOTASPNumber: checks a given number against the IS-683A OTASP dial string and carrier - * OTASP dial string. - * - * @param dialStr the number to look up. - * @return true if the number is in IS-683A OTASP dial string or carrier OTASP dial string - */ + /** + * isOTASPNumber: checks a given number against the IS-683A OTASP dial string and carrier + * OTASP dial string. + * + * @param dialStr the number to look up. + * @return true if the number is in IS-683A OTASP dial string or carrier OTASP dial string + */ @Override - public boolean isOtaSpNumber(String dialStr){ - boolean isOtaSpNum = false; - if (dialStr != null) { - isOtaSpNum = isIs683OtaSpDialStr(dialStr); - if(isOtaSpNum == false){ - isOtaSpNum = isCarrierOtaSpNum(dialStr); - } - } - if (DBG) Log.d(LOG_TAG, "isOtaSpNumber " + isOtaSpNum); - return isOtaSpNum; - } + public boolean isOtaSpNumber(String dialStr){ + boolean isOtaSpNum = false; + String dialableStr = PhoneNumberUtils.extractNetworkPortion(dialStr); + if (dialableStr != null) { + isOtaSpNum = isIs683OtaSpDialStr(dialableStr); + if (isOtaSpNum == false) { + isOtaSpNum = isCarrierOtaSpNum(dialableStr); + } + } + if (DBG) Log.d(LOG_TAG, "isOtaSpNumber " + isOtaSpNum); + return isOtaSpNum; + } @Override public int getCdmaEriIconIndex() {