diff mbox

[v4,1/6] tests: fix memory leak in virtio-scsi-test

Message ID 1476435535-9408-2-git-send-email-lvivier@redhat.com
State New
Headers show

Commit Message

Laurent Vivier Oct. 14, 2016, 8:58 a.m. UTC
vs is allocated in qvirtio_scsi_pci_init() and never freed.

Signed-off-by: Laurent Vivier <lvivier@redhat.com>
---
 tests/virtio-scsi-test.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Greg Kurz Oct. 14, 2016, 4:18 p.m. UTC | #1
On Fri, 14 Oct 2016 10:58:50 +0200
Laurent Vivier <lvivier@redhat.com> wrote:

> vs is allocated in qvirtio_scsi_pci_init() and never freed.
> 
> Signed-off-by: Laurent Vivier <lvivier@redhat.com>
> ---

Reviewed-by: Greg Kurz <groug@kaod.org>

>  tests/virtio-scsi-test.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/virtio-scsi-test.c b/tests/virtio-scsi-test.c
> index 79088bb..94d75b1 100644
> --- a/tests/virtio-scsi-test.c
> +++ b/tests/virtio-scsi-test.c
> @@ -64,6 +64,7 @@ static void qvirtio_scsi_pci_free(QVirtIOSCSI *vs)
>      qvirtio_pci_device_disable(container_of(vs->dev, QVirtioPCIDevice, vdev));
>      g_free(vs->dev);
>      qpci_free_pc(vs->bus);
> +    g_free(vs);
>  }
>  
>  static uint64_t qvirtio_scsi_alloc(QVirtIOSCSI *vs, size_t alloc_size,
diff mbox

Patch

diff --git a/tests/virtio-scsi-test.c b/tests/virtio-scsi-test.c
index 79088bb..94d75b1 100644
--- a/tests/virtio-scsi-test.c
+++ b/tests/virtio-scsi-test.c
@@ -64,6 +64,7 @@  static void qvirtio_scsi_pci_free(QVirtIOSCSI *vs)
     qvirtio_pci_device_disable(container_of(vs->dev, QVirtioPCIDevice, vdev));
     g_free(vs->dev);
     qpci_free_pc(vs->bus);
+    g_free(vs);
 }
 
 static uint64_t qvirtio_scsi_alloc(QVirtIOSCSI *vs, size_t alloc_size,