From 37e169cd1429c76a0223d8fdd77622ead718bbff Mon Sep 17 00:00:00 2001
From: satok <satok@google.com>
Date: Fri, 11 May 2012 11:57:48 +0900
Subject: [PATCH] Add a safety net to set range span in the spell checker

Bug: 6476578
Change-Id: I60ad82f80770dcb67fd9663dac64dc81634d7014
---
 core/java/android/widget/SpellChecker.java | 14 ++++++++++++--
 1 file changed, 12 insertions(+), 2 deletions(-)

diff --git a/core/java/android/widget/SpellChecker.java b/core/java/android/widget/SpellChecker.java
index ebf8a4a8f0dc3..6584384ee7806 100644
--- a/core/java/android/widget/SpellChecker.java
+++ b/core/java/android/widget/SpellChecker.java
@@ -473,8 +473,16 @@ public class SpellChecker implements SpellCheckerSessionListener {
         private Object mRange = new Object();
 
         public void parse(int start, int end) {
-            if (end > start) {
-                setRangeSpan((Editable) mTextView.getText(), start, end);
+            final int max = mTextView.length();
+            final int parseEnd;
+            if (end > max) {
+                Log.w(TAG, "Parse invalid region, from " + start + " to " + end);
+                parseEnd = max;
+            } else {
+                parseEnd = end;
+            }
+            if (parseEnd > start) {
+                setRangeSpan((Editable) mTextView.getText(), start, parseEnd);
                 parse();
             }
         }
@@ -612,6 +620,8 @@ public class SpellChecker implements SpellCheckerSessionListener {
                         break;
                     }
                     if (spellCheckEnd <= spellCheckStart) {
+                        Log.w(TAG, "Trying to spellcheck invalid region, from "
+                                + start + " to " + end);
                         break;
                     }
                     if (createSpellCheckSpan) {