Lack of signedPeerRecord Validation in C++ IdentifyMessageProcessor Leads to Address Injection
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- cpp
- Área
- networking, security
Línea de trabajo
Comienza con src/protocol/identify/identify_msg_processor.cpp, especialmente con IdentifyMessageProcessor::identifyReceived, y compara el arreglo de Go libp2p mencionado para el comportamiento de validación. Se considera terminado cuando se verifican los envelopes de signedPeerRecord, se comprueban ambas relaciones de PeerId con respecto a la conexión y al registro, y las direcciones certificadas solo se aceptan cuando todas las comprobaciones tienen éxito.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
The C++ implementation of libp2p’s Identify protocol (IdentifyMessageProcessor) does not validate or process the signedPeerRecord field in Identify messages.
This allows malicious peers to inject or forward third-party signed peer records, leading to address poisoning and potential identity spoofing, similar to previously fixed vulnerabilities in Go and JS libp2p implementations.
Expected behavior
When an Identify message includes a signedPeerRecord:
The envelope signature must be verified.
The public key inside the envelope must derive a PeerId equal to the connection’s remotePeerId.
The PeerRecord.peerId must also match this derived PeerId.
Only if all checks succeed should the certified addresses be accepted and stored.
Actual behavior
The current C++ code (IdentifyMessageProcessor::identifyReceived) only verifies the publicKey field.
It does not parse or validate signedPeerRecord.
A malicious peer can forward another peer’s valid signed record, causing addresses of an unrelated peer to be stored.
Relevant log output
Possible Solution
Go libp2p issue: Identity protocol accepts signed peer records without validation
Go libp2p fix: x
The revelent_file
Version
No response
Would you like to work on fixing this bug ?
Yes
- Lenguaje dominante
- C++
- Estrellas
- 492
- Forks
- 130
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
Aún no hemos revisado los archivos de configuración de este proyecto. Empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 libp2p/cpp-libp2p
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
libp2p/cpp-libp2p#351 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 48/100
libp2p/cpp-libp2p#347 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
libp2p/cpp-libp2p#341 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
libp2p/cpp-libp2p#324 · 1 comentario · 1 reacción ·
-
Build is failing all the timeAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 30/100
libp2p/cpp-libp2p#313 · 1 comentario ·
Todos los issues de libp2p/cpp-libp2p
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Icinga/icinga2#11058 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
component: split-view platform: windows
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
zen-browser/desktop#15616 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
-
area/ysql kind/bug priority/medium status/awaiting-triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
yugabyte/yugabyte-db#34415 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
WayfireWM/wayfire#3148 · 1 comentario ·
Los mantenedores suelen responder en 1 día