[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] vhost-user-test: fix a memory leak
From: |
Pan Nengyuan |
Subject: |
Re: [PATCH] vhost-user-test: fix a memory leak |
Date: |
Thu, 12 Dec 2019 09:41:08 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2 |
On 2019/12/11 15:57, Thomas Huth wrote:
> Hi!
>
> On 11/12/2019 01.55, address@hidden wrote:
> [...]
>> diff --git a/tests/vhost-user-test.c b/tests/vhost-user-test.c
>> index 91ea373..54be931 100644
>> --- a/tests/vhost-user-test.c
>> +++ b/tests/vhost-user-test.c
>> @@ -717,6 +717,8 @@ static void test_migrate(void *obj, void *arg,
>> QGuestAllocator *alloc)
>> guint64 size;
>>
>> if (!wait_for_fds(s)) {
>> + g_free(uri);
>> + test_server_free(dest);
>> return;
>> }
>
> Well spotted. But I'd prefer to rather move the allocation of these
> resources after the if-statement instead of doing the allocation at the
> declaration of the variables already. Or maybe use a "goto out" and jump
> to the end of the function instead? ... whatever you prefer, but
> duplicating the "free" functions sounds like a cumbersome solution to me.
>
Yes, I think use a "goto out" is more better. I will change it.
Thanks.
> Thanks,
> Thomas
>
>