`CBORParser.getText(Writer)` writes stale text for numeric tokens, NPE for binary

Cerrado Apto para principiantes
  • ¿Algo no está bien?
#844 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 1 día

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
84/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
java
Área
backend-api-design

Línea de trabajo

Comienza localizando CBORParser.getText(Writer) y compáralo con getText() y la corrección de Smile propuesta en #835. Añade cobertura de regresión para los tokens numéricos y VALUE_EMBEDDED_OBJECT usando la reproducción proporcionada, verificando que la salida del writer y las longitudes devueltas coincidan con el comportamiento esperado sin una excepción.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

cbor

Same problem as #835 (Smile), in CBORParser.getText(Writer):

  1. For numeric tokens, it writes the contents of _textBuffer (or _sharedString). CBOR never decodes numbers into _textBuffer, so whatever text was last decoded gets written, usually the previous String value. The return value is that text's length.
  2. For VALUE_EMBEDDED_OBJECT (binary), t.asCharArray() returns null, so writer.write(null) throws NullPointerException. getText() returns null for the same token.

getText() handles both cases correctly.

Reproduction
CBORFactory f = new CBORFactory();
ByteArrayOutputStream bo = new ByteArrayOutputStream();
try (JsonGenerator g = f.createGenerator(bo)) {
    g.writeStartArray();
    g.writeString("abcdefghijklmnopqrstuvwxyz0123456789");
    g.writeNumber(42);
    g.writeNumber(1.25);
    g.writeNumber(new BigInteger("123456789012345678901234567890"));
    g.writeBinary(new byte[] { 1, 2, 3 });
    g.writeEndArray();
}
try (JsonParser p = f.createParser(bo.toByteArray())) {
    JsonToken t;
    while ((t = p.nextToken()) != null) {
        if (t.isScalarValue()) {
            StringWriter w = new StringWriter();
            int n = p.getText(w);
            System.out.println(t + ": getText()=" + p.getText() + ", getText(Writer)=" + w + " (" + n + ")");
        }
    }
}

Observed (2.x; same code on 2.21), for both byte[] and InputStream input:

Token getText() getText(Writer)
VALUE_STRING "abcdefghij..." "abcdefghij..." (36)
VALUE_NUMBER_INT 42 "42" "abcdefghij..." (36)
VALUE_NUMBER_FLOAT 1.25 "1.25" "abcdefghij..." (36)
VALUE_NUMBER_INT (BigInteger) "1234567890..." "abcdefghij..." (36)
VALUE_EMBEDDED_OBJECT null NullPointerException
Suggested fix

As in #835: handle VALUE_STRING (and FIELD_NAME) directly, and fall back to super.getText(writer) for everything else. jackson-core's default goes through getText() and writes nothing for null.

Lenguaje dominante
Java
Estrellas
348
Forks
159
Merge medio
1 d 17 h
PR fusionados (30 d)
45

Preparar el entorno

Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de FasterXML/jackson-dataformats-binary

Todos los issues de FasterXML/jackson-dataformats-binary

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.