diff options
author | Guy Harris <guy@alum.mit.edu> | 2019-04-06 18:35:00 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2019-04-07 02:23:58 +0000 |
commit | 2c1dcfed4f68fb5b37659fc8f85180504ae2f152 (patch) | |
tree | 93826d8b00b057e71089175f751a388aee292f8e /ui/capture.c | |
parent | 4c8a226c075732f83e77757ccaff31f32d92f9e8 (diff) |
The wtap from which we're reading to get statistics isn't a statistic itself.
Move it to the capture_session structure from the info_data_t structure,
and pass it as an argument to capture_info_new_packets().
Change-Id: I822392bbf48eeb27ba9e17b73775d2fc4349bc17
Reviewed-on: https://code.wireshark.org/review/32765
Petri-Dish: Guy Harris <guy@alum.mit.edu>
Tested-by: Petri Dish Buildbot
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'ui/capture.c')
-rw-r--r-- | ui/capture.c | 17 |
1 files changed, 9 insertions, 8 deletions
diff --git a/ui/capture.c b/ui/capture.c index 002160c262..9c60d29da6 100644 --- a/ui/capture.c +++ b/ui/capture.c @@ -145,6 +145,8 @@ capture_start(capture_options *capture_opts, capture_session *cap_session, info_ capture_callback_invoke(capture_cb_capture_prepared, cap_session); if (capture_opts->show_info) { + cap_session->wtap = NULL; + if (cap_data->counts.counts_hash != NULL) { /* Clean up any previous lists of packet counts */ @@ -155,7 +157,6 @@ capture_start(capture_options *capture_opts, capture_session *cap_session, info_ cap_data->counts.other = 0; cap_data->counts.total = 0; - cap_data->wtap = NULL; cap_data->ui.counts = &cap_data->counts; capture_info_ui_create(&cap_data->ui, cap_session); @@ -447,12 +448,12 @@ capture_input_new_file(capture_session *cap_session, gchar *new_file) } if(capture_opts->show_info) { - if (cap_session->cap_data_info->wtap != NULL) { - wtap_close(cap_session->cap_data_info->wtap); + if (cap_session->wtap != NULL) { + wtap_close(cap_session->wtap); } - cap_session->cap_data_info->wtap = wtap_open_offline(new_file, WTAP_TYPE_AUTO, &err, &err_info, FALSE); - if (!cap_session->cap_data_info->wtap) { + cap_session->wtap = wtap_open_offline(new_file, WTAP_TYPE_AUTO, &err, &err_info, FALSE); + if (!cap_session->wtap) { err_msg = g_strdup_printf(cf_open_error_message(err, err_info, FALSE, WTAP_FILE_TYPE_SUBTYPE_UNKNOWN), new_file); g_warning("capture_input_new_file: %d (%s)", err, err_msg); @@ -508,7 +509,7 @@ capture_input_new_packets(capture_session *cap_session, int to_read) } if(capture_opts->show_info) - capture_info_new_packets(to_read, cap_session->cap_data_info); + capture_info_new_packets(to_read, cap_session->wtap, cap_session->cap_data_info); } @@ -704,8 +705,8 @@ capture_input_closed(capture_session *cap_session, gchar *msg) if(capture_opts->show_info) { capture_info_ui_destroy(&cap_session->cap_data_info->ui); - if(cap_session->cap_data_info->wtap) - wtap_close(cap_session->cap_data_info->wtap); + if(cap_session->wtap) + wtap_close(cap_session->wtap); } cap_session->state = CAPTURE_STOPPED; |