[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PULL 13/27] hw/xen: Add xenstore operations to allow redirection to
From: |
Peter Maydell |
Subject: |
Re: [PULL 13/27] hw/xen: Add xenstore operations to allow redirection to internal emulation |
Date: |
Tue, 4 Apr 2023 18:35:38 +0100 |
On Tue, 7 Mar 2023 at 18:27, David Woodhouse <dwmw2@infradead.org> wrote:
>
> From: Paul Durrant <pdurrant@amazon.com>
>
> Signed-off-by: Paul Durrant <pdurrant@amazon.com>
> Signed-off-by: David Woodhouse <dwmw@amazon.co.uk>
> Reviewed-by: Paul Durrant <paul@xen.org>
> ---
Hi; Coverity points out a memory leak in this code (CID 1508098):
> +static struct qemu_xs_handle *libxenstore_open(void)
> +{
> + struct xs_handle *xsh = xs_open(0);
> + struct qemu_xs_handle *h = g_new0(struct qemu_xs_handle, 1);
Here we allocate memory...
> +
> + if (!xsh) {
> + return NULL;
...but here we can return without freeing it...
> + }
> +
> + h = g_new0(struct qemu_xs_handle, 1);
...and here we allocate a second time and overwrite the
pointer to the first allocation.
Deleting the first call to g_new0() would fix both of these.
> + h->xsh = xsh;
> +
> + notifier_list_init(&h->notifiers);
> + qemu_set_fd_handler(xs_fileno(h->xsh), watch_event, NULL, h);
> +
> + return h;
> +}
thanks
-- PMM
- Re: [PULL 13/27] hw/xen: Add xenstore operations to allow redirection to internal emulation,
Peter Maydell <=