Received-SPF: pass (google.com: best guess record for domain of linux-bluetooth-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
MIME-Version: 1.0
References: <CGME20191203094848epcas5p296450885dea6e4939072c7b8b50e71fa@epcas5p2.samsung.com>
 <1575366522-7742-1-git-send-email-s.syam@samsung.com>
In-Reply-To: <1575366522-7742-1-git-send-email-s.syam@samsung.com>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Sun, 8 Dec 2019 06:11:46 +0200
Message-ID: <CABBYNZ+BAGZv6vw_aK=t9J6RUhtefVLWFPD9Q2nQZJfejxiUuA@mail.gmail.com>
Subject: Re: [PATCH 1/1] client: Fix memory leak
To:     Syam Sidhardhan <s.syam@samsung.com>
Cc:     "linux-bluetooth@vger.kernel.org" <linux-bluetooth@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

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