summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--examples/network/secureudpclient/association.cpp8
-rw-r--r--examples/network/secureudpserver/server.cpp6
-rw-r--r--src/network/ssl/qdtls.cpp12
-rw-r--r--src/network/ssl/qdtls.h8
-rw-r--r--tests/auto/network/ssl/qdtls/tst_qdtls.cpp48
5 files changed, 41 insertions, 41 deletions
diff --git a/examples/network/secureudpclient/association.cpp b/examples/network/secureudpclient/association.cpp
index 02009fe233..6b510909f1 100644
--- a/examples/network/secureudpclient/association.cpp
+++ b/examples/network/secureudpclient/association.cpp
@@ -74,8 +74,8 @@ DtlsAssociation::DtlsAssociation(const QHostAddress &address, quint16 port,
DtlsAssociation::~DtlsAssociation()
{
- if (crypto.connectionEncrypted())
- crypto.sendShutdownAlert(&socket);
+ if (crypto.isConnectionEncrypted())
+ crypto.shutdown(&socket);
}
void DtlsAssociation::startHandshake()
@@ -108,7 +108,7 @@ void DtlsAssociation::readyRead()
}
dgram.resize(bytesRead);
- if (crypto.connectionEncrypted()) {
+ if (crypto.isConnectionEncrypted()) {
const QByteArray plainText = crypto.decryptDatagram(&socket, dgram);
if (plainText.size()) {
emit serverResponse(name, dgram, plainText);
@@ -128,7 +128,7 @@ void DtlsAssociation::readyRead()
emit errorMessage(tr("%1: handshake error - %2").arg(name, crypto.dtlsErrorString()));
return;
}
- if (crypto.connectionEncrypted()) {
+ if (crypto.isConnectionEncrypted()) {
emit infoMessage(tr("%1: encrypted connection established!").arg(name));
pingTimer.start();
pingTimeout();
diff --git a/examples/network/secureudpserver/server.cpp b/examples/network/secureudpserver/server.cpp
index c5887c9adb..763024e4f4 100644
--- a/examples/network/secureudpserver/server.cpp
+++ b/examples/network/secureudpserver/server.cpp
@@ -158,7 +158,7 @@ void DtlsServer::readyRead()
if (client == knownClients.end())
return handleNewConnection(peerAddress, peerPort, dgram);
- if ((*client)->connectionEncrypted()) {
+ if ((*client)->isConnectionEncrypted()) {
decryptDatagram(*client, dgram);
if ((*client)->dtlsError() == QDtlsError::RemoteClosedConnectionError)
knownClients.erase(client);
@@ -226,7 +226,7 @@ void DtlsServer::doHandshake(DtlsConnection newConnection, const QByteArray &cli
void DtlsServer::decryptDatagram(DtlsConnection connection, const QByteArray &clientMessage)
{
- Q_ASSERT(connection->connectionEncrypted());
+ Q_ASSERT(connection->isConnectionEncrypted());
const QString peerInfo = peer_info(connection->peerAddress(), connection->peerPort());
const QByteArray dgram = connection->decryptDatagram(&serverSocket, clientMessage);
@@ -243,7 +243,7 @@ void DtlsServer::decryptDatagram(DtlsConnection connection, const QByteArray &cl
void DtlsServer::shutdown()
{
for (DtlsConnection &connection : knownClients)
- connection->sendShutdownAlert(&serverSocket);
+ connection->shutdown(&serverSocket);
knownClients.clear();
serverSocket.close();
diff --git a/src/network/ssl/qdtls.cpp b/src/network/ssl/qdtls.cpp
index 58158036ca..cdb600538f 100644
--- a/src/network/ssl/qdtls.cpp
+++ b/src/network/ssl/qdtls.cpp
@@ -420,7 +420,7 @@ bool QDtls::continueHandshake(QUdpSocket *socket, const QByteArray &datagram)
return d->continueHandshake(socket, datagram);
}
-bool QDtls::resumeHandshakeAfterError(QUdpSocket *socket)
+bool QDtls::resumeHandshake(QUdpSocket *socket)
{
Q_D(QDtls);
@@ -438,7 +438,7 @@ bool QDtls::resumeHandshakeAfterError(QUdpSocket *socket)
return d->resumeHandshake(socket);
}
-bool QDtls::abortHandshakeAfterError(QUdpSocket *socket)
+bool QDtls::abortHandshake(QUdpSocket *socket)
{
Q_D(QDtls);
@@ -457,7 +457,7 @@ bool QDtls::abortHandshakeAfterError(QUdpSocket *socket)
return true;
}
-bool QDtls::sendShutdownAlert(QUdpSocket *socket)
+bool QDtls::shutdown(QUdpSocket *socket)
{
Q_D(QDtls);
@@ -477,7 +477,7 @@ bool QDtls::sendShutdownAlert(QUdpSocket *socket)
return true;
}
-bool QDtls::connectionEncrypted() const
+bool QDtls::isConnectionEncrypted() const
{
Q_D(const QDtls);
@@ -507,7 +507,7 @@ qint64 QDtls::writeDatagramEncrypted(QUdpSocket *socket, const QByteArray &dgram
return -1;
}
- if (!connectionEncrypted()) {
+ if (!isConnectionEncrypted()) {
d->setDtlsError(QDtlsError::InvalidOperation,
tr("Cannot write a datagram, not in encrypted state"));
return -1;
@@ -525,7 +525,7 @@ QByteArray QDtls::decryptDatagram(QUdpSocket *socket, const QByteArray &dgram)
return {};
}
- if (!connectionEncrypted()) {
+ if (!isConnectionEncrypted()) {
d->setDtlsError(QDtlsError::InvalidOperation,
tr("Cannot read a datagram, not in encrypted state"));
return {};
diff --git a/src/network/ssl/qdtls.h b/src/network/ssl/qdtls.h
index 7df92f31a7..88c10c8309 100644
--- a/src/network/ssl/qdtls.h
+++ b/src/network/ssl/qdtls.h
@@ -151,11 +151,11 @@ public:
bool doHandshake(QUdpSocket *socket, const QByteArray &dgram = {});
bool handleTimeout(QUdpSocket *socket);
- bool resumeHandshakeAfterError(QUdpSocket *socket);
- bool abortHandshakeAfterError(QUdpSocket *socket);
- bool sendShutdownAlert(QUdpSocket *socket);
+ bool resumeHandshake(QUdpSocket *socket);
+ bool abortHandshake(QUdpSocket *socket);
+ bool shutdown(QUdpSocket *socket);
- bool connectionEncrypted() const;
+ bool isConnectionEncrypted() const;
QSslCipher sessionCipher() const;
QSsl::SslProtocol sessionProtocol() const;
diff --git a/tests/auto/network/ssl/qdtls/tst_qdtls.cpp b/tests/auto/network/ssl/qdtls/tst_qdtls.cpp
index 31bdf1d5a4..571d341126 100644
--- a/tests/auto/network/ssl/qdtls/tst_qdtls.cpp
+++ b/tests/auto/network/ssl/qdtls/tst_qdtls.cpp
@@ -68,7 +68,7 @@ bool dtlsErrorIsCleared(DtlsPtr &dtls)
#define QDTLS_VERIFY_NO_ERROR(obj) QVERIFY(dtlsErrorIsCleared(obj))
#define QDTLS_VERIFY_HANDSHAKE_SUCCESS(obj) \
- QVERIFY(obj->connectionEncrypted()); \
+ QVERIFY(obj->isConnectionEncrypted()); \
QCOMPARE(obj->handshakeState(), QDtls::HandshakeComplete); \
QDTLS_VERIFY_NO_ERROR(obj); \
QCOMPARE(obj->peerVerificationErrors().size(), 0)
@@ -249,7 +249,7 @@ void tst_QDtls::construction()
QCOMPARE(dtls.dtlsConfiguration(), QSslConfiguration::defaultDtlsConfiguration());
QCOMPARE(dtls.handshakeState(), QDtls::HandshakeNotStarted);
- QCOMPARE(dtls.connectionEncrypted(), false);
+ QCOMPARE(dtls.isConnectionEncrypted(), false);
QCOMPARE(dtls.sessionCipher(), QSslCipher());
QCOMPARE(dtls.sessionProtocol(), QSsl::UnknownProtocol);
@@ -406,12 +406,12 @@ void tst_QDtls::handshake()
QVERIFY(!testLoop.timeout());
- QVERIFY(serverCrypto->connectionEncrypted());
+ QVERIFY(serverCrypto->isConnectionEncrypted());
QDTLS_VERIFY_NO_ERROR(serverCrypto);
QCOMPARE(serverCrypto->handshakeState(), QDtls::HandshakeComplete);
QCOMPARE(serverCrypto->peerVerificationErrors().size(), 0);
- QVERIFY(clientCrypto->connectionEncrypted());
+ QVERIFY(clientCrypto->isConnectionEncrypted());
QDTLS_VERIFY_NO_ERROR(clientCrypto);
QCOMPARE(clientCrypto->handshakeState(), QDtls::HandshakeComplete);
QCOMPARE(clientCrypto->peerVerificationErrors().size(), 0);
@@ -613,9 +613,9 @@ void tst_QDtls::protocolVersionMatching()
QDTLS_VERIFY_HANDSHAKE_SUCCESS(serverCrypto);
QDTLS_VERIFY_HANDSHAKE_SUCCESS(clientCrypto);
} else {
- QCOMPARE(serverCrypto->connectionEncrypted(), false);
+ QCOMPARE(serverCrypto->isConnectionEncrypted(), false);
QVERIFY(serverCrypto->handshakeState() != QDtls::HandshakeComplete);
- QCOMPARE(clientCrypto->connectionEncrypted(), false);
+ QCOMPARE(clientCrypto->isConnectionEncrypted(), false);
QVERIFY(clientCrypto->handshakeState() != QDtls::HandshakeComplete);
}
}
@@ -649,7 +649,7 @@ void tst_QDtls::verificationErrors()
QCOMPARE(clientCrypto->dtlsError(), QDtlsError::PeerVerificationError);
QCOMPARE(clientCrypto->handshakeState(), QDtls::PeerVerificationFailed);
- QVERIFY(!clientCrypto->connectionEncrypted());
+ QVERIFY(!clientCrypto->isConnectionEncrypted());
QVERIFY(verificationErrorDetected(QSslError::HostNameMismatch));
QVERIFY(verificationErrorDetected(QSslError::SelfSignedCertificate));
@@ -661,11 +661,11 @@ void tst_QDtls::verificationErrors()
QFETCH(const bool, abortHandshake);
if (abortHandshake) {
- QVERIFY(!clientCrypto->abortHandshakeAfterError(nullptr));
+ QVERIFY(!clientCrypto->abortHandshake(nullptr));
QCOMPARE(clientCrypto->dtlsError(), QDtlsError::InvalidInputParameters);
- QVERIFY(clientCrypto->abortHandshakeAfterError(&clientSocket));
+ QVERIFY(clientCrypto->abortHandshake(&clientSocket));
QDTLS_VERIFY_NO_ERROR(clientCrypto);
- QVERIFY(!clientCrypto->connectionEncrypted());
+ QVERIFY(!clientCrypto->isConnectionEncrypted());
QCOMPARE(clientCrypto->handshakeState(), QDtls::HandshakeNotStarted);
QCOMPARE(clientCrypto->sessionCipher(), QSslCipher());
QCOMPARE(clientCrypto->sessionProtocol(), QSsl::UnknownProtocol);
@@ -675,11 +675,11 @@ void tst_QDtls::verificationErrors()
QCOMPARE(clientCrypto->peerVerificationErrors().size(), 0);
} else {
clientCrypto->ignoreVerificationErrors(clientCrypto->peerVerificationErrors());
- QVERIFY(!clientCrypto->resumeHandshakeAfterError(nullptr));
+ QVERIFY(!clientCrypto->resumeHandshake(nullptr));
QCOMPARE(clientCrypto->dtlsError(), QDtlsError::InvalidInputParameters);
- QVERIFY(clientCrypto->resumeHandshakeAfterError(&clientSocket));
+ QVERIFY(clientCrypto->resumeHandshake(&clientSocket));
QDTLS_VERIFY_HANDSHAKE_SUCCESS(clientCrypto);
- QVERIFY(clientCrypto->connectionEncrypted());
+ QVERIFY(clientCrypto->isConnectionEncrypted());
QCOMPARE(clientCrypto->handshakeState(), QDtls::HandshakeComplete);
QCOMPARE(clientCrypto->peerVerificationErrors().size(), 0);
}
@@ -772,7 +772,7 @@ void tst_QDtls::verifyServerCertificate()
QDTLS_VERIFY_HANDSHAKE_SUCCESS(serverCrypto);
QDTLS_VERIFY_HANDSHAKE_SUCCESS(clientCrypto);
} else {
- QVERIFY(!clientCrypto->connectionEncrypted());
+ QVERIFY(!clientCrypto->isConnectionEncrypted());
QCOMPARE(clientCrypto->handshakeState(), QDtls::PeerVerificationFailed);
QVERIFY(clientCrypto->peerVerificationErrors().size());
QVERIFY(clientCrypto->writeDatagramEncrypted(&clientSocket, "something") < 0);
@@ -907,12 +907,12 @@ void tst_QDtls::verifyClientCertificate()
QDTLS_VERIFY_HANDSHAKE_SUCCESS(serverCrypto);
QDTLS_VERIFY_HANDSHAKE_SUCCESS(clientCrypto);
} else {
- QVERIFY(!serverCrypto->connectionEncrypted());
+ QVERIFY(!serverCrypto->isConnectionEncrypted());
QCOMPARE(serverCrypto->handshakeState(), QDtls::PeerVerificationFailed);
QVERIFY(serverCrypto->dtlsErrorString().size() > 0);
QVERIFY(serverCrypto->peerVerificationErrors().size() > 0);
- QVERIFY(!clientCrypto->connectionEncrypted());
+ QVERIFY(!clientCrypto->isConnectionEncrypted());
QDTLS_VERIFY_NO_ERROR(clientCrypto);
QCOMPARE(clientCrypto->handshakeState(), QDtls::HandshakeInProgress);
}
@@ -942,7 +942,7 @@ void tst_QDtls::blacklistedCerificate()
QVERIFY(!testLoop.timeout());
QCOMPARE(clientCrypto->handshakeState(), QDtls::PeerVerificationFailed);
QCOMPARE(clientCrypto->dtlsError(), QDtlsError::PeerVerificationError);
- QVERIFY(!clientCrypto->connectionEncrypted());
+ QVERIFY(!clientCrypto->isConnectionEncrypted());
QVERIFY(verificationErrorDetected(QSslError::CertificateBlacklisted));
}
@@ -972,12 +972,12 @@ void tst_QDtls::readWriteEncrypted()
QDTLS_VERIFY_NO_ERROR(clientCrypto);
QVERIFY(clientCrypto->writeDatagramEncrypted(&clientSocket, serverExpectedPlainText) <= 0);
QCOMPARE(clientCrypto->dtlsError(), QDtlsError::InvalidOperation);
- QVERIFY(!clientCrypto->sendShutdownAlert(&clientSocket));
+ QVERIFY(!clientCrypto->shutdown(&clientSocket));
QCOMPARE(clientCrypto->dtlsError(), QDtlsError::InvalidOperation);
QDTLS_VERIFY_NO_ERROR(serverCrypto);
QVERIFY(serverCrypto->writeDatagramEncrypted(&serverSocket, clientExpectedPlainText) <= 0);
QCOMPARE(serverCrypto->dtlsError(), QDtlsError::InvalidOperation);
- QVERIFY(!serverCrypto->sendShutdownAlert(&serverSocket));
+ QVERIFY(!serverCrypto->shutdown(&serverSocket));
QCOMPARE(serverCrypto->dtlsError(), QDtlsError::InvalidOperation);
// 1. Initiate a handshake:
@@ -1018,19 +1018,19 @@ void tst_QDtls::readWriteEncrypted()
DtlsPtr &crypto = serverSideShutdown ? serverCrypto : clientCrypto;
QUdpSocket *socket = serverSideShutdown ? &serverSocket : &clientSocket;
// 6. Parameter validation:
- QVERIFY(!crypto->sendShutdownAlert(nullptr));
+ QVERIFY(!crypto->shutdown(nullptr));
QCOMPARE(crypto->dtlsError(), QDtlsError::InvalidInputParameters);
// 7. Send shutdown alert:
- QVERIFY(crypto->sendShutdownAlert(socket));
+ QVERIFY(crypto->shutdown(socket));
QDTLS_VERIFY_NO_ERROR(crypto);
QCOMPARE(crypto->handshakeState(), QDtls::HandshakeNotStarted);
- QVERIFY(!crypto->connectionEncrypted());
+ QVERIFY(!crypto->isConnectionEncrypted());
// 8. Receive this read notification and handle it:
testLoop.enterLoopMSecs(dataExchangeTimeoutMS);
QVERIFY(!testLoop.timeout());
DtlsPtr &peerCrypto = serverSideShutdown ? clientCrypto : serverCrypto;
- QVERIFY(!peerCrypto->connectionEncrypted());
+ QVERIFY(!peerCrypto->isConnectionEncrypted());
QCOMPARE(peerCrypto->handshakeState(), QDtls::HandshakeNotStarted);
QCOMPARE(peerCrypto->dtlsError(), QDtlsError::RemoteClosedConnectionError);
}
@@ -1066,7 +1066,7 @@ void tst_QDtls::datagramFragmentation()
QVERIFY(clientCrypto->writeDatagramEncrypted(&clientSocket, QByteArray(1024 * 17, Qt::Uninitialized)) <= 0);
QVERIFY(clientCrypto->dtlsError() != QDtlsError::NoError);
// Error to write does not mean QDtls is broken:
- QVERIFY(clientCrypto->connectionEncrypted());
+ QVERIFY(clientCrypto->isConnectionEncrypted());
QVERIFY(clientCrypto->writeDatagramEncrypted(&clientSocket, "Hello, I'm a tiny datagram") > 0);
QDTLS_VERIFY_NO_ERROR(clientCrypto);
}