[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH V2] vhost-user-test: fix a memory leak
From: |
Pan Nengyuan |
Subject: |
Re: [PATCH V2] vhost-user-test: fix a memory leak |
Date: |
Mon, 13 Jan 2020 10:32:57 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2 |
On 1/12/2020 6:39 PM, Thomas Huth wrote:
> On 10/01/2020 15.07, Thomas Huth wrote:
>> On 20/12/2019 02.26, address@hidden wrote:
>>> From: Pan Nengyuan <address@hidden>
>>>
>>> Spotted by ASAN.
>>>
>>> Reported-by: Euler Robot <address@hidden>
>>> Signed-off-by: Pan Nengyuan <address@hidden>
>>> ---
>>> Changes V2 to V1:
>>> - use a "goto cleanup", instead of duplicating the "free" functions.
>>> - free "dest_cmdline" at the end.
>>> ---
>>
>> Reviewed-by: Thomas Huth <address@hidden>
>>
>> ... and picked up to my qtest-next tree.
>
> ... and now I had to unqueue the patch again. It is reproducibly causing
> one of the gitlab CI pipelines to fail with a timeout, e.g.:
>
> https://gitlab.com/huth/qemu/-/jobs/400101552
>
> Not sure what is going on here, though, there is no obvious error
> message in the output... this needs some more investigation... do you
> have a gitlab account and could have a look?
>
OK, I will register a account and have a look.
> Thomas
>
> .
>