diff options
author | Timur Pocheptsov <timur.pocheptsov@qt.io> | 2021-01-25 11:28:48 +0100 |
---|---|---|
committer | Timur Pocheptsov <timur.pocheptsov@qt.io> | 2021-01-28 16:21:07 +0100 |
commit | 1a2dd9276759742a7eda704341ee618d296ad8bf (patch) | |
tree | d6d0b7f8cf473c2d682aeae12bc2bad0680ace0b /src/network/ssl/qsslsocket_mac.cpp | |
parent | 795ea19ca0a7e484793655b35f19de8dbc88e987 (diff) |
TLS backend: merge with its factory
QTlsBackend is a factory itself - it creates TLS/X509 objects. Having
an intermediary between Factory->Backend->TLS primitive does not look
very natural thus let's squash the first two parts. Backend is a factory
creating TLS primitives, but its static functions also provide information
about backends availablei and give access to those backends.
Fixes: QTBUG-90606
Task-number: QTBUG-65922
Change-Id: I8409d81fd11fb46e6ab4465b4937a7680a8c2447
Reviewed-by: Alex Blasche <alexander.blasche@qt.io>
Reviewed-by: MÃ¥rten Nordheim <marten.nordheim@qt.io>
Diffstat (limited to 'src/network/ssl/qsslsocket_mac.cpp')
-rw-r--r-- | src/network/ssl/qsslsocket_mac.cpp | 19 |
1 files changed, 3 insertions, 16 deletions
diff --git a/src/network/ssl/qsslsocket_mac.cpp b/src/network/ssl/qsslsocket_mac.cpp index abbcf8a6ac..6370a83dd3 100644 --- a/src/network/ssl/qsslsocket_mac.cpp +++ b/src/network/ssl/qsslsocket_mac.cpp @@ -84,20 +84,7 @@ class SecureTransportBackend : public QTlsBackend private: QString backendName() const override { - return QTlsBackendFactory::builtinBackendNames[QTlsBackendFactory::nameIndexSecureTransport]; - } -}; - -class SecureTransportBackendFactory : public QTlsBackendFactory -{ -private: - QString backendName() const override - { - return QTlsBackendFactory::builtinBackendNames[QTlsBackendFactory::nameIndexSecureTransport]; - } - QTlsBackend *create() const override - { - return new SecureTransportBackend; + return builtinBackendNames[nameIndexSecureTransport]; } QList<QSsl::SslProtocol> supportedProtocols() const override @@ -135,7 +122,7 @@ private: } }; -Q_GLOBAL_STATIC(SecureTransportBackendFactory, factory) +Q_GLOBAL_STATIC(SecureTransportBackend, backend) #ifdef Q_OS_MACOS /* @@ -1618,7 +1605,7 @@ void QSslSocketPrivate::registerAdHocFactory() { // TLSTODO: this is a temporary solution, waiting for // backends to move to ... plugins. - if (!factory()) + if (!backend()) qCWarning(lcSsl, "Failed to create backend factory"); } |