Re: [PATCH BlueZ] client: Fix memory leak due to the option to register an agent

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Eramoto,

On Tue, Jul 4, 2017 at 10:21 AM, ERAMOTO Masaya
<eramoto.masaya@xxxxxxxxxxxxxx> wrote:
> Hi Luiz,
>
> Would you apply my patch to git master branch if there are not any problems?
> I confirmed that it was applied to the latest master branch with no problem.
>
> Regards,
> Eramoto
>
> On 2017年06月21日 19:07, ERAMOTO Masaya wrote:
>> When bluetoothctl runs with the command line option "-a <capability>", it
>> does not free the variable auto_register_agent registering the initial
>> agent and allocates new memory for it.
>>
>> This frees the allocated memory before allocating new memory.
>> (The related commit id is 6db3470c2ea161b4b808ad1fc80dfd7e014fd359.)
>>
>> ---
>>  client/main.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/client/main.c b/client/main.c
>> index 2f269af..9b01dfb 100644
>> --- a/client/main.c
>> +++ b/client/main.c
>> @@ -2358,6 +2358,7 @@ static gboolean parse_agent(const char *key, const char *value,
>>       if (!value)
>>               return FALSE;
>>
>> +     g_free(auto_register_agent);
>>       auto_register_agent = g_strdup(value);
>>
>>       return TRUE;
>>

Applied, thanks.

-- 
Luiz Augusto von Dentz
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux