diff options
author | Peter Wu <peter@lekensteyn.nl> | 2017-06-12 16:21:29 +0200 |
---|---|---|
committer | Peter Wu <peter@lekensteyn.nl> | 2017-06-17 14:22:56 +0000 |
commit | 553b1e0215ad5826906aafbe0e38c1e44cf9e1c3 (patch) | |
tree | 3cf0b3d5d3e668785146f64b099bcce10d792ae8 /wsutil/nstime.h | |
parent | 800a856fb4784e31347e15a8ec825d2daeb62350 (diff) | |
download | wireshark-553b1e0215ad5826906aafbe0e38c1e44cf9e1c3.tar.gz |
Do not process UI events while reading from live capture
For at least Qt, the main_window_update callback is not necessary to
make the stop button work. When restarting a live capture during a
flood (via Ctrl-R), this callback actually results in an infinite loop
in MainWindow::captureStop since the capture state never changes from
FILE_READ_IN_PROGRESS.
Remove this callback to ensure that the problematic
pipeActivated / sync_pipe_input_cb / capture_input_new_packets /
main_window_update / ... / on_actionCaptureRestart_triggered /
testCaptureFileClose / captureStop sequence is avoided.
Even though captureStop invokes capture_stop, I guess that this does not
change the state because the pipeActivated callback is already active.
Bug: 10917
Change-Id: I6ca4fa946963928b7bc8a53ca14f9a9a3a35eaa7
Reviewed-on: https://code.wireshark.org/review/22097
Reviewed-by: Jaap Keuter <jaap.keuter@xs4all.nl>
Reviewed-by: Peter Wu <peter@lekensteyn.nl>
Diffstat (limited to 'wsutil/nstime.h')
0 files changed, 0 insertions, 0 deletions