LEAmDNS: _parseResponse keeps every answer record of a packet on the heap, so one large answer can abort with out of memory

Abierto
  • ¿Algo no está bien?
#9,343 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
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
52/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
arduino, cpp
Área
embedded-iot

Línea de trabajo

Start in libraries/ESP8266mDNS/src/LEAmDNS_Control.cpp: trace _parseResponse through _readRRAnswer and _processAnswers, then compare with _parseQuery's record handling. The report proposes processing and freeing each response record before reading the next; done means response handling no longer retains all packet records at once while preserving probe and service query behavior. The reporter says they have a patched copy, but provides no test or captured packet.

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

Descripción

What happens

An ESP8266 (core 3.1.2, a GeekMagic SmallTV-Ultra with about 6 KB of free heap while running) restarted after
11.4 hours with an out-of-memory abort inside LEAmDNS. Our crash record, decoded with addr2line:

operator new (288 bytes) failed
  <- MDNSResponder::_readRRAnswer
  <- MDNSResponder::_parseResponse
  <- MDNSResponder::_parseMessage
  <- MDNSResponder::_process
free heap 952 bytes, largest free block 248 bytes

The lowest values recorded before, during those hours, were about 4.4 KB free and a 1.5 KB largest block. So
about 3.5 KB went within a moment. The sketch only advertises (MDNS.begin, addService, addServiceTxt, and
MDNS.update() in loop()); it sends no queries.

Why, from reading the code (master, libraries/ESP8266mDNS/src/LEAmDNS_Control.cpp)
  • _parseResponse reads answers only while a service query waits or a probe runs (line 742-743). Probing runs
    after MDNS.begin and again after every interface status change (LEAmDNS.cpp, the statusChangeCB that
    calls _restart()), for example a Wi-Fi reconnect.
  • In that state, every record of the incoming packet (ANCount + NSCount + ARCount, taken from the packet
    header) is read with _readRRAnswer into its own heap object and kept in a list (lines 764-775).
  • The list goes to _processAnswers as a whole (line 794) and is freed only after that (lines 804-810).

So the heap needed grows with the number of records in someone else's answer. A phone, TV or speaker that
answers with many records during the probe window can take several KB at once. new aborts on failure in this
core, so the library's own checks for a failed allocation never run, and the result is a restart.

What we did not measure: we have not captured the packet, and we don't know for sure that a reconnect came just
before. That part is our reading of the code.

_parseQuery handles known answers differently: it frees each record right after reading it (so one record at a
time), which is why we think the same can work for responses.

Possible fix

Handle each answer record in _parseResponse as soon as it is read, and free it before reading the next, so the
peak is one record instead of the whole packet. That means splitting _processAnswers so it can take one record
at a time. A simpler, partial alternative: skip (read past without allocating) records whose names cannot
concern this responder.

We are trying the first approach in a patched copy for our own firmware and can open a PR if this direction is
welcome.

Lenguaje dominante
C++
Estrellas
16.7k
Forks
13k
Merge medio
1 d 2 h
PR fusionados (30 d)
7

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 esp8266/Arduino

Todos los issues de esp8266/Arduino

Issues similares

Más issues de C++

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.