Mark the socket as not connected before releasing socket_mutex

Issue 190 ( https://github.com/eclipse/paho.mqtt.c/issues/190 ) is
probably caused by some race condition.
One possible cause for such a race condition is in the previous
implementation of MQTTAsync_closeOnly(). Which had released the semaphore
before the socket is marked as unused. This way, another thread may believe,
that it should wait that the socket becomes available (which never happens).
Signed-off-by: Juergen Kosel <juergen.kosel@softing.com>

src/MQTTAsync.c

@@ -1993,11 +1993,11 @@ void MQTTAsync_closeOnly(Clients* client)
 		SSLSocket_close(&client->net);
 #endif
 		Socket_close(client->net.socket);
-		Thread_unlock_mutex(socket_mutex);
 		client->net.socket = 0;
 #if defined(OPENSSL)
 		client->net.ssl = NULL;
 #endif
+		Thread_unlock_mutex(socket_mutex);
 	}
 	client->connected = 0;
 	client->connect_state = 0;