From 77fd78d5c9eb0dbab96a16ac5441f33a1abaa764 Mon Sep 17 00:00:00 2001 From: Adam Ierymenko Date: Fri, 12 Jul 2013 10:13:24 -0400 Subject: Little cleanup and docs. --- node/PacketDecoder.cpp | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'node/PacketDecoder.cpp') diff --git a/node/PacketDecoder.cpp b/node/PacketDecoder.cpp index 93126e16..f7c2eef2 100644 --- a/node/PacketDecoder.cpp +++ b/node/PacketDecoder.cpp @@ -51,7 +51,8 @@ bool PacketDecoder::tryDecode(const RuntimeEnvironment *_r) if (_step == DECODE_STEP_WAITING_FOR_ORIGINAL_SUBMITTER_LOOKUP) { // This means we've already decoded, decrypted, decompressed, and // validated, and we're processing a MULTICAST_FRAME. We're waiting - // for a lookup on the frame's original submitter. + // for a lookup on the frame's original submitter. So try again and + // see if we have it. return _doMULTICAST_FRAME(_r,peer); } @@ -65,7 +66,8 @@ bool PacketDecoder::tryDecode(const RuntimeEnvironment *_r) } else { // Unencrypted is tolerated in case we want to run this on // devices where squeezing out cycles matters. HMAC is - // what's really important. + // what's really important. But log it in debug to catch any + // packets being mistakenly sent in the clear. TRACE("ODD: %s from %s(%s) wasn't encrypted",Packet::verbString(verb()),source().toString().c_str(),_remoteAddress.toString().c_str()); } @@ -108,6 +110,7 @@ bool PacketDecoder::tryDecode(const RuntimeEnvironment *_r) return true; } } else { + _step = DECODE_STEP_WAITING_FOR_SENDER_LOOKUP; _r->sw->requestWhois(source()); return false; } -- cgit v1.2.3