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

OOB read in bencode_decode: strtoll called on non-NUL-terminated buffer

Abierto Apto para principiantes
#2,199 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 3 días

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
67/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
c
Área
security

Línea de trabajo

Comienza en lib/bencode.c en __bencode_decode_integer alrededor de la línea 448, luego lee cómo bencode_decode_expect recibe el buffer de ng_client.c alrededor de la línea 176. Comprueba cómo el analizador de enteros maneja la longitud de entrada y la terminación NUL. Done significa que una entrada malformada como i se rechaza sin leer más allá del buffer, mientras los enteros válidos todavía se decodifican.

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

Descripción

__bencode_decode_integer (lib/bencode.c:448) calls strtoll(s, &convend, 10) on the length-delimited input buffer. strtoll requires NUL termination but bencode buffers are not NUL-terminated.

When the integer payload begins with whitespace (e.g. i , hex 69 20), strtoll skips it and scans past the buffer boundary looking for digits, causing a heap-buffer-overflow read.

Reachable from: ng_client.c receives into a 4096-byte arena buffer (arena_alloc, no NUL terminator) and passes it to bencode_decode_expect at line 176. A remote NG peer response with a valid cookie and malformed bencode body triggers the read.

Impact: Out-of-bounds read, process crash (DoS). No write primitive.

Fix: Copy the integer text into a stack buffer with NUL termination before calling strtoll, or bound the scan manually without strtoll.

Reproduce:

# 2-byte payload: "i " (0x69 0x20)
echo -n "i " | base64
# aSA=

Found by Bright Security using automated fuzz testing.

Lenguaje dominante
Perl
Estrellas
992
Forks
459
Métricas de merge de PR
Sin PR fusionados en 30 d

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 sipwise/rtpengine

Todos los issues de sipwise/rtpengine

Issues similares

Más issues de Perl

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.