diff options
author | Martin Kaiser <wireshark@kaiser.cx> | 2015-08-11 21:57:11 +0200 |
---|---|---|
committer | Gerald Combs <gerald@wireshark.org> | 2015-08-11 21:27:13 +0000 |
commit | 14c75250ab484b8ff872a88f6c9a826dd3cce4d8 (patch) | |
tree | 3879a2175f8f30ce174d3572decd14bab809f482 | |
parent | f1a4db34b3144c0fe45ee5a636ae11aa061f0d6d (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.cpp | 1 |
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; } |