Hacktoberfest 2026: los issues que los mantenedores marcaron para octubre, abiertos y aptos para principiantes. Explorar issues de Hacktoberfest

DecodeBytes decodes in place and overwrites the caller's ciphertext (non-GCM modes)

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

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
68/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Área
cryptography, testing

Línea de trabajo

Empieza con TDECFormattedCipher.DecodeBytes, donde Result := Source hace que ambos arrays compartan memoria. Solo la rama cmGCM llama a SetLength, así que copia la entrada en Result antes de Decode para todos los modos. La corrección secundaria está en TestTFormat_UU.TestIsValidTypeless, donde @SrcBuf debe cambiar a @SrcBuf[0]. Está terminado cuando el array de entrada del llamador no cambia después de decodificar en todos los modos y las pruebas pasan en Win64.

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

Descripción

TDECFormattedCipher.DecodeBytes overwrites the caller's input array for every mode except GCM.

Cause

DecodeBytes starts with Result := Source. For dynamic arrays this only copies the reference, so Result and Source point to the same memory. Only for cmGCM is SetLength(Result, Length(Source)) called, which creates a separate copy. For all other modes Decode(Source[0], Result[0], ...) writes the plaintext straight into the caller's array.

Effect

After Plain := Cipher.DecodeBytes(CipherText), CipherText contains the plaintext too. Code that decodes the same buffer twice, or keeps the ciphertext for later use (logging, retry, comparing), silently works on wrong data.

Repro

Cipher := TCipher_Twofish.Create;
Cipher.Mode := cmCBCx;
Cipher.Init(Key, IV, $FF);
Original := Copy(Source);
Plain := Cipher.DecodeBytes(Source);
// Source is now equal to Plain instead of Original

I came across this in the ChaCha20 speed tests on development_chacha20poly1305, which decode the same buffer once per CPU mode. Every decode after the first one failed.

Side note

TestTFormat_UU.TestIsValidTypeless fails now and then on Win64. It uses p := @SrcBuf, which points at the array variable instead of the data, so IsValid checks the bytes of a heap pointer. It should be @SrcBuf[0]. I fix that in the same PR.

Lenguaje dominante
HTML
Estrellas
286
Forks
78
Merge medio
1 d 4 h
PR fusionados (30 d)
4

Preparar el entorno

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 MHumm/DelphiEncryptionCompendium

Todos los issues de MHumm/DelphiEncryptionCompendium

Issues similares

Más issues de Cryptography

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.