From f6e4ee9a4aafa9624c0693986db075980dd27195 Mon Sep 17 00:00:00 2001 From: woogerboy21 Date: Wed, 9 Nov 2016 08:40:18 -0500 Subject: [PATCH] Fix #2270 (#2271) Fix #2270 Check userinfo first before trying to use it. --- common/server_protocolhandler.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/common/server_protocolhandler.cpp b/common/server_protocolhandler.cpp index 671cfda0..f2d35a10 100644 --- a/common/server_protocolhandler.cpp +++ b/common/server_protocolhandler.cpp @@ -387,7 +387,7 @@ void Server_ProtocolHandler::pingClockTimeout() if (timeRunning - lastDataReceived > server->getMaxPlayerInactivityTime()) prepareDestroy(); - if (QString::fromStdString(userInfo->privlevel()).toLower() == "none") { + if (!userInfo || QString::fromStdString(userInfo->privlevel()).toLower() == "none") { if ((server->getIdleClientTimeout() > 0) && (idleClientWarningSent)) { if (timeRunning - lastActionReceived > server->getIdleClientTimeout()) { prepareDestroy(); @@ -472,7 +472,7 @@ Response::ResponseCode Server_ProtocolHandler::cmdLogin(const Command_Login &cmd } // limit the number of non-privileged users that can connect to the server based on configuration settings - if (QString::fromStdString(userInfo->privlevel()).toLower() == "none") { + if (!userInfo || QString::fromStdString(userInfo->privlevel()).toLower() == "none") { if (server->getMaxUserLimitEnabled()) { if (server->getUsersCount() > server->getMaxUserTotal()) { qDebug() << "Max Users Total Limit Reached, please increase the max_users_total setting.";