do not set socket error twice in EAGAIN case in QNativeSocketEnginePrivate::nativeCon...

strerror(EAGAIN) returns "Resource temporarily unavailable", strerror(EALREADY)
returns "Operation already in progress" which is deffinetly not invalid
socket error ("Invalid socket descriptor")

Signed-off-by: Ivailo Monev <xakepa10@gmail.com>

diff --git a/src/network/socket/qnativesocketengine_unix.cpp b/src/network/socket/qnativesocketengine_unix.cpp
index 5dc186f..791d4be 100644 (file)
--- a/src/network/socket/qnativesocketengine_unix.cpp
+++ b/src/network/socket/qnativesocketengine_unix.cpp
@@ -358,12 +358,11 @@ bool QNativeSocketEnginePrivate::nativeConnect(const QHostAddress &addr, quint16
             break;
         case EINPROGRESS:
         case EALREADY:
-            setError(QAbstractSocket::UnfinishedSocketOperationError, InvalidSocketErrorString);
+            setError(QAbstractSocket::UnfinishedSocketOperationError, ResourceErrorString);
             socketState = QAbstractSocket::ConnectingState;
             break;
         case EAGAIN:
-            setError(QAbstractSocket::UnfinishedSocketOperationError, InvalidSocketErrorString);
-            setError(QAbstractSocket::SocketResourceError, ResourceErrorString);
+            setError(QAbstractSocket::UnfinishedSocketOperationError, ResourceErrorString);
             break;
         case EACCES:
         case EPERM: