[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/4] tests: fix bdrv-drain leak
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH 2/4] tests: fix bdrv-drain leak |
Date: |
Sat, 11 Aug 2018 16:03:24 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 |
On 2018-08-10 08:28, Markus Armbruster wrote:
> Marc-André Lureau <address@hidden> writes:
>
>> Spotted by ASAN:
>>
>> =================================================================
>> ==5378==ERROR: LeakSanitizer: detected memory leaks
>>
>> Direct leak of 65536 byte(s) in 1 object(s) allocated from:
>> #0 0x7f788f83bc48 in malloc (/lib64/libasan.so.5+0xeec48)
>> #1 0x7f788c9923c5 in g_malloc (/lib64/libglib-2.0.so.0+0x523c5)
>> #2 0x5622a1fe37bc in coroutine_trampoline
>> /home/elmarco/src/qq/util/coroutine-ucontext.c:116
>> #3 0x7f788a15d75f in __correctly_grouped_prefixwc
>> (/lib64/libc.so.6+0x4c75f)
>>
>> Signed-off-by: Marc-André Lureau <address@hidden>
>> ---
>> tests/test-bdrv-drain.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/tests/test-bdrv-drain.c b/tests/test-bdrv-drain.c
>> index 17bb8508ae..abc8bbe6f0 100644
>> --- a/tests/test-bdrv-drain.c
>> +++ b/tests/test-bdrv-drain.c
>> @@ -948,6 +948,7 @@ static void coroutine_fn test_co_delete_by_drain(void
>> *opaque)
>> }
>>
>> dbdd->done = true;
>> + g_free(buffer);
>> }
>>
>> /**
>
> Screwed up in commit 4c8158e359d.
>
> Max, could the coroutine's stack accomodate the buffer?
I don't know. How big is a coroutine's stack? The buffer size
apparently is 64k. (Which is my opinion is generally too much to put on
any stack.)
But why would you want to put it on the stack anyway? It's not like it
would make the patch any simpler.
Max
signature.asc
Description: OpenPGP digital signature
Re: [Qemu-devel] [Qemu-block] [PATCH 2/4] tests: fix bdrv-drain leak, Max Reitz, 2018/08/13
[Qemu-devel] [PATCH 3/4] monitor: fix oob command leak, Marc-André Lureau, 2018/08/09
[Qemu-devel] [PATCH 4/4] RFC: fix megasas leak, Marc-André Lureau, 2018/08/09
Re: [Qemu-devel] [PATCH 0/4] Memory leak fixes, Eric Blake, 2018/08/09