|
From: | Philippe Mathieu-Daudé |
Subject: | Re: [PATCH] ui/cocoa: Fix the leak of qemu_console_get_label |
Date: | Tue, 15 Feb 2022 09:02:08 +0100 |
User-agent: | Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:91.0) Gecko/20100101 Thunderbird/91.5.1 |
On 13/2/22 12:11, BALATON Zoltan wrote:
On Sun, 13 Feb 2022, Akihiko Odaki wrote:Signed-off-by: Akihiko Odaki <akihiko.odaki@gmail.com> --- ui/cocoa.m | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/ui/cocoa.m b/ui/cocoa.m index ac18e14ce01..fdf52a7c2f7 100644 --- a/ui/cocoa.m +++ b/ui/cocoa.m @@ -1680,7 +1680,10 @@ static void create_initial_menus(void) /* Returns a name for a given console */ static NSString * getConsoleName(QemuConsole * console) {- return [NSString stringWithFormat: @"%s", qemu_console_get_label(console)];+ char *label = qemu_console_get_label(console);I guess you could do g_autofree char *label to save a g_free but not a big deal and only saves one line here so it's also good as it is.
Good idea. Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
[Prev in Thread] | Current Thread | [Next in Thread] |