Lack of signedPeerRecord Validation in C++ IdentifyMessageProcessor Leads to Address Injection
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Bug
- Chiarezza
- Specificata chiaramente
- Stato di attività
- Ferma
- Stack tecnologico
- cpp
- Ambito
- networking, security
Direzione di ricerca
Inizia con src/protocol/identify/identify_msg_processor.cpp, in particolare con IdentifyMessageProcessor::identifyReceived, e confronta la correzione di Go libp2p indicata per quanto riguarda il comportamento di validazione. Il lavoro è completato quando gli envelope signedPeerRecord vengono verificati, entrambe le relazioni PeerId vengono controllate rispetto alla connessione e al record e gli indirizzi certificati vengono accettati solo quando tutti i controlli hanno esito positivo.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
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
- Lingua principale
- C++
- Stelle
- 492
- Fork
- 130
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di libp2p/cpp-libp2p
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
libp2p/cpp-libp2p#351 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 48/100
libp2p/cpp-libp2p#347 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
libp2p/cpp-libp2p#341 ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 45/100
libp2p/cpp-libp2p#324 · 1 commento · 1 reazione ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 30/100
libp2p/cpp-libp2p#313 · 1 commento ·
Tutte le issue di libp2p/cpp-libp2p
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
google/libultrahdr#485 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
godotengine/godot#123776 ·
-
bug
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 60/100
-
good first issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
-
good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
ros2/common_interfaces#344 ·