[librest] tests/proxy: Fix memory leaks
- From: Christophe Fergeau <teuf src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [librest] tests/proxy: Fix memory leaks
- Date: Wed, 3 Sep 2014 08:56:37 +0000 (UTC)
commit 0ad11f04ae8b1285b300170e9f39a45fb5d581ed
Author: Christophe Fergeau <cfergeau redhat com>
Date: Tue Sep 2 18:41:50 2014 +0200
tests/proxy: Fix memory leaks
https://bugzilla.gnome.org/show_bug.cgi?id=735921
tests/proxy.c | 19 +++++++++++++++++--
1 files changed, 17 insertions(+), 2 deletions(-)
---
diff --git a/tests/proxy.c b/tests/proxy.c
index a1bed65..7f97e9c 100644
--- a/tests/proxy.c
+++ b/tests/proxy.c
@@ -152,18 +152,22 @@ echo_test (RestProxy *proxy)
if (rest_proxy_call_get_status_code (call) != SOUP_STATUS_OK) {
g_printerr ("wrong response code\n");
errors++;
+ g_object_unref (call);
return;
}
if (rest_proxy_call_get_payload_length (call) != 6) {
g_printerr ("wrong length returned\n");
errors++;
+ g_object_unref (call);
return;
}
if (g_strcmp0 ("echome", rest_proxy_call_get_payload (call)) != 0) {
g_printerr ("wrong string returned\n");
errors++;
+ g_object_unref (call);
return;
}
+ g_object_unref (call);
}
static void
@@ -187,18 +191,22 @@ reverse_test (RestProxy *proxy)
if (rest_proxy_call_get_status_code (call) != SOUP_STATUS_OK) {
g_printerr ("wrong response code\n");
errors++;
+ g_object_unref (call);
return;
}
if (rest_proxy_call_get_payload_length (call) != 9) {
g_printerr ("wrong length returned\n");
errors++;
+ g_object_unref (call);
return;
}
if (g_strcmp0 ("emesrever", rest_proxy_call_get_payload (call)) != 0) {
g_printerr ("wrong string returned\n");
errors++;
+ g_object_unref (call);
return;
}
+ g_object_unref (call);
}
static void
@@ -206,10 +214,13 @@ status_ok_test (RestProxy *proxy, int status)
{
RestProxyCall *call;
GError *error = NULL;
+ char *status_str;
call = rest_proxy_new_call (proxy);
rest_proxy_call_set_function (call, "status");
- rest_proxy_call_add_param (call, "status", g_strdup_printf ("%d", status));
+ status_str = g_strdup_printf ("%d", status);
+ rest_proxy_call_add_param (call, "status", status_str);
+ g_free (status_str);
if (!rest_proxy_call_run (call, NULL, &error)) {
g_printerr ("Call failed: %s\n", error->message);
@@ -233,10 +244,13 @@ status_error_test (RestProxy *proxy, int status)
{
RestProxyCall *call;
GError *error = NULL;
+ char *status_str;
call = rest_proxy_new_call (proxy);
rest_proxy_call_set_function (call, "status");
- rest_proxy_call_add_param (call, "status", g_strdup_printf ("%d", status));
+ status_str = g_strdup_printf ("%d", status);
+ rest_proxy_call_add_param (call, "status", status_str);
+ g_free (status_str);
if (rest_proxy_call_run (call, NULL, &error)) {
g_printerr ("Call succeeded should have failed");
@@ -244,6 +258,7 @@ status_error_test (RestProxy *proxy, int status)
g_object_unref (call);
return;
}
+ g_error_free (error);
if (rest_proxy_call_get_status_code (call) != status) {
g_printerr ("wrong response code, got %d\n", rest_proxy_call_get_status_code (call));
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]