2
0
Эх сурвалжийг харах

Don't assume roots validated the identity, just in case they did not.

Adam Ierymenko 4 жил өмнө
parent
commit
9cfb807fcb

+ 4 - 1
node/IncomingPacket.cpp

@@ -511,7 +511,10 @@ bool IncomingPacket::_doOK(const RuntimeEnvironment *RR,void *tPtr,const SharedP
 		case Packet::VERB_WHOIS:
 			if (RR->topology->isUpstream(peer->identity())) {
 				const Identity id(*this,ZT_PROTO_VERB_WHOIS__OK__IDX_IDENTITY);
-				RR->sw->doAnythingWaitingForPeer(tPtr,RR->topology->addPeer(tPtr,SharedPtr<Peer>(new Peer(RR,RR->identity,id))));
+				// Good idea to locally validate here even if roots are doing so. In a truly distributed
+				// system there should not be single points of failure for global trust assertions.
+				if (id.locallyValidate())
+					RR->sw->doAnythingWaitingForPeer(tPtr,RR->topology->addPeer(tPtr,SharedPtr<Peer>(new Peer(RR,RR->identity,id))));
 			}
 			break;