[Qt] paymentserver: do not log NULL certificates

Philip Kaufmann · Philip Kaufmann · commit 5a53d7cda376 · 2015-04-15T14:31:48.000+02:00
- also add a few more comments in PaymentServer::LoadRootCAs
diff --git a/src/qt/paymentserver.cpp b/src/qt/paymentserver.cpp
@@ -143,13 +143,20 @@ void PaymentServer::LoadRootCAs(X509_STORE* _store)
 
     int nRootCerts = 0;
     const QDateTime currentTime = QDateTime::currentDateTime();
-    foreach (const QSslCertificate& cert, certList)
-    {
+
+    foreach (const QSslCertificate& cert, certList) {
+        // Don't log NULL certificates
+        if (cert.isNull())
+            continue;
+
+        // Not yet active/valid, or expired certificate
         if (currentTime < cert.effectiveDate() || currentTime > cert.expiryDate()) {
             ReportInvalidCertificate(cert);
             continue;
         }
+
 #if QT_VERSION >= 0x050000
+        // Blacklisted certificate
         if (cert.isBlacklisted()) {
             ReportInvalidCertificate(cert);
             continue;
