diff options
author | Guy Harris <guy@alum.mit.edu> | 2017-12-08 00:30:55 -0800 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2017-12-08 08:31:41 +0000 |
commit | 8aa14236f8b0ea69901ea9ed87ebabd0d278e973 (patch) | |
tree | b49b5d6b0ee5d647cf33054c5f280d2dc4fa4f99 /ui/qt | |
parent | f29b3deb6d867b16fb8bc4bd5f66a5b6326a6727 (diff) |
Have the frame_tvbuff.c routines not use the global cfile.
Have the routines that create them take a pointer to a struct
packet_provider_data, store that in the tvbuff data, and use it to get
the wtap from which packets are being read.
While we're at it, don't include globals.h in any header files, and
include it in source files iff the source file actually uses cfile. Add
whatever includes that requires.
Change-Id: I9f1ee391f951dc427ff62c80f67aa4877a37c229
Reviewed-on: https://code.wireshark.org/review/24733
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'ui/qt')
-rw-r--r-- | ui/qt/address_editor_frame.cpp | 3 | ||||
-rw-r--r-- | ui/qt/export_pdu_dialog.cpp | 1 | ||||
-rw-r--r-- | ui/qt/gsm_map_summary_dialog.cpp | 1 | ||||
-rw-r--r-- | ui/qt/iax2_analysis_dialog.cpp | 3 | ||||
-rw-r--r-- | ui/qt/lte_rlc_graph_dialog.cpp | 2 | ||||
-rw-r--r-- | ui/qt/models/packet_list_record.cpp | 4 | ||||
-rw-r--r-- | ui/qt/mtp3_summary_dialog.cpp | 1 | ||||
-rw-r--r-- | ui/qt/packet_dialog.cpp | 2 | ||||
-rw-r--r-- | ui/qt/packet_list.cpp | 4 | ||||
-rw-r--r-- | ui/qt/rtp_analysis_dialog.cpp | 3 | ||||
-rw-r--r-- | ui/qt/utils/frame_information.cpp | 2 |
11 files changed, 14 insertions, 12 deletions
diff --git a/ui/qt/address_editor_frame.cpp b/ui/qt/address_editor_frame.cpp index a56464b76e..45f4ad0ec2 100644 --- a/ui/qt/address_editor_frame.cpp +++ b/ui/qt/address_editor_frame.cpp @@ -83,7 +83,8 @@ void AddressEditorFrame::editAddresses(CaptureFile &cf, int column) col_custom_prime_edt(&edt, &cap_file_->cinfo); epan_dissect_run(&edt, cap_file_->cd_t, &cap_file_->phdr, - frame_tvbuff_new_buffer(cap_file_->current_frame, &cap_file_->buf), cap_file_->current_frame, &cap_file_->cinfo); + frame_tvbuff_new_buffer(&cap_file_->provider, cap_file_->current_frame, &cap_file_->buf), + cap_file_->current_frame, &cap_file_->cinfo); epan_dissect_fill_in_columns(&edt, TRUE, TRUE); /* First check selected column */ diff --git a/ui/qt/export_pdu_dialog.cpp b/ui/qt/export_pdu_dialog.cpp index 7316c680a7..98edce311f 100644 --- a/ui/qt/export_pdu_dialog.cpp +++ b/ui/qt/export_pdu_dialog.cpp @@ -25,7 +25,6 @@ #include "export_pdu_dialog.h" #include <ui_export_pdu_dialog.h> -#include "globals.h" #include <wiretap/pcap-encap.h> #include <epan/tap.h> diff --git a/ui/qt/gsm_map_summary_dialog.cpp b/ui/qt/gsm_map_summary_dialog.cpp index e32ee545f4..a7a00d62a1 100644 --- a/ui/qt/gsm_map_summary_dialog.cpp +++ b/ui/qt/gsm_map_summary_dialog.cpp @@ -30,7 +30,6 @@ #include <glib.h> -#include "globals.h" #include "summary.h" #include <epan/packet.h> diff --git a/ui/qt/iax2_analysis_dialog.cpp b/ui/qt/iax2_analysis_dialog.cpp index 92707f8cd3..2958ad95ab 100644 --- a/ui/qt/iax2_analysis_dialog.cpp +++ b/ui/qt/iax2_analysis_dialog.cpp @@ -335,7 +335,8 @@ Iax2AnalysisDialog::Iax2AnalysisDialog(QWidget &parent, CaptureFile &cf) : epan_dissect_init(&edt, cap_file_.capFile()->epan, TRUE, FALSE); epan_dissect_prime_with_dfilter(&edt, sfcode); epan_dissect_run(&edt, cap_file_.capFile()->cd_t, &cap_file_.capFile()->phdr, - frame_tvbuff_new_buffer(fdata, &cap_file_.capFile()->buf), fdata, NULL); + frame_tvbuff_new_buffer(&cap_file_.capFile()->provider, fdata, &cap_file_.capFile()->buf), + fdata, NULL); // This shouldn't happen (the menu item should be disabled) but check anyway if (!dfilter_apply_edt(sfcode, &edt)) { diff --git a/ui/qt/lte_rlc_graph_dialog.cpp b/ui/qt/lte_rlc_graph_dialog.cpp index 71478dddfb..922a9bcb37 100644 --- a/ui/qt/lte_rlc_graph_dialog.cpp +++ b/ui/qt/lte_rlc_graph_dialog.cpp @@ -40,8 +40,6 @@ #include "wireshark_application.h" #include "simple_dialog.h" -#include "globals.h" - #include <epan/dissectors/packet-rlc-lte.h> #include <ui/tap-rlc-graph.h> diff --git a/ui/qt/models/packet_list_record.cpp b/ui/qt/models/packet_list_record.cpp index 8e1551071b..176e6d5998 100644 --- a/ui/qt/models/packet_list_record.cpp +++ b/ui/qt/models/packet_list_record.cpp @@ -185,7 +185,9 @@ void PacketListRecord::dissect(capture_file *cap_file, bool dissect_color) * XXX - need to catch an OutOfMemoryError exception and * attempt to recover from it. */ - epan_dissect_run(&edt, cap_file->cd_t, &phdr, frame_tvbuff_new_buffer(fdata_, &buf), fdata_, cinfo); + epan_dissect_run(&edt, cap_file->cd_t, &phdr, + frame_tvbuff_new_buffer(&cap_file->provider, fdata_, &buf), + fdata_, cinfo); if (dissect_columns) { /* "Stringify" non frame_data vals */ diff --git a/ui/qt/mtp3_summary_dialog.cpp b/ui/qt/mtp3_summary_dialog.cpp index b97294f78e..9b30cbd533 100644 --- a/ui/qt/mtp3_summary_dialog.cpp +++ b/ui/qt/mtp3_summary_dialog.cpp @@ -30,7 +30,6 @@ #include <glib.h> -#include "globals.h" #include "summary.h" #include <epan/tap.h> diff --git a/ui/qt/packet_dialog.cpp b/ui/qt/packet_dialog.cpp index ea17a20cd4..bf3e481b58 100644 --- a/ui/qt/packet_dialog.cpp +++ b/ui/qt/packet_dialog.cpp @@ -63,7 +63,7 @@ PacketDialog::PacketDialog(QWidget &parent, CaptureFile &cf, frame_data *fdata) col_custom_prime_edt(&edt_, &(cap_file_.capFile()->cinfo)); epan_dissect_run(&edt_, cap_file_.capFile()->cd_t, &phdr_, - frame_tvbuff_new(fdata, packet_data_), + frame_tvbuff_new(&cap_file_.capFile()->provider, fdata, packet_data_), fdata, &(cap_file_.capFile()->cinfo)); epan_dissect_fill_in_columns(&edt_, TRUE, TRUE); diff --git a/ui/qt/packet_list.cpp b/ui/qt/packet_list.cpp index 7163a10a20..a7966b7f33 100644 --- a/ui/qt/packet_list.cpp +++ b/ui/qt/packet_list.cpp @@ -1024,7 +1024,9 @@ QString PacketList::getFilterFromRowAndColumn() epan_dissect_init(&edt, cap_file_->epan, have_custom_cols(&cap_file_->cinfo), FALSE); col_custom_prime_edt(&edt, &cap_file_->cinfo); - epan_dissect_run(&edt, cap_file_->cd_t, &cap_file_->phdr, frame_tvbuff_new_buffer(fdata, &cap_file_->buf), fdata, &cap_file_->cinfo); + epan_dissect_run(&edt, cap_file_->cd_t, &cap_file_->phdr, + frame_tvbuff_new_buffer(&cap_file_->provider, fdata, &cap_file_->buf), + fdata, &cap_file_->cinfo); epan_dissect_fill_in_columns(&edt, TRUE, TRUE); if ((cap_file_->cinfo.columns[ctx_column_].col_custom_occurrence) || diff --git a/ui/qt/rtp_analysis_dialog.cpp b/ui/qt/rtp_analysis_dialog.cpp index 145fad1f75..bdc8c44179 100644 --- a/ui/qt/rtp_analysis_dialog.cpp +++ b/ui/qt/rtp_analysis_dialog.cpp @@ -1607,7 +1607,8 @@ void RtpAnalysisDialog::findStreams() epan_dissect_prime_with_dfilter(&edt, sfcode); epan_dissect_prime_with_hfid(&edt, hfid_rtp_ssrc); epan_dissect_run(&edt, cap_file_.capFile()->cd_t, &cap_file_.capFile()->phdr, - frame_tvbuff_new_buffer(fdata, &cap_file_.capFile()->buf), fdata, NULL); + frame_tvbuff_new_buffer(&cap_file_.capFile()->provider, fdata, &cap_file_.capFile()->buf), + fdata, NULL); /* * Packet must be an RTPv2 packet with an SSRC; we use the filter to diff --git a/ui/qt/utils/frame_information.cpp b/ui/qt/utils/frame_information.cpp index c81b6736c2..5f2aa8277e 100644 --- a/ui/qt/utils/frame_information.cpp +++ b/ui/qt/utils/frame_information.cpp @@ -61,7 +61,7 @@ void FrameInformation::loadFrameTree() col_custom_prime_edt(&edt_, &(cap_file_->capFile()->cinfo)); epan_dissect_run(&edt_, cap_file_->capFile()->cd_t, &phdr_, - frame_tvbuff_new(fi_, packet_data_), + frame_tvbuff_new(&cap_file_->capFile()->provider, fi_, packet_data_), fi_, &(cap_file_->capFile()->cinfo)); epan_dissect_fill_in_columns(&edt_, TRUE, TRUE); } |