Fix #1035

FasterXML · May 22, 2023 · d9b72a5 · d9b72a5
1 parent 40d733e
commit d9b72a5
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/release-notes/VERSION-2.x b/release-notes/VERSION-2.x
@@ -19,6 +19,8 @@ a pure JSON library.
 #1007: Improve error message for `StreamReadConstraints` violations
 #1015: `JsonFactory` implementations should respect `CANONICALIZE_FIELD_NAMES`
  (contributed by Carter K)
+#1035: Root cause for failing test for `testMangledIntsBytes()` in `ParserErrorHandlingTest`
+ (reported by @harsha-99)
 
 2.15.2 (not yet released)
 

diff --git a/src/main/java/com/fasterxml/jackson/core/json/UTF8DataInputJsonParser.java b/src/main/java/com/fasterxml/jackson/core/json/UTF8DataInputJsonParser.java
@@ -1083,10 +1083,9 @@ protected JsonToken _parseUnsignedNumber(int c) throws IOException
         }
         _textBuffer.setCurrentLength(outPtr);
         // As per [core#105], need separating space between root values; check here
+        _nextByte = c;
         if (_parsingContext.inRoot()) {
             _verifyRootSpace();
-        } else {
-            _nextByte = c;
         }
         // And there we have it!
         return resetInt(false, intLen);

diff --git a/src/test/java/com/fasterxml/jackson/core/read/ParserErrorHandlingTest.java b/src/test/java/com/fasterxml/jackson/core/read/ParserErrorHandlingTest.java
@@ -23,10 +23,7 @@ public void testInvalidKeywordsChars() throws Exception {
     public void testMangledIntsBytes() throws Exception {
         _testMangledNumbersInt(MODE_INPUT_STREAM);
         _testMangledNumbersInt(MODE_INPUT_STREAM_THROTTLED);
-
-        // 02-Jun-2017, tatu: Fails to fail; should check whether this is expected
-        //   (since DataInput can't do look-ahead)
-//        _testMangledNumbersInt(MODE_DATA_INPUT);
+        _testMangledNumbersInt(MODE_DATA_INPUT);
     }
 
     public void testMangledFloatsBytes() throws Exception {