Use the return from BufferedSocket::ProcessRead() and don't just assume its true

2 files changed, 4 insertions, 4 deletions

diff --git a/modules/extra/m_proxyscan.cpp b/modules/extra/m_proxyscan.cpp
index b77da2d3d..e14f82df6 100644
--- a/modules/extra/m_proxyscan.cpp
+++ b/modules/extra/m_proxyscan.cpp
@@ -127,13 +127,13 @@ class HTTPProxyConnect : public ProxyConnect, public BufferedSocket
 
 	bool ProcessRead() anope_override
 	{
-		BufferedSocket::ProcessRead();
+		bool b = BufferedSocket::ProcessRead();
 		if (this->GetLine() == ProxyCheckString)
 		{
 			this->Ban();
 			return false;
 		}
-		return true;
+		return b;
 	}
 };
 
diff --git a/src/uplink.cpp b/src/uplink.cpp
index 085ef889b..f8841c58f 100644
--- a/src/uplink.cpp
+++ b/src/uplink.cpp
@@ -118,13 +118,13 @@ UplinkSocket::~UplinkSocket()
 
 bool UplinkSocket::ProcessRead()
 {
-	BufferedSocket::ProcessRead();
+	bool b = BufferedSocket::ProcessRead();
 	for (Anope::string buf; (buf = this->GetLine()).empty() == false;)
 	{
 		Anope::Process(buf);
 		User::QuitUsers();
 	}
-	return true;
+	return b;
 }
 
 void UplinkSocket::OnConnect()