From: Alex Rousskov <rousskov@measurement-factory.com>
Date: Thu, 29 Jun 2017 13:36:45 +0000 (+1200)
Subject: Fix message packing error handling in mgr and snmp SMP Forwarders.
X-Git-Tag: SQUID_4_0_21~7
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=009d3b5bab674d6321865d5bb25d5cd319b800f2;p=thirdparty%2Fsquid.git

Fix message packing error handling in mgr and snmp SMP Forwarders.

A missing "return" resulted in Forwarders sending garbage (or worse) to
Coordinator.
---

diff --git a/src/ipc/Forwarder.cc b/src/ipc/Forwarder.cc
index 13e051fff3..58a72915fa 100644
--- a/src/ipc/Forwarder.cc
+++ b/src/ipc/Forwarder.cc
@@ -62,6 +62,7 @@ Ipc::Forwarder::start()
         // assume the pack() call failed because the message did not fit
         // TODO: add a more specific exception?
         handleError();
+        return;
     }
 
     SendMessage(Ipc::Port::CoordinatorAddr(), message);