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/qt | |
| 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/qt')
| -rw-r--r-- | ui/qt/main_window_slots.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/ui/qt/main_window_slots.cpp b/ui/qt/main_window_slots.cpp index be4a5ad8a0..7bbb4486d9 100644 --- a/ui/qt/main_window_slots.cpp +++ b/ui/qt/main_window_slots.cpp @@ -1424,9 +1424,7 @@ void MainWindow::actionEditCopyTriggered(MainWindow::CopySelected selection_type break; case CopySelectedValue: if (cap_file_->edt != 0) { - gchar* field_str = get_node_field_value(cap_file_->finfo_selected, cap_file_->edt); - clip.append(field_str); - g_free(field_str); + clip.append(get_node_field_value(cap_file_->finfo_selected, cap_file_->edt)); } break; } |