diff options
| author | Evan Huus <eapache@gmail.com> | 2014-05-09 15:16:14 +0000 |
|---|---|---|
| committer | Evan Huus <eapache@gmail.com> | 2014-05-09 15:19:59 +0000 |
| commit | 877942c3fd1523e2ea9a6a820ae57514a4e2fad8 (patch) | |
| tree | e3c933a5ec2955bae22deb06a55baf62f4a16f72 /ui/gtk | |
| parent | 532c3bc7837ba739c5a290a2db3a44fbe08d25c0 (diff) | |
Revert "Use g_ alloced memory (instead of emem) when printing fields. emem shouldn't be used for GUI functions."
It's causing a few different test failures - I've tracked down at least one of them, but the others are weirder and will require more digging.
This reverts commit 9edba650d11776fa1d5aefe4820890898380c1c2.
Change-Id: I897f8cf1cfbb2a189b2054e5002f59757befa47f
Reviewed-on: https://code.wireshark.org/review/1575
Reviewed-by: Evan Huus <eapache@gmail.com>
Diffstat (limited to 'ui/gtk')
| -rw-r--r-- | ui/gtk/main.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/ui/gtk/main.c b/ui/gtk/main.c index e49e0531a0..56264fb305 100644 --- a/ui/gtk/main.c +++ b/ui/gtk/main.c @@ -668,9 +668,8 @@ copy_selected_plist_cb(GtkWidget *w _U_, gpointer data _U_, COPY_SELECTED_E acti break; case COPY_SELECTED_VALUE: if (cfile.edt !=0 ) { - gchar* field_str = get_node_field_value(cfile.finfo_selected, cfile.edt); - g_string_append(gtk_text_str, field_str); - g_free(field_str); + g_string_append(gtk_text_str, + get_node_field_value(cfile.finfo_selected, cfile.edt)); } break; default: |