diff mbox series

block/blkdebug: fix memory leak

Message ID 1e903f928eb3da332cc95e2a6f87243bd9fe66e4.camel@gmail.com
State New
Headers show
Series block/blkdebug: fix memory leak | expand

Commit Message

Elena Afanasova Oct. 9, 2020, 7:09 p.m. UTC 
Spotted by PVS-Studio

Signed-off-by: Elena Afanasova <eafanasova@gmail.com>
---
 block/blkdebug.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Eric Blake Oct. 9, 2020, 7:56 p.m. UTC | #1 
On 10/9/20 2:09 PM, Elena Afanasova wrote:
> Spotted by PVS-Studio

> 

> Signed-off-by: Elena Afanasova <eafanasova@gmail.com>

> ---

>  block/blkdebug.c | 1 +

>  1 file changed, 1 insertion(+)


Reviewed-by: Eric Blake <eblake@redhat.com>


> 

> diff --git a/block/blkdebug.c b/block/blkdebug.c

> index eecbf3e5c4..54da719dd1 100644

> --- a/block/blkdebug.c

> +++ b/block/blkdebug.c

> @@ -215,6 +215,7 @@ static int add_rule(void *opaque, QemuOpts *opts, Error **errp)

>                                   BLKDEBUG_IO_TYPE__MAX, &local_error);

>          if (local_error) {

>              error_propagate(errp, local_error);

> +            g_free(rule);

>              return -1;

>          }

>          if (iotype != BLKDEBUG_IO_TYPE__MAX) {

> 


-- 
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org
Laurent Vivier Oct. 12, 2020, 2:38 p.m. UTC | #2 
Le 09/10/2020 à 21:09, Elena Afanasova a écrit :
> Spotted by PVS-Studio
> 
> Signed-off-by: Elena Afanasova <eafanasova@gmail.com>
> ---
>  block/blkdebug.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/block/blkdebug.c b/block/blkdebug.c
> index eecbf3e5c4..54da719dd1 100644
> --- a/block/blkdebug.c
> +++ b/block/blkdebug.c
> @@ -215,6 +215,7 @@ static int add_rule(void *opaque, QemuOpts *opts, Error **errp)
>                                   BLKDEBUG_IO_TYPE__MAX, &local_error);
>          if (local_error) {
>              error_propagate(errp, local_error);
> +            g_free(rule);
>              return -1;
>          }
>          if (iotype != BLKDEBUG_IO_TYPE__MAX) {
> 

Applied to my trivial-patches branch.

Thanks,
Laurent
diff mbox series

Patch

diff --git a/block/blkdebug.c b/block/blkdebug.c
index eecbf3e5c4..54da719dd1 100644
--- a/block/blkdebug.c
+++ b/block/blkdebug.c
@@ -215,6 +215,7 @@  static int add_rule(void *opaque, QemuOpts *opts, Error **errp)
                                  BLKDEBUG_IO_TYPE__MAX, &local_error);
         if (local_error) {
             error_propagate(errp, local_error);
+            g_free(rule);
             return -1;
         }
         if (iotype != BLKDEBUG_IO_TYPE__MAX) {