[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 13/13] tests: fix coroutine leak in /basic/en
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v2 13/13] tests: fix coroutine leak in /basic/entered |
Date: |
Mon, 18 Dec 2017 13:25:09 +0000 |
User-agent: |
Mutt/1.9.1 (2017-09-22) |
On Fri, Dec 15, 2017 at 04:06:59PM +0100, Marc-André Lureau wrote:
> The coroutine is not finished by the time the test ends, resulting in
> ASAN warning:
>
> ==7005==ERROR: LeakSanitizer: detected memory leaks
>
> Direct leak of 312 byte(s) in 1 object(s) allocated from:
> #0 0x7fd35290fa38 in __interceptor_calloc (/lib64/libasan.so.4+0xdea38)
> #1 0x7fd3506c5f75 in g_malloc0 ../glib/gmem.c:124
> #2 0x55994af03e47 in qemu_coroutine_new
> /home/elmarco/src/qemu/util/coroutine-ucontext.c:144
> #3 0x55994aefed99 in qemu_coroutine_create
> /home/elmarco/src/qemu/util/qemu-coroutine.c:76
> #4 0x55994ac1eb50 in verify_entered_step_1
> /home/elmarco/src/qemu/tests/test-coroutine.c:80
> #5 0x55994af03c75 in coroutine_trampoline
> /home/elmarco/src/qemu/util/coroutine-ucontext.c:119
> #6 0x7fd34ec02bef (/lib64/libc.so.6+0x50bef)
>
> Do not yield() to let the coroutine terminate.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> tests/test-coroutine.c | 1 -
> 1 file changed, 1 deletion(-)
Looks like a copy-paste mistake, the second yield shouldn't be there.
Reviewed-by: Stefan Hajnoczi <address@hidden>
signature.asc
Description: PGP signature
- Re: [Qemu-devel] [PATCH v2 08/13] tests: fix migration-test leak, (continued)
- [Qemu-devel] [PATCH v2 11/13] tests: fix qmp-test leak, Marc-André Lureau, 2017/12/15
- [Qemu-devel] [PATCH v2 10/13] qemu-config: fix leak in query-command-line-options, Marc-André Lureau, 2017/12/15
- [Qemu-devel] [PATCH v2 12/13] ucontext: annotate coroutine stack for ASAN, Marc-André Lureau, 2017/12/15
- [Qemu-devel] [PATCH v2 13/13] tests: fix coroutine leak in /basic/entered, Marc-André Lureau, 2017/12/15
- Re: [Qemu-devel] [PATCH v2 13/13] tests: fix coroutine leak in /basic/entered,
Stefan Hajnoczi <=
- Re: [Qemu-devel] [PATCH v2 00/13] Various build-sys and ASAN related fixes, Marc-André Lureau, 2017/12/15