All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/1] client: Fix memory leak
       [not found] <CGME20191203094848epcas5p296450885dea6e4939072c7b8b50e71fa@epcas5p2.samsung.com>
@ 2019-12-03  9:48 ` Syam Sidhardhan
  2019-12-08  4:11   ` Luiz Augusto von Dentz
  0 siblings, 1 reply; 2+ messages in thread
From: Syam Sidhardhan @ 2019-12-03  9:48 UTC (permalink / raw)
  To: linux-bluetooth

---
 client/main.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/client/main.c b/client/main.c
index 68dabda..8bd0bac 100644
--- a/client/main.c
+++ b/client/main.c
@@ -1792,11 +1792,11 @@ static void remove_device(GDBusProxy *proxy)
 {
 	char *path;
 
-	path = g_strdup(g_dbus_proxy_get_path(proxy));
-
 	if (!default_ctrl)
 		return;
 
+	path = g_strdup(g_dbus_proxy_get_path(proxy));
+
 	if (g_dbus_proxy_method_call(default_ctrl->proxy, "RemoveDevice",
 						remove_device_setup,
 						remove_device_reply,
-- 
2.7.4


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH 1/1] client: Fix memory leak
  2019-12-03  9:48 ` [PATCH 1/1] client: Fix memory leak Syam Sidhardhan
@ 2019-12-08  4:11   ` Luiz Augusto von Dentz
  0 siblings, 0 replies; 2+ messages in thread
From: Luiz Augusto von Dentz @ 2019-12-08  4:11 UTC (permalink / raw)
  To: Syam Sidhardhan; +Cc: linux-bluetooth

Hi Syam,

On Tue, Dec 3, 2019 at 11:51 AM Syam Sidhardhan <s.syam@samsung.com> wrote:
>
> ---
>  client/main.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/client/main.c b/client/main.c
> index 68dabda..8bd0bac 100644
> --- a/client/main.c
> +++ b/client/main.c
> @@ -1792,11 +1792,11 @@ static void remove_device(GDBusProxy *proxy)
>  {
>         char *path;
>
> -       path = g_strdup(g_dbus_proxy_get_path(proxy));
> -
>         if (!default_ctrl)
>                 return;
>
> +       path = g_strdup(g_dbus_proxy_get_path(proxy));
> +
>         if (g_dbus_proxy_method_call(default_ctrl->proxy, "RemoveDevice",
>                                                 remove_device_setup,
>                                                 remove_device_reply,
> --
> 2.7.4

Applied, thanks.

-- 
Luiz Augusto von Dentz

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2019-12-08  4:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <CGME20191203094848epcas5p296450885dea6e4939072c7b8b50e71fa@epcas5p2.samsung.com>
2019-12-03  9:48 ` [PATCH 1/1] client: Fix memory leak Syam Sidhardhan
2019-12-08  4:11   ` Luiz Augusto von Dentz

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.