diff options
author | Gerald Combs <gerald@wireshark.org> | 2016-06-22 12:44:02 -0700 |
---|---|---|
committer | Anders Broman <a.broman58@gmail.com> | 2016-06-23 03:44:54 +0000 |
commit | 300e055ab6e3ede51de2258c83d9aeab4b7f3701 (patch) | |
tree | 0db3e2c2984b3e3a7d47207cc13c35de0ecc77e5 /ui/qt/main_window.h | |
parent | 933c079598079e1786b07cda5e1b48684ca0ac3c (diff) | |
download | wireshark-300e055ab6e3ede51de2258c83d9aeab4b7f3701.tar.gz |
Qt: Use queued signals+slots for filter actions
Many of our dialogs can send an "apply this display filter" signal to
the main window. Applying a display filter in turn creates a nested
event loop via cf_read+update_progress_dlg. If the "apply" signal+slot
is directly connected (which is the default, and which means we're
calling into a function table) we can close the dialog while the signal
is firing, which means we return into a deleted object.
Make all of the filterAction signals+slots queued instead of direct.
Bug: 12523
Change-Id: Ica331054c0aa52a7f33bd8df1fa65ecd09fdc292
Reviewed-on: https://code.wireshark.org/review/16080
Reviewed-by: Gerald Combs <gerald@wireshark.org>
Petri-Dish: Gerald Combs <gerald@wireshark.org>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'ui/qt/main_window.h')
-rw-r--r-- | ui/qt/main_window.h | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/ui/qt/main_window.h b/ui/qt/main_window.h index c112c75222..63295d28d2 100644 --- a/ui/qt/main_window.h +++ b/ui/qt/main_window.h @@ -231,6 +231,7 @@ signals: void reloadFields(); void packetInfoChanged(struct _packet_info *pinfo); void fieldFilterChanged(const QByteArray field_filter); + void filterAction(QString filter, FilterAction::Action action, FilterAction::ActionType type); public slots: // in main_window_slots.cpp @@ -329,7 +330,7 @@ private slots: void displayFilterButtonClicked(); // Handle FilterAction signals - void filterAction(QString& filter, FilterAction::Action action, FilterAction::ActionType type); + void queuedFilterAction(QString filter, FilterAction::Action action, FilterAction::ActionType type); /** Pass stat cmd arguments to a slot. * @param menu_path slot Partial slot name, e.g. "StatisticsIOGraph". |