diff options
| author | Marc-André Lureau <marcandre.lureau@redhat.com> | 2018-02-19 18:27:15 +0100 |
|---|---|---|
| committer | Greg Kurz <groug@kaod.org> | 2018-02-19 18:27:32 +0100 |
| commit | 6ce7177ae2e547999ef9e6f467415963991e28da (patch) | |
| tree | ca2e6e60255e48b1119b587a4cf5102d1bdd2c5b /hw/9pfs | |
| parent | e446a1eb5e358e47193a03a7a5f3ce9b8b5e36ea (diff) | |
| download | qemu-6ce7177ae2e547999ef9e6f467415963991e28da.zip qemu-6ce7177ae2e547999ef9e6f467415963991e28da.tar.gz qemu-6ce7177ae2e547999ef9e6f467415963991e28da.tar.bz2 | |
9p: fix leak in synth_name_to_path()
Leak found thanks to ASAN:
Direct leak of 8 byte(s) in 1 object(s) allocated from:
#0 0x55995789ac90 in __interceptor_malloc (/home/elmarco/src/qemu/build/x86_64-softmmu/qemu-system-x86_64+0x1510c90)
#1 0x7f0a91190f0c in g_malloc /home/elmarco/src/gnome/glib/builddir/../glib/gmem.c:94
#2 0x5599580a281c in v9fs_path_copy /home/elmarco/src/qemu/hw/9pfs/9p.c:196:17
#3 0x559958f9ec5d in coroutine_trampoline /home/elmarco/src/qemu/util/coroutine-ucontext.c:116:9
#4 0x7f0a8766ebbf (/lib64/libc.so.6+0x50bbf)
Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>
Signed-off-by: Greg Kurz <groug@kaod.org>
Diffstat (limited to 'hw/9pfs')
| -rw-r--r-- | hw/9pfs/9p-synth.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/hw/9pfs/9p-synth.c b/hw/9pfs/9p-synth.c index 18082df..54239c9 100644 --- a/hw/9pfs/9p-synth.c +++ b/hw/9pfs/9p-synth.c @@ -495,6 +495,7 @@ static int synth_name_to_path(FsContext *ctx, V9fsPath *dir_path, } out: /* Copy the node pointer to fid */ + g_free(target->data); target->data = g_memdup(&node, sizeof(void *)); target->size = sizeof(void *); return 0; |