epmd: a client that closes early crashes the server on the PORT_PLEASE2 and NAMES responses
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
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- erlang
- Área
- backend
Línea de trabajo
Lee libs/avm_network/src/epmd.erl, centrándote en las ramas de process_req/3 para ?PORT_PLEASE2_REQ y ?NAMES_REQ, y compara cómo gestionan el envío con la rama ALIVE2 modificada por #2345. Confirma cómo se gestionan los fallos de envío y que el servidor conserva sus nodos registrados; el trabajo estará terminado cuando cualquiera de las respuestas pueda fallar al enviarse sin que el epmd gen_server se caiga.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Affected: release-0.7 at ca1583068, libs/avm_network/src/epmd.erl,
process_req/3 for ?PORT_PLEASE2_REQ and ?NAMES_REQ.
#2345 fixes the same problem for the ALIVE2 response only.
What happens
The responses are sent with ok = socket:send(Socket, ...).
When the client has already closed the connection, socket:send/2 does not
return ok (on release-0.7 it returns {ok, Data} for a closed connection,
with #2345 it returns {error, closed} or {error, epipe}). The badmatch
crashes the epmd gen_server, and every registered node is forgotten.
How to reproduce
Start the epmd, then from any client:
{ok, S} = socket:open(inet, stream, tcp),
ok = socket:setopt(S, {socket, linger}, #{onoff => true, linger => 0}),
ok = socket:connect(S, #{family => inet, addr => loopback, port => 4369}),
ok = socket:send(S, <<2:16, 122, "x">>), % PORT_PLEASE2_REQ for node "x"
ok = socket:close(S). % reset, no reply is read
The server's reply fails and the epmd process crashes. The race is easiest to
hit with a slow server, for example under valgrind.
Suggested fix
Handle {error, _} like the ALIVE2 branch does after #2345: close the socket
and keep the state.
- Lenguaje dominante
- Erlang
- Estrellas
- 2.3k
- Forks
- 171
- Merge medio
- 4 d 11 h
- PR fusionados (30 d)
- 20
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de atomvm/AtomVM
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 3 días
-
lwIP sockets: a failed listen or connect leaves the socket lock held, a later close hangs foreverAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 3 días
Todos los issues de atomvm/AtomVM
Issues similares
-
needs-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
coder/claudecode.nvim#587 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
digitalbazaar/http-client#60 ·
-
component/fhir type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
OpenConceptLab/ocl_issues#3026 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
fivetran/dbt_recurly#55 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
topmonks/hlidac-shopu#3687 ·
Los mantenedores suelen responder en 1 día