aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMartin Kaiser <wireshark@kaiser.cx>2015-08-11 21:57:11 +0200
committerGerald Combs <gerald@wireshark.org>2015-08-11 21:27:13 +0000
commit14c75250ab484b8ff872a88f6c9a826dd3cce4d8 (patch)
tree3879a2175f8f30ce174d3572decd14bab809f482
parentf1a4db34b3144c0fe45ee5a636ae11aa061f0d6d (diff)
[Qt Expert Info dialogue] remove our tap listener when the dialogue is destroyed
if we don't do this, there'll be a dangling tap listener and we'll see a crash when its reset method is called this can be triggered by opening another expert info dialogue, this will invoke cf_retap_packets() and reset_tap_listeners() which in turn calls the reset method of each registered tap listener Bug: 11432 Change-Id: I8fc13351666c875e1a3641f31bada8e80d167ab2 Reviewed-on: https://code.wireshark.org/review/9979 Petri-Dish: Martin Kaiser <wireshark@kaiser.cx> Reviewed-by: Gerald Combs <gerald@wireshark.org>
-rw-r--r--ui/qt/expert_info_dialog.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/ui/qt/expert_info_dialog.cpp b/ui/qt/expert_info_dialog.cpp
index 2d7ecfc96a..e1423e7b3d 100644
--- a/ui/qt/expert_info_dialog.cpp
+++ b/ui/qt/expert_info_dialog.cpp
@@ -205,6 +205,7 @@ ExpertInfoDialog::ExpertInfoDialog(QWidget &parent, CaptureFile &capture_file) :
ExpertInfoDialog::~ExpertInfoDialog()
{
+ remove_tap_listener(this);
delete ui;
}