]> granicus.if.org Git - graphviz/commitdiff
smyrna: fix memory leak in 'mTestgvpr'
authorMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 9 Jul 2022 15:17:44 +0000 (08:17 -0700)
committerMatthew Fernandez <matthew.fernandez@gmail.com>
Sat, 16 Jul 2022 00:06:35 +0000 (17:06 -0700)
Even when a text field is empty, calling `gtk_text_buffer_get_text` returns a
dynamically allocated (empty) string that must be freed.

cmd/smyrna/gui/menucallbacks.c

index 1191312f4a3a3b36f7dde67559d005648644f718..aa806a0a72b166a186c8206dd631f77b79d11acb 100644 (file)
@@ -286,8 +286,10 @@ void mTestgvpr(GtkWidget * widget, gpointer user_data)
     gtk_text_buffer_get_end_iter(gtkbuf, &endit);
     bf2 = gtk_text_buffer_get_text(gtkbuf, &startit, &endit, 0);
 
-    if (*args == '\0' && *bf2 == '\0')
+    if (*args == '\0' && *bf2 == '\0') {
+       g_free(bf2);
        return;
+    }
 
     argc = 1;
     if (*args != '\0')