Add test to show hexadecimal not supported - but fix issue in UTF8DataInputJsonParser (#786)

Author: pjfanning

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

@@ -1051,11 +1051,13 @@ protected JsonToken _parseUnsignedNumber(int c) throws IOException
         int outPtr;
 
         // One special case: if first char is 0, must not be followed by a digit.
-        // Gets bit tricky as we only want to retain 0 if it's the full value
+        // Gets a bit tricky as we only want to retain 0 if it's the full value
         if (c == INT_0) {
             c = _handleLeadingZeroes();
             if (c <= INT_9 && c >= INT_0) { // skip if followed by digit
                 outPtr = 0;
+            } else if (c == 'x' || c == 'X') {
+                return _handleInvalidNumberStart(c, false);
             } else {
                 outBuf[0] = '0';
                 outPtr = 1;

src/test/java/com/fasterxml/jackson/core/read/NonStandardNumberParsingTest.java

@@ -20,19 +20,77 @@ protected JsonFactory jsonFactory() {
         return JSON_F;
     }
 
+    /**
+     * The format "0xc0ffee" is not valid JSON, so this should fail
+     */
+    public void testHexadecimal() throws Exception {
+        for (int mode : ALL_MODES) {
+            try (JsonParser p = createParser(mode, " 0xc0ffee ")) {
+                p.nextToken();
+                fail("Should not pass");
+            } catch (JsonParseException e) {
+                verifyException(e, "Unexpected character ('x'");
+            }
+        }
+    }
+
+    public void testHexadecimalBigX() throws Exception {
+        for (int mode : ALL_MODES) {
+            try (JsonParser p = createParser(mode, " 0XC0FFEE ")) {
+                p.nextToken();
+                fail("Should not pass");
+            } catch (JsonParseException e) {
+                verifyException(e, "Unexpected character ('x'");
+            }
+        }
+    }
+
+    public void testNegativeHexadecimal() throws Exception {
+        for (int mode : ALL_MODES) {
+            try (JsonParser p = createParser(mode, " -0xc0ffee ")) {
+                p.nextToken();
+                fail("Should not pass");
+            } catch (JsonParseException e) {
+                verifyException(e, "Unexpected character ('x'");
+            }
+        }
+    }
+
+    //JSON does not allow numbers to have f or d suffixes
+    public void testFloatMarker() throws Exception {
+        for (int mode : ALL_MODES) {
+            try (JsonParser p = createParser(mode, " -0.123f ")) {
+                p.nextToken();
+                fail("Should not pass");
+            } catch (JsonParseException e) {
+                verifyException(e, "Unexpected character ('f'");
+            }
+        }
+    }
+
+    //JSON does not allow numbers to have f or d suffixes
+    public void testDoubleMarker() throws Exception {
+        for (int mode : ALL_MODES) {
+            try (JsonParser p = createParser(mode, " -0.123d ")) {
+                p.nextToken();
+                fail("Should not pass");
+            } catch (JsonParseException e) {
+                verifyException(e, "Unexpected character ('d'");
+            }
+        }
+    }
+
     /**
      * The format ".NNN" (as opposed to "0.NNN") is not valid JSON, so this should fail
      */
     public void testLeadingDotInDecimal() throws Exception {
         for (int mode : ALL_MODES) {
-            JsonParser p = createParser(mode, " .123 ");
-            try {
+            try (JsonParser p = createParser(mode, " .123 ")) {
                 p.nextToken();
                 fail("Should not pass");
             } catch (JsonParseException e) {
                 verifyException(e, "Unexpected character ('.'");
             }
-            p.close();
         }
     }