X-Git-Url: https://git.quassel-irc.org/?p=quassel.git;a=blobdiff_plain;f=src%2Fcore%2Fidentserver.cpp;h=c1cb5d036bc34805decceb72607f12c78b0415f5;hp=06e6cf5ad7eb557b2b68b05172930b7f269c8ff4;hb=dcc39bc640adc4dc1b326162dbe8d682cb035447;hpb=afc90943c558eaca765530fa90ff8b656dc04a99 diff --git a/src/core/identserver.cpp b/src/core/identserver.cpp index 06e6cf5a..c1cb5d03 100644 --- a/src/core/identserver.cpp +++ b/src/core/identserver.cpp @@ -1,5 +1,5 @@ /*************************************************************************** - * Copyright (C) 2005-2018 by the Quassel Project * + * Copyright (C) 2005-2019 by the Quassel Project * * devel@quassel-irc.org * * * * This program is free software; you can redistribute it and/or modify * @@ -18,53 +18,47 @@ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA. * ***************************************************************************/ -#include -#include +#include "identserver.h" + +#include #include "corenetwork.h" -#include "identserver.h" -IdentServer::IdentServer(bool strict, QObject *parent) : QObject(parent), _strict(strict), _socketId(0), _requestId(0) { - connect(&_server, SIGNAL(newConnection()), this, SLOT(incomingConnection())); - connect(&_v6server, SIGNAL(newConnection()), this, SLOT(incomingConnection())); +IdentServer::IdentServer(QObject* parent) + : QObject(parent) +{ + connect(&_server, &QTcpServer::newConnection, this, &IdentServer::incomingConnection); + connect(&_v6server, &QTcpServer::newConnection, this, &IdentServer::incomingConnection); } -IdentServer::~IdentServer() = default; - -bool IdentServer::startListening() { +bool IdentServer::startListening() +{ uint16_t port = Quassel::optionValue("ident-port").toUShort(); bool success = false; - if (_v6server.listen(QHostAddress("::1"), port)) { - quInfo() << qPrintable( - tr("Listening for identd clients on IPv6 %1 port %2") - .arg("::1") - .arg(_v6server.serverPort()) - ); + if (_v6server.listen(QHostAddress("::"), port)) { + qInfo() << qPrintable(tr("Listening for identd clients on IPv6 %1 port %2").arg("::").arg(_v6server.serverPort())); success = true; } - if (_server.listen(QHostAddress("127.0.0.1"), port)) { - success = true; + if (_server.listen(QHostAddress("0.0.0.1"), port)) { + qInfo() << qPrintable(tr("Listening for identd clients on IPv4 %1 port %2").arg("0.0.0.1").arg(_server.serverPort())); - quInfo() << qPrintable( - tr("Listening for identd clients on IPv4 %1 port %2") - .arg("127.0.0.1") - .arg(_server.serverPort()) - ); + success = true; } if (!success) { - quError() << qPrintable( - tr("Identd could not open any network interfaces to listen on! No identd functionality will be available")); + qWarning() << qPrintable(tr("Identd could not open any network interfaces to listen on! No identd functionality will be available")); } return success; } -void IdentServer::stopListening(const QString &msg) { +void IdentServer::stopListening(const QString& msg) +{ bool wasListening = false; + if (_server.isListening()) { wasListening = true; _server.close(); @@ -73,25 +67,29 @@ void IdentServer::stopListening(const QString &msg) { wasListening = true; _v6server.close(); } + if (wasListening) { if (msg.isEmpty()) - quInfo() << "No longer listening for identd clients."; + qInfo() << "No longer listening for identd clients."; else - quInfo() << qPrintable(msg); + qInfo() << qPrintable(msg); } } -void IdentServer::incomingConnection() { - auto *server = qobject_cast(sender()); +void IdentServer::incomingConnection() +{ + auto server = qobject_cast(sender()); Q_ASSERT(server); while (server->hasPendingConnections()) { - QTcpSocket *socket = server->nextPendingConnection(); - connect(socket, SIGNAL(readyRead()), this, SLOT(respond())); + QTcpSocket* socket = server->nextPendingConnection(); + connect(socket, &QIODevice::readyRead, this, &IdentServer::respond); + connect(socket, &QAbstractSocket::disconnected, socket, &QObject::deleteLater); } } -void IdentServer::respond() { - QTcpSocket *socket = qobject_cast(sender()); +void IdentServer::respond() +{ + auto* socket = qobject_cast(sender()); Q_ASSERT(socket); qint64 transactionId = _socketId; @@ -117,57 +115,73 @@ void IdentServer::respond() { Request request{socket, localPort, query, transactionId, _requestId++}; if (!success) { - request.respondError("NO-USER"); - } else if (responseAvailable(request)) { + request.respondError("INVALID-PORT"); + } + else if (responseAvailable(request)) { // success - } else if (lowestSocketId() < transactionId) { + } + else if (lowestSocketId() < transactionId) { _requestQueue.emplace_back(request); - } else { + } + else { request.respondError("NO-USER"); } } -void Request::respondSuccess(const QString &user) { - QString data = query + " : USERID : Quassel : " + user + "\r\n"; - - socket->write(data.toUtf8()); - socket->flush(); - socket->close(); +void Request::respondSuccess(const QString& user) +{ + if (socket) { + QString data = query + " : USERID : Quassel : " + user + "\r\n"; + socket->write(data.toUtf8()); + socket->flush(); + socket->close(); + } } -void Request::respondError(const QString &error) { - QString data = query + " : ERROR : " + error + "\r\n"; - - socket->write(data.toUtf8()); - socket->flush(); - socket->close(); +void Request::respondError(const QString& error) +{ + if (socket) { + QString data = query + " : ERROR : " + error + "\r\n"; + socket->write(data.toUtf8()); + socket->flush(); + socket->close(); + } } -bool IdentServer::responseAvailable(Request request) { +bool IdentServer::responseAvailable(Request request) const +{ if (!_connections.contains(request.localPort)) { return false; } - QString user = _connections[request.localPort]; - request.respondSuccess(user); + request.respondSuccess(_connections[request.localPort]); return true; } -bool IdentServer::addSocket(const CoreIdentity *identity, const QHostAddress &localAddress, quint16 localPort, - const QHostAddress &peerAddress, quint16 peerPort, qint64 socketId) { +void IdentServer::addSocket(const CoreIdentity* identity, + const QHostAddress& localAddress, + quint16 localPort, + const QHostAddress& peerAddress, + quint16 peerPort, + qint64 socketId) +{ Q_UNUSED(localAddress) Q_UNUSED(peerAddress) Q_UNUSED(peerPort) - const CoreNetwork *network = qobject_cast(sender()); - _connections[localPort] = network->coreSession()->strictCompliantIdent(identity);; + const CoreNetwork* network = qobject_cast(sender()); + _connections[localPort] = network->coreSession()->strictCompliantIdent(identity); + ; processWaiting(socketId); - return true; } - -bool IdentServer::removeSocket(const CoreIdentity *identity, const QHostAddress &localAddress, quint16 localPort, - const QHostAddress &peerAddress, quint16 peerPort, qint64 socketId) { +void IdentServer::removeSocket(const CoreIdentity* identity, + const QHostAddress& localAddress, + quint16 localPort, + const QHostAddress& peerAddress, + quint16 peerPort, + qint64 socketId) +{ Q_UNUSED(identity) Q_UNUSED(localAddress) Q_UNUSED(peerAddress) @@ -175,16 +189,17 @@ bool IdentServer::removeSocket(const CoreIdentity *identity, const QHostAddress _connections.remove(localPort); processWaiting(socketId); - return true; } -qint64 IdentServer::addWaitingSocket() { +qint64 IdentServer::addWaitingSocket() +{ qint64 newSocketId = _socketId++; _waiting.push_back(newSocketId); return newSocketId; } -qint64 IdentServer::lowestSocketId() { +qint64 IdentServer::lowestSocketId() const +{ if (_waiting.empty()) { return std::numeric_limits::max(); } @@ -192,24 +207,30 @@ qint64 IdentServer::lowestSocketId() { return _waiting.front(); } -void IdentServer::removeWaitingSocket(qint64 socketId) { +void IdentServer::removeWaitingSocket(qint64 socketId) +{ _waiting.remove(socketId); } -void IdentServer::processWaiting(qint64 socketId) { +void IdentServer::processWaiting(qint64 socketId) +{ removeWaitingSocket(socketId); - _requestQueue.remove_if([=](Request request) { + + _requestQueue.remove_if([this, socketId](Request request) { if (socketId < request.transactionId && responseAvailable(request)) { return true; - } else if (lowestSocketId() < request.transactionId) { + } + else if (lowestSocketId() < request.transactionId) { return false; - } else { + } + else { request.respondError("NO-USER"); return true; } }); } -bool operator==(const Request &a, const Request &b) { +bool operator==(const Request& a, const Request& b) +{ return a.requestId == b.requestId; }