Fix #317

Author: cowtowncoder

release-notes/CREDITS

@@ -88,3 +88,8 @@ Mike Naseef (mtnaseef@github)
   * Reported #307: JsonGenerationException: Split surrogate on writeRaw() input thrown for
    input of a certain size
    (2.7.7)
+
+Allar Haav (haav@github)
+  * Reportef #317: ArrayIndexOutOfBoundsException: 200 on floating point number with exactly
+  200-length decimal part
+   (2.7.8)

release-notes/VERSION

@@ -14,6 +14,12 @@ JSON library.
 === Releases ===
 ------------------------------------------------------------------------
 
+2.7.8 (not yet released)
+
+#317: ArrayIndexOutOfBoundsException: 200 on floating point number with exactly
+  200-length decimal part
+ (reported by Allar H)
+
 2.7.7 (27-Aug-2016)
 
 #307: JsonGenerationException: Split surrogate on writeRaw() input thrown for

src/main/java/com/fasterxml/jackson/core/json/ReaderBasedJsonParser.java

@@ -1415,6 +1415,10 @@ private final JsonToken _parseNumber2(boolean neg, int startPtr) throws IOExcept
         int fractLen = 0;
         // And then see if we get other parts
         if (c == '.') { // yes, fraction
+            if (outPtr >= outBuf.length) {
+                outBuf = _textBuffer.finishCurrentSegment();
+                outPtr = 0;
+            }
             outBuf[outPtr++] = c;
 
             fract_loop:

src/main/java/com/fasterxml/jackson/core/json/UTF8StreamJsonParser.java

@@ -1541,6 +1541,10 @@ private final JsonToken _parseFloat(char[] outBuf, int outPtr, int c,
 
         // And then see if we get other parts
         if (c == INT_PERIOD) { // yes, fraction
+            if (outPtr >= outBuf.length) {
+                outBuf = _textBuffer.finishCurrentSegment();
+                outPtr = 0;
+            }
             outBuf[outPtr++] = (char) c;
 
             fract_loop:
@@ -1615,7 +1619,7 @@ private final JsonToken _parseFloat(char[] outBuf, int outPtr, int c,
         // Ok; unless we hit end-of-input, need to push last char read back
         if (!eof) {
             --_inputPtr;
-            // As per #105, need separating space between root values; check here
+            // As per [core#105], need separating space between root values; check here
             if (_parsingContext.inRoot()) {
                 _verifyRootSpace(c);
             }

src/test/java/com/fasterxml/jackson/core/json/TestNumericValues.java

@@ -1,5 +1,8 @@
 package com.fasterxml.jackson.core.json;
 
+import java.io.ByteArrayInputStream;
+import java.io.IOException;
+import java.io.StringReader;
 import java.math.BigDecimal;
 import java.math.BigInteger;
 
@@ -550,6 +553,35 @@ public void testInvalidIntAccess() throws Exception
         jp.close();
     }
 
+    // [core#317]
+    public void testLongerFloatingPoint() throws Exception
+    {
+        StringBuilder input = new StringBuilder();
+        for (int i = 1; i < 201; i++) {
+            input.append(1);
+        }
+        input.append(".0");
+        final String DOC = input.toString();
+
+        // test out with both Reader and ByteArrayInputStream
+        JsonParser p;
+
+        p = FACTORY.createParser(new StringReader(DOC));
+        _testLongerFloat(p, DOC);
+        p.close();
+        
+        p = FACTORY.createParser(new ByteArrayInputStream(DOC.getBytes("UTF-8")));
+        _testLongerFloat(p, DOC);
+        p.close();
+    }
+
+    private void _testLongerFloat(JsonParser p, String text) throws IOException
+    {
+        assertToken(JsonToken.VALUE_NUMBER_FLOAT, p.nextToken());
+        assertEquals(text, p.getText());
+        assertNull(p.nextToken());
+    }
+
     /*
     /**********************************************************
     /* Helper methods