diff options
| author | Mikael Kanstrup <mikael.kanstrup@gmail.com> | 2016-03-19 09:36:57 +0100 |
|---|---|---|
| committer | Anders Broman <a.broman58@gmail.com> | 2016-12-25 08:00:25 +0000 |
| commit | 802362ee1a44d4a6494092f40e61155569ee5a6a (patch) | |
| tree | 7fe4f92ff37ac3e9ba2f334404fc9e1a929c5fd3 /ui/iface_lists.c | |
| parent | 194433a503bf1f90f1037e6c1e1e97303f6c3ffa (diff) | |
Avoid recursive scan_local_interfaces operation
When the local networks interfaces changes quickly or when refreshing the
list of network interfaces there's a risk of recursive calls into
scan_local_interfaces. The recursive calls are a result of calling
update_cb to process UI events during function operation which in turn
again discover a network interface change. This results in strange
duplicate entries of network interfaces and crashes.
To avoid recursive calls a check is added to stop running the function while
already updating. This patch is really just a workaround for the problem.
Ideally some asynchronous operation should be implemented instead to avoid
the UI update_cb callback alltogether.
Bug: 11553
Bug: 12263
Change-Id: I3b74d8f196677e0e261a395aff558dd9f685b538
Reviewed-on: https://code.wireshark.org/review/14492
Reviewed-by: Michael Mann <mmann78@netscape.net>
Petri-Dish: Michael Mann <mmann78@netscape.net>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Anders Broman <a.broman58@gmail.com>
Diffstat (limited to 'ui/iface_lists.c')
| -rw-r--r-- | ui/iface_lists.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/ui/iface_lists.c b/ui/iface_lists.c index 429cc0f67e..b53c2cb955 100644 --- a/ui/iface_lists.c +++ b/ui/iface_lists.c @@ -110,7 +110,17 @@ scan_local_interfaces(void (*update_cb)(void)) GString *ip_str; interface_options interface_opts; gboolean found = FALSE; + static gboolean running = FALSE; + if (running) { + /* scan_local_interfaces internally calls update_cb to process UI events + to avoid stuck UI while running possibly slow operations. A side effect + of this is that new interface changes can be detected before completing + the last one. + This return avoids recursive scan_local_interfaces operation. */ + return; + } + running = TRUE; if (global_capture_opts.all_ifaces->len > 0) { for (i = (int)global_capture_opts.all_ifaces->len-1; i >= 0; i--) { @@ -361,6 +371,7 @@ scan_local_interfaces(void (*update_cb)(void)) global_capture_opts.num_selected++; } } + running = FALSE; } /* |