diff options
author | Aleksander Morgado <aleksander@aleksander.es> | 2014-01-24 20:44:14 +0100 |
---|---|---|
committer | Aleksander Morgado <aleksander@aleksander.es> | 2014-01-24 21:09:10 +0100 |
commit | 1547be518165464fc0700c65aa864fcc35597717 (patch) | |
tree | c69cd727173a8cb549d4e76bd22b1f04027e2197 /src | |
parent | 26793e10d6e32ba8595cbc70970e8d02aa3169b9 (diff) |
bearer-mbim: fix several leaked 'MbimMessage' objects
mbim_device_command_finish() returns always a new reference to the response.
Diffstat (limited to 'src')
-rw-r--r-- | src/mm-bearer-mbim.c | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/src/mm-bearer-mbim.c b/src/mm-bearer-mbim.c index 72cabebc..c25460a9 100644 --- a/src/mm-bearer-mbim.c +++ b/src/mm-bearer-mbim.c @@ -404,6 +404,9 @@ ip_configuration_query_ready (MbimDevice *device, g_free (ipv6dnsserver); } + if (response) + mbim_message_unref (response); + if (error) { g_simple_async_result_take_error (ctx->result, error); connect_context_complete_and_free (ctx); @@ -462,6 +465,9 @@ connect_set_ready (MbimDevice *device, } } + if (response) + mbim_message_unref (response); + if (error) { g_simple_async_result_take_error (ctx->result, error); connect_context_complete_and_free (ctx); @@ -515,6 +521,9 @@ provisioned_contexts_query_ready (MbimDevice *device, g_error_free (error); } + if (response) + mbim_message_unref (response); + /* Keep on */ ctx->step++; connect_context_step (ctx); @@ -571,6 +580,9 @@ packet_service_set_ready (MbimDevice *device, } } + if (response) + mbim_message_unref (response); + if (error) { /* Don't make NoDeviceSupport errors fatal; just try to keep on the * connection sequence even with this error. */ @@ -956,6 +968,9 @@ disconnect_set_ready (MbimDevice *device, } } + if (response) + mbim_message_unref (response); + if (error) { g_simple_async_result_take_error (ctx->result, error); disconnect_context_complete_and_free (ctx); |