diff options
author | Philip Withnall <withnall@endlessm.com> | 2017-04-05 11:36:12 +0100 |
---|---|---|
committer | Simon McVittie <smcv@debian.org> | 2017-04-05 16:22:52 +0100 |
commit | bb43433b639075d4e79cb319b22fbb137b752635 (patch) | |
tree | c7f127f8aacc92ecdda5be083144e7cbe84a93df | |
parent | 9e864c2e4b43b81db110bf506968fd0caa8558a1 (diff) |
test: Fix a couple of memory leaks in test-corrupt
Spotted while testing bug #100568.
Signed-off-by: Philip Withnall <withnall@endlessm.com>
Reviewed-by: Simon McVittie <smcv@collabora.com>
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=100568
-rw-r--r-- | test/corrupt.c | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/test/corrupt.c b/test/corrupt.c index 56a0cdf2..d6f85021 100644 --- a/test/corrupt.c +++ b/test/corrupt.c @@ -99,14 +99,16 @@ test_connect (Fixture *f, gconstpointer addr G_GNUC_UNUSED) { dbus_bool_t have_mem; + char *address = NULL; g_assert (f->server_conn == NULL); - f->client_conn = dbus_connection_open_private ( - dbus_server_get_address (f->server), &f->e); + address = dbus_server_get_address (f->server); + f->client_conn = dbus_connection_open_private (address, &f->e); assert_no_error (&f->e); g_assert (f->client_conn != NULL); test_connection_setup (f->ctx, f->client_conn); + dbus_free (address); while (f->server_conn == NULL) { @@ -352,6 +354,7 @@ teardown (Fixture *f, { if (f->client_conn != NULL) { + test_connection_shutdown (f->ctx, f->client_conn); dbus_connection_close (f->client_conn); dbus_connection_unref (f->client_conn); f->client_conn = NULL; @@ -359,6 +362,7 @@ teardown (Fixture *f, if (f->server_conn != NULL) { + test_connection_shutdown (f->ctx, f->server_conn); dbus_connection_close (f->server_conn); dbus_connection_unref (f->server_conn); f->server_conn = NULL; |