From 377024031981694beefdbc23cdf807da486cdf18 Mon Sep 17 00:00:00 2001
From: Kai Koehne <kai.koehne@theqtcompany.com>
Date: Thu, 29 Jan 2015 11:18:33 +0100
Subject: RemoteServer: Exit early if authorization failed

There's no point in listening to the socket after the
authorization failed. It will then be closed by the client,
anyway.

Change-Id: I26eb2023e08ac3b16ecb894a89ffa0bfeddc62b0
Reviewed-by: Niels Weber <niels.weber@theqtcompany.com>
Reviewed-by: Karsten Heimrich <karsten.heimrich@theqtcompany.com>
---
 src/libs/installer/remoteserverconnection.cpp | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

(limited to 'src/libs')

diff --git a/src/libs/installer/remoteserverconnection.cpp b/src/libs/installer/remoteserverconnection.cpp
index 7bd4a45e8..c942b19e3 100644
--- a/src/libs/installer/remoteserverconnection.cpp
+++ b/src/libs/installer/remoteserverconnection.cpp
@@ -87,8 +87,10 @@ void RemoteServerConnection::run()
             stream >> key;
             sendData(stream, (authorized = (key == m_authorizationKey)));
             socket.flush();
-            if (!authorized)
+            if (!authorized) {
                 socket.close();
+                return;
+            }
         } else if (authorized) {
             if (command.isEmpty())
                 continue;
-- 
cgit v1.2.3