qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] tests/qtest/fuzz: fix memleak in qos_fuzz.c
@ 2024-05-21 10:31 Dmitry Frolov
  2024-06-11 12:31 ` Дмитрий Фролов
                   ` (2 more replies)
  0 siblings, 3 replies; 4+ messages in thread
From: Dmitry Frolov @ 2024-05-21 10:31 UTC (permalink / raw)
  To: alxndr; +Cc: qemu-devel, qemu-trivial, sdl.qemu, Dmitry Frolov

Found with fuzzing for qemu-8.2, but also relevant for master

Signed-off-by: Dmitry Frolov <frolov@swemel.ru>
---
 tests/qtest/fuzz/qos_fuzz.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/qtest/fuzz/qos_fuzz.c b/tests/qtest/fuzz/qos_fuzz.c
index b71e945c5f..d3839bf999 100644
--- a/tests/qtest/fuzz/qos_fuzz.c
+++ b/tests/qtest/fuzz/qos_fuzz.c
@@ -180,6 +180,7 @@ static void walk_path(QOSGraphNode *orig_path, int len)
 
         fuzz_path_vec = path_vec;
     } else {
+        g_string_free(cmd_line, true);
         g_free(path_vec);
     }
 
-- 
2.43.0



^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] tests/qtest/fuzz: fix memleak in qos_fuzz.c
  2024-05-21 10:31 [PATCH] tests/qtest/fuzz: fix memleak in qos_fuzz.c Dmitry Frolov
@ 2024-06-11 12:31 ` Дмитрий Фролов
  2024-06-13 10:04 ` Thomas Huth
  2024-06-13 15:55 ` Alexander Bulekov
  2 siblings, 0 replies; 4+ messages in thread
From: Дмитрий Фролов @ 2024-06-11 12:31 UTC (permalink / raw)
  To: alxndr; +Cc: qemu-devel, qemu-trivial, sdl.qemu

ping

https://patchew.org/QEMU/20240521103106.119021-3-frolov@swemel.ru/

On 21.05.2024 13:31, Dmitry Frolov wrote:
> Found with fuzzing for qemu-8.2, but also relevant for master
>
> Signed-off-by: Dmitry Frolov <frolov@swemel.ru>
> ---
>   tests/qtest/fuzz/qos_fuzz.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/tests/qtest/fuzz/qos_fuzz.c b/tests/qtest/fuzz/qos_fuzz.c
> index b71e945c5f..d3839bf999 100644
> --- a/tests/qtest/fuzz/qos_fuzz.c
> +++ b/tests/qtest/fuzz/qos_fuzz.c
> @@ -180,6 +180,7 @@ static void walk_path(QOSGraphNode *orig_path, int len)
>   
>           fuzz_path_vec = path_vec;
>       } else {
> +        g_string_free(cmd_line, true);
>           g_free(path_vec);
>       }
>   



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] tests/qtest/fuzz: fix memleak in qos_fuzz.c
  2024-05-21 10:31 [PATCH] tests/qtest/fuzz: fix memleak in qos_fuzz.c Dmitry Frolov
  2024-06-11 12:31 ` Дмитрий Фролов
@ 2024-06-13 10:04 ` Thomas Huth
  2024-06-13 15:55 ` Alexander Bulekov
  2 siblings, 0 replies; 4+ messages in thread
From: Thomas Huth @ 2024-06-13 10:04 UTC (permalink / raw)
  To: Dmitry Frolov, alxndr; +Cc: qemu-devel, qemu-trivial, sdl.qemu

On 21/05/2024 12.31, Dmitry Frolov wrote:
> Found with fuzzing for qemu-8.2, but also relevant for master
> 
> Signed-off-by: Dmitry Frolov <frolov@swemel.ru>
> ---
>   tests/qtest/fuzz/qos_fuzz.c | 1 +
>   1 file changed, 1 insertion(+)
> 
> diff --git a/tests/qtest/fuzz/qos_fuzz.c b/tests/qtest/fuzz/qos_fuzz.c
> index b71e945c5f..d3839bf999 100644
> --- a/tests/qtest/fuzz/qos_fuzz.c
> +++ b/tests/qtest/fuzz/qos_fuzz.c
> @@ -180,6 +180,7 @@ static void walk_path(QOSGraphNode *orig_path, int len)
>   
>           fuzz_path_vec = path_vec;
>       } else {
> +        g_string_free(cmd_line, true);
>           g_free(path_vec);
>       }
>   

Reviewed-by: Thomas Huth <thuth@redhat.com>



^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] tests/qtest/fuzz: fix memleak in qos_fuzz.c
  2024-05-21 10:31 [PATCH] tests/qtest/fuzz: fix memleak in qos_fuzz.c Dmitry Frolov
  2024-06-11 12:31 ` Дмитрий Фролов
  2024-06-13 10:04 ` Thomas Huth
@ 2024-06-13 15:55 ` Alexander Bulekov
  2 siblings, 0 replies; 4+ messages in thread
From: Alexander Bulekov @ 2024-06-13 15:55 UTC (permalink / raw)
  To: Dmitry Frolov; +Cc: qemu-devel, qemu-trivial, sdl.qemu

Reviewed-by: Alexander Bulekov <alxndr@bu.edu>

On 240521 1331, Dmitry Frolov wrote:
> Found with fuzzing for qemu-8.2, but also relevant for master
> 
> Signed-off-by: Dmitry Frolov <frolov@swemel.ru>
> ---
>  tests/qtest/fuzz/qos_fuzz.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/tests/qtest/fuzz/qos_fuzz.c b/tests/qtest/fuzz/qos_fuzz.c
> index b71e945c5f..d3839bf999 100644
> --- a/tests/qtest/fuzz/qos_fuzz.c
> +++ b/tests/qtest/fuzz/qos_fuzz.c
> @@ -180,6 +180,7 @@ static void walk_path(QOSGraphNode *orig_path, int len)
>  
>          fuzz_path_vec = path_vec;
>      } else {
> +        g_string_free(cmd_line, true);
>          g_free(path_vec);
>      }
>  
> -- 
> 2.43.0
> 


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2024-06-13 15:55 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-05-21 10:31 [PATCH] tests/qtest/fuzz: fix memleak in qos_fuzz.c Dmitry Frolov
2024-06-11 12:31 ` Дмитрий Фролов
2024-06-13 10:04 ` Thomas Huth
2024-06-13 15:55 ` Alexander Bulekov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).