Saxon

Test n_string_single_doublequote crashes with java.lang.StringIndexOutOfBoundsException: String index out of range: 1

The test contains a single-character string, a double quote character.

Solution: in JSONParser.readToken, case '"', test the value of "position" at the start of the loop instead of the end.

Test n_structure_100000_opening_arrays crashes with a StackOverflowError because arrays are too deeply nested. A JSON parser is allowed to impose limits, but we should try to fail cleanly.

Solution: wrap recursive calls of parseConstruct() in a try/catch that catches StackOverflowError, and produce a "too deeply nested" diagnostic.

Test -t:n_string_escaped_emoji crashes with "string index out of range". This test contains a backslash followed by a non-BMP character.

The crash actually occurs while reporting the syntax error. "Unknown escape sequence \X" where X is the high surrogate part of the non-BMP character.

The standard error listener tries to avoid putting non-Latin characters in error messages and is attempting to expand the character. It detects the presence of a non-BMP character and attempts to convert the string to a GeneralUnicodeString; this calls StringValue.expand() which trusts the supplied string to be well-formed and crashes because it contains an unpaired surrogate.

Decided to fix this at the point we generate the error message (in JsonParser and JsonReceiver)

(Spam deleted)