Fix #173

Author: cowtowncoder

release-notes/CREDITS

@@ -35,3 +35,7 @@ Steve van Loben Sels
 Shay Banon
   * Reported #145: NPE at BytesToNameCanonicalizer
    (2.4.2)
+
+Alex Soto: (lordofthejars@github)
+  * Reported #173: An exception is thrown for a valid JsonPointer expression
+

release-notes/VERSION

@@ -17,6 +17,9 @@ json-specificity (naming due to historical reasons).
 
 #157: ArrayIndexOutOfBoundsException: 200 on numbers with more than 200 digits.
  (reported by Lars P, larsp@github)
+#173: An exception is thrown for a valid JsonPointer expression
+ (reported by Alex S)
+
 - Fix `JsonGenerator.setFeatureMask()` to better handle dynamic changes.
 
 2.4.3 (02-Oct-2014)

src/main/java/com/fasterxml/jackson/core/JsonPointer.java

@@ -190,7 +190,7 @@ private final static int _parseIndex(String str) {
             return -1;
         }
         for (int i = 0; i < len; ++i) {
-            char c = str.charAt(i++);
+            char c = str.charAt(i);
             if (c > '9' || c < '0') {
                 return -1;
             }

src/test/java/com/fasterxml/jackson/core/TestJsonPointer.java

@@ -36,6 +36,12 @@ public void testSimplePath() throws Exception
         assertEquals(-1, ptr.getMatchingIndex());
     }
 
+    public void testWonkyNumber173() throws Exception
+    {
+        JsonPointer ptr = JsonPointer.compile("/1e0");
+        assertFalse(ptr.matches());
+    }
+    
     public void testQuotedPath() throws Exception
     {
         final String INPUT = "/w~1out/til~0de/a~1b";