X-Git-Url: https://git.quassel-irc.org/?p=quassel.git;a=blobdiff_plain;f=src%2Fcore%2Fsslserver.cpp;h=83d7e6035aca4d8ca18b1baa8f6442ec8d840451;hp=183bbe18c7a486bb34f9d81b473f81806b4a7a7e;hb=39328183a6a87c6eb10a9dbbffcd5d65bf154a1f;hpb=37110ceaa070167b4f40ed449ac9ea130503a792

diff --git a/src/core/sslserver.cpp b/src/core/sslserver.cpp
index 183bbe18..83d7e603 100644
--- a/src/core/sslserver.cpp
+++ b/src/core/sslserver.cpp
@@ -66,16 +66,13 @@ SslServer::SslServer(QObject *parent)
 QTcpSocket *SslServer::nextPendingConnection()
 {
     if (_pendingConnections.isEmpty())
-        return 0;
+        return nullptr;
     else
         return _pendingConnections.takeFirst();
 }
 
-#if QT_VERSION >= 0x050000
+
 void SslServer::incomingConnection(qintptr socketDescriptor)
-#else
-void SslServer::incomingConnection(int socketDescriptor)
-#endif
 {
     QSslSocket *serverSocket = new QSslSocket(this);
     if (serverSocket->setSocketDescriptor(socketDescriptor)) {
@@ -196,20 +193,16 @@ bool SslServer::setCertificate(const QString &path, const QString &keyPath)
 
     // We allow the core to offer SSL anyway, so no "return false" here. Client will warn about the cert being invalid.
     const QDateTime now = QDateTime::currentDateTime();
-    if (now < untestedCert.effectiveDate())
+    if (now < untestedCert.effectiveDate()) {
         quWarning() << "SslServer: Certificate won't be valid before" << untestedCert.effectiveDate().toString();
-
-    else if (now > untestedCert.expiryDate())
+    }
+    else if (now > untestedCert.expiryDate()) {
         quWarning() << "SslServer: Certificate expired on" << untestedCert.expiryDate().toString();
-
-    else { // Qt4's isValid() checks for time range and blacklist; avoid a double warning, hence the else block
-#if QT_VERSION < 0x050000
-        if (!untestedCert.isValid())
-#else
-        if (untestedCert.isBlacklisted())
-#endif
-            quWarning() << "SslServer: Certificate blacklisted";
     }
+    else if (untestedCert.isBlacklisted()) {
+        quWarning() << "SslServer: Certificate blacklisted";
+    }
+
     if (untestedKey.isNull()) {
         quWarning() << "SslServer:" << qPrintable(keyPath) << "contains no key data";
         return false;
@@ -230,7 +223,6 @@ QSslKey SslServer::loadKey(QFile *keyFile)
 {
     QSslKey key;
     key = QSslKey(keyFile, QSsl::Rsa);
-#if QT_VERSION >= 0x050500
     if (key.isNull()) {
         if (!keyFile->reset()) {
             quWarning() << "SslServer: IO error reading key file";
@@ -238,7 +230,6 @@ QSslKey SslServer::loadKey(QFile *keyFile)
         }
         key = QSslKey(keyFile, QSsl::Ec);
     }
-#endif
     return key;
 }