Attachment 128522 Details for Bug 99124 – call-content: fix another memory leak

[patch] call-content: fix another memory leak

1004-call-content-fix-another-memory-leak.patch (text/plain), 1.19 KB, created by Fabrice Bellet on 2016-12-17 12:25:02 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Fabrice Bellet

Created: 2016-12-17 12:25:02 UTC

Size: 1.19 KB

patch

obsolete

>From 90cd8d213be34bc1e23c4a66e4d2b5134fdce86b Mon Sep 17 00:00:00 2001
>From: Fabrice Bellet <fabrice@bellet.info>
>Date: Thu, 15 Dec 2016 11:26:47 +0100
>Subject: [PATCH] call-content: fix another memory leak
>
>---
> telepathy-farstream/call-content.c | 11 +++++++----
> 1 file changed, 7 insertions(+), 4 deletions(-)
>
>diff --git a/telepathy-farstream/call-content.c b/telepathy-farstream/call-content.c
>index b4b7e78..85089d3 100644
>--- a/telepathy-farstream/call-content.c
>+++ b/telepathy-farstream/call-content.c
>@@ -2213,16 +2213,19 @@ tf_call_content_error_literal (TfCallContent *self,
>     const gchar *detailed_reason,
>     const gchar *message)
> {
>+  GValueArray *array;
>+
>   g_debug ("Content error: %s", message);
>-  tp_cli_call_content_interface_media_call_fail (
>-      self->proxy, -1,
>-      tp_value_array_build (4,
>+  array = tp_value_array_build (4,
>           G_TYPE_UINT, 0,
>           G_TYPE_UINT, reason,
>           G_TYPE_STRING, detailed_reason,
>           G_TYPE_STRING, message,
>-          G_TYPE_INVALID),
>+          G_TYPE_INVALID);
>+  tp_cli_call_content_interface_media_call_fail (
>+      self->proxy, -1, array,
>       NULL, NULL, NULL, NULL);
>+  tp_value_array_free (array);
> }
> 
> 
>-- 
>2.7.4
>

Actions: View | Splinter Review

Attachments on bug 99124: 128520 | 128521 | 128522

Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.