From 4219797afc2f5369c27cf7be752e9bc3bdb8057d Mon Sep 17 00:00:00 2001 From: jsh Date: Mon, 19 Oct 2009 16:11:37 -0700 Subject: Log exception if we hit an error parsing an incoming SMS message. This will help debug b/2197566. We should have been logging the error anyway. --- telephony/java/com/android/internal/telephony/SMSDispatcher.java | 1 + .../java/com/android/internal/telephony/cdma/CdmaSMSDispatcher.java | 1 + 2 files changed, 2 insertions(+) diff --git a/telephony/java/com/android/internal/telephony/SMSDispatcher.java b/telephony/java/com/android/internal/telephony/SMSDispatcher.java index d26a092..53c0bef 100644 --- a/telephony/java/com/android/internal/telephony/SMSDispatcher.java +++ b/telephony/java/com/android/internal/telephony/SMSDispatcher.java @@ -303,6 +303,7 @@ public abstract class SMSDispatcher extends Handler { notifyAndAcknowledgeLastIncomingSms(handled, result, null); } } catch (RuntimeException ex) { + Log.e(TAG, "Exception dispatching message", ex); notifyAndAcknowledgeLastIncomingSms(false, Intents.RESULT_SMS_GENERIC_ERROR, null); } diff --git a/telephony/java/com/android/internal/telephony/cdma/CdmaSMSDispatcher.java b/telephony/java/com/android/internal/telephony/cdma/CdmaSMSDispatcher.java index 7baf770..d895cd4 100644 --- a/telephony/java/com/android/internal/telephony/cdma/CdmaSMSDispatcher.java +++ b/telephony/java/com/android/internal/telephony/cdma/CdmaSMSDispatcher.java @@ -100,6 +100,7 @@ final class CdmaSMSDispatcher extends SMSDispatcher { // If sms is null, means there was a parsing error. if (smsb == null) { + Log.e(TAG, "dispatchMessage: message is null"); return Intents.RESULT_SMS_GENERIC_ERROR; } -- cgit v1.1