Added a missing close() function after a unsuccessful connect() call,…

… and removed WSACleanup() when WSAStartup failed.
haakonnessjoen · May 23, 2014 · 9a8b9e1 · 9a8b9e1
1 parent 66887fa
commit 9a8b9e1
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/librouteros.c b/librouteros.c
@@ -349,7 +349,6 @@ struct ros_connection *ros_connect(char *address, int port) {
 #ifdef _WIN32
 	if ((retval = WSAStartup(0x202, &wsaData)) != 0) {
 		fprintf(stderr,"Server: WSAStartup() failed with error %d\n", retval);
-		WSACleanup();
 		free(conn);
 		return NULL;
 	}
@@ -383,7 +382,10 @@ struct ros_connection *ros_connect(char *address, int port) {
 #endif
 	) {
 #ifdef _WIN32
+		closesocket(conn->socket);
 		WSACleanup();
+#else
+		close(conn->socket);
 #endif
 		free(conn);
 		return NULL;