diff mbox series

tests/qtest/fuzz: fix memleak in qos_fuzz.c

Message ID 20240521103106.119021-3-frolov@swemel.ru
State New
Headers show
Series tests/qtest/fuzz: fix memleak in qos_fuzz.c | expand

Commit Message

Dmitry Frolov May 21, 2024, 10:31 a.m. UTC 
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(+)

Comments

Dmitry Frolov June 11, 2024, 12:31 p.m. UTC | #1 
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);
>       }
>
Thomas Huth June 13, 2024, 10:04 a.m. UTC | #2 
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>
Alexander Bulekov June 13, 2024, 3:55 p.m. UTC | #3 
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
>
diff mbox series

Patch

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);
     }