1 files changed, 34 insertions, 49 deletions

diff --git a/server_communication.c b/server_communication.c
index e8ab7b8..72e3386 100644
--- a/server_communication.c
+++ b/server_communication.c
@@ -9,8 +9,8 @@ enum connection_type {
 	SERVER_CONNECTION_TYPE_COMMAND,
 };
 
-static GSocket *server_connect(struct server *server, const enum connection_type type) {
-	GError *error = NULL, *conn_error = NULL;
+static GSocket *server_connect(struct server *server, const enum connection_type type, GError **error) {
+	GError *conn_error = NULL;
 	GSocketConnectable *addr;
 	GSocketAddressEnumerator *enumerator;
 	GSocketAddress *sockaddr;
@@ -33,46 +33,38 @@ static GSocket *server_connect(struct server *server, const enum connection_type
 
 	client = g_socket_client_new();
 
-	while(conn == NULL && (sockaddr = g_socket_address_enumerator_next(enumerator, NULL, &error))) {
+	while(conn == NULL && (sockaddr = g_socket_address_enumerator_next(enumerator, NULL, error))) {
 		conn = g_socket_client_connect(client, (GSocketConnectable*)sockaddr, NULL, &conn_error);
 		g_object_unref(sockaddr);
 	}
 
-	if(sockaddr == NULL) {
-		if(error != NULL) {
-			g_warning(error->message);
-			g_error_free(error);
-		} else {
-			g_warning("server_connect: no connectable addresses found for host %s",
-					server->host);
+	if(conn != NULL) {
+		if(conn_error != NULL) {
+			g_error_free(conn_error);
+		}
+		return g_socket_connection_get_socket(conn);
+	} else if(*error != NULL) {
+		if(conn_error != NULL) {
+			g_error_free(conn_error);
 		}
 		return NULL;
-	}
-
-	if(conn == NULL) {
-		g_warning(conn_error->message);
-		g_error_free(conn_error);
+	} else {
+		g_propagate_error(error, conn_error);
 		return NULL;
 	}
-
-	GSocket *socket = g_socket_connection_get_socket(conn);
-
-	g_debug("connected");
-
-	return socket;
 }
 
-static GSocket *server_connect_http(struct server *server) {
-	return server_connect(server, SERVER_CONNECTION_TYPE_HTTP);
+static GSocket *server_connect_http(struct server *server, GError **error) {
+	return server_connect(server, SERVER_CONNECTION_TYPE_HTTP, error);
 }
 
-static GSocket *server_connect_command(struct server *server) {
-	return server_connect(server, SERVER_CONNECTION_TYPE_COMMAND);
+static GSocket *server_connect_command(struct server *server, GError **error) {
+	return server_connect(server, SERVER_CONNECTION_TYPE_COMMAND, error);
 }
 
-gboolean server_ping(struct server *server) {
+gboolean server_ping(struct server *server, GError **error) {
 	GTimer *timer = g_timer_new();
-	GSocket *socket = server_connect_command(server);
+	GSocket *socket = server_connect_command(server, error);
 
 	if(socket == NULL) {
 		return FALSE;
@@ -94,8 +86,8 @@ gboolean server_ping(struct server *server) {
 	return result;
 }
 
-static gchar **server_get_stringlist(struct server *server, const gchar *cmd) {
-	GSocket *socket = server_connect_command(server);
+static gchar **server_get_stringlist(struct server *server, const gchar *cmd, GError **error) {
+	GSocket *socket = server_connect_command(server, error);
 
 	if(socket == NULL) {
 		return NULL;
@@ -118,26 +110,25 @@ static gchar **server_get_stringlist(struct server *server, const gchar *cmd) {
 	return data;
 }
 
-gchar **server_find(struct server *server, const gchar *type, const gchar *str) {
+gchar **server_find(struct server *server, const gchar *type, const gchar *str, GError **error) {
 	gchar *cmd = g_strdup_printf("findl %s %s\nexit\n", type, str);
-	gchar **data = server_get_stringlist(server, cmd);
+	gchar **data = server_get_stringlist(server, cmd, error);
 	g_free(cmd);
 
 	return data;
 }
 
-gchar **server_list(struct server *server, const gchar *directory) {
+gchar **server_list(struct server *server, const gchar *directory, GError **error) {
 	gchar *cmd = g_strdup_printf("list %s\nexit\n", directory);
-	gchar **data = server_get_stringlist(server, cmd);
+	gchar **data = server_get_stringlist(server, cmd, error);
 	g_free(cmd);
 
 	return data;
 }
 
 gboolean server_get(struct server *server, const gchar *type,
-		const gchar *localfile, const gchar *remotefile) {
-	GError *error = NULL;
-	GSocket *socket = server_connect_http(server);
+		const gchar *localfile, const gchar *remotefile, GError **error) {
+	GSocket *socket = server_connect_http(server, error);
 
 	if(socket == NULL) {
 		return FALSE;
@@ -154,43 +145,37 @@ gboolean server_get(struct server *server, const gchar *type,
 	gchar *line = NULL;
 	gsize size;
 
-	while((line = g_data_input_stream_read_line(input, &size, NULL, &error)) != NULL) {
+	while((line = g_data_input_stream_read_line(input, &size, NULL, error)) != NULL) {
 		g_debug(line);
 		if(size == 0) {
 			break;
 		}
 	}
 
-	if(error != NULL) {
-		g_warning(error->message);
-		g_error_free(error);
+	if(*error != NULL) {
 		return FALSE;
 	}
 
 	GFile *file = g_file_new_for_path(localfile);
-	GFileOutputStream *output = g_file_replace(file, NULL, FALSE, G_FILE_CREATE_NONE, NULL, &error);
+	GFileOutputStream *output = g_file_replace(file, NULL, FALSE, G_FILE_CREATE_NONE, NULL, error);
 
 	if(output == NULL) {
-		g_warning(error->message);
-		g_error_free(error);
 		return FALSE;
 	}
 
 	g_output_stream_splice((GOutputStream*)output, (GInputStream*)input,
 			G_OUTPUT_STREAM_SPLICE_CLOSE_SOURCE | G_OUTPUT_STREAM_SPLICE_CLOSE_TARGET,
-			NULL, &error);
+			NULL, error);
 
 	g_object_unref(input);
 	g_object_unref(output);
 	g_object_unref(file);
+	g_object_unref(connection);
+	g_object_unref(socket);
 
-	if(error != NULL) {
-		g_warning(error->message);
-		g_error_free(error);
+	if(*error != NULL) {
 		return FALSE;
 	}
 
-	g_object_unref(connection);
-
 	return TRUE;
 }