diff options
author | Michael Mann <mmann78@netscape.net> | 2014-05-08 22:33:29 -0400 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2014-05-09 04:31:10 +0000 |
commit | 9edba650d11776fa1d5aefe4820890898380c1c2 (patch) | |
tree | a752b0e58d7343fac194034bcad9734f749628eb /ui/gtk | |
parent | f5a0d7e89753622bb11a6e27292c459a87ca9c67 (diff) |
Use g_ alloced memory (instead of emem) when printing fields. emem shouldn't be used for GUI functions.
emem was exposed because of its memory limits trying to output PDML for a very large byte field in a capture file.
bug:10081
Change-Id: I6346dfdfb5f6381e16761a99291c4be7851185d9
Reviewed-on: https://code.wireshark.org/review/1566
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'ui/gtk')
-rw-r--r-- | ui/gtk/main.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/ui/gtk/main.c b/ui/gtk/main.c index 6635c69b2e..a5b345edd5 100644 --- a/ui/gtk/main.c +++ b/ui/gtk/main.c @@ -668,8 +668,9 @@ copy_selected_plist_cb(GtkWidget *w _U_, gpointer data _U_, COPY_SELECTED_E acti break; case COPY_SELECTED_VALUE: if (cfile.edt !=0 ) { - g_string_append(gtk_text_str, - get_node_field_value(cfile.finfo_selected, cfile.edt)); + gchar* field_str = get_node_field_value(cfile.finfo_selected, cfile.edt); + g_string_append(gtk_text_str, field_str); + g_free(field_str); } break; default: |