commit 005b2e29742d7b9f3ecc505a50b11d91bd44c818
author Mindy Pereira <mindyp@google.com> Tue Nov 01 10:26:58 2011 -0700
committer Mindy Pereira <mindyp@google.com> Tue Nov 01 10:26:58 2011 -0700
tree a272485672af82c6450252aa6171bdd40eec1c42
parent 11fc59a03a7ca177fd6f16d1e68fa28509e30fba

Protect against negative offsets when calling replace.

Only seems to happen via the monkey.

Fixes bug:5541885 java.lang.ArrayIndexOutOfBoundsException: length=122; index=-5

Change-Id: Iba684c53982f5363990947d2b5b9d7544c952d85

src/com/android/ex/chips/RecipientEditTextView.java

diff --git a/src/com/android/ex/chips/RecipientEditTextView.java b/src/com/android/ex/chips/RecipientEditTextView.java
index 4810087..ed73d71 100644
--- a/src/com/android/ex/chips/RecipientEditTextView.java
+++ b/src/com/android/ex/chips/RecipientEditTextView.java
@@ -1426,7 +1426,7 @@
         Editable editable = getText();
         QwertyKeyListener.markAsReplaced(editable, start, end, "");
         CharSequence chip = createChip(entry, false);
-        if (chip != null) {
+        if (chip != null && start >= 0 && end >= 0) {
             editable.replace(start, end, chip);
         }
         sanitizeBetween();