[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qga: fix 'driver' leak in guest-get-fsinfo
From: |
Michael Roth |
Subject: |
Re: [Qemu-devel] [PATCH] qga: fix 'driver' leak in guest-get-fsinfo |
Date: |
Fri, 06 Jul 2018 08:21:07 -0500 |
User-agent: |
alot/0.6 |
Quoting Marc-André Lureau (2018-07-05 11:16:29)
> 'driver' is leaked when the loop is not broken.
>
> Leak introduced by commit 743c71d03c20d64f2bae5fba6f26cdf5e4b1bda6,
> spotted by ASAN.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
Thanks, applied to qga tree:
https://github.com/mdroth/qemu/commits/qga
> ---
> qga/commands-posix.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/qga/commands-posix.c b/qga/commands-posix.c
> index 233f78a406..c46767b0dd 100644
> --- a/qga/commands-posix.c
> +++ b/qga/commands-posix.c
> @@ -890,6 +890,7 @@ static void build_guest_fsinfo_for_real_device(char const
> *syspath,
> break;
> }
>
> + g_free(driver);
> if (sscanf(p, "/%x:%x:%x.%x%n",
> pci, pci + 1, pci + 2, pci + 3, &pcilen) == 4) {
> p += pcilen;
> --
> 2.18.0.rc1
>