Fix #176 for 2.3

FasterXML · Dec 31, 2014 · 5a7f8d7 · 5a7f8d7
1 parent 1a9cc45
commit 5a7f8d7
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/release-notes/VERSION b/release-notes/VERSION
@@ -3,6 +3,7 @@ Version: 2.3.5 (xx-xxx-2014)
 
 #152: Exception for property names longer than 256k
 #173: An exception is thrown for a valid JsonPointer expression
+#176: `JsonPointer` should not consider "00" to be valid index
 
 ------------------------------------------------------------------------
 === History: ===

diff --git a/src/main/java/com/fasterxml/jackson/core/JsonPointer.java b/src/main/java/com/fasterxml/jackson/core/JsonPointer.java
@@ -184,13 +184,21 @@ public JsonPointer tail() {
 
     private final static int _parseIndex(String str) {
         final int len = str.length();
-        // [Issue#133]: beware of super long indexes; assume we never
+        // [core#133]: beware of super long indexes; assume we never
         // have arrays over 2 billion entries so ints are fine.
         if (len == 0 || len > 10) {
             return -1;
         }
-        for (int i = 0; i < len; ++i) {
-            char c = str.charAt(i);
+        // [core#176]: no leading zeroes allowed
+        char c = str.charAt(0);
+        if (c <= '0') {
+            return (len == 1 && c == '0') ? 0 : -1;
+        }
+        if (c > '9') {
+            return -1;
+        }
+        for (int i = 1; i < len; ++i) {
+            c = str.charAt(i);
             if (c > '9' || c < '0') {
                 return -1;
             }