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