URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=71f8b97193d3e489f7683dd9265d2c8eff2d433a
Submitter: "Anders Broman <a.broman58@xxxxxxxxx>"
Changed: branch: master-3.2
Repository: wireshark
Commits:
71f8b97 by Orgad Shaneh (orgad.shaneh@xxxxxxxxxxxxxx):
RTP: Fix rtpdump export
Broke by 87973bf516432756f6948e969dbe9d6aa8a27017.
RtpStreamDialog allocates its own rtpstream_info_t, and copies the original stream's info.
Then RtpStreamDialog::on_actionExportAsRtpDump_triggered calls
rtpstream_save with this copy.
On save, reset_tap_listeners is called, and it clears all the tree items,
destroying the allocated copy *before* it is used for the actual export.
Trace:
1 rtpstream_info_free_all tap-rtp-common.c 104
2 RtpStreamTreeWidgetItem::~RtpStreamTreeWidgetItem rtp_stream_dialog.cpp 85
3 RtpStreamTreeWidgetItem::~RtpStreamTreeWidgetItem rtp_stream_dialog.cpp 86
4 QTreeModel::clear()
5 RtpStreamDialog::tapReset rtp_stream_dialog.cpp 309
6 rtpstream_reset_cb tap-rtp-common.c 172
7 reset_tap_listeners tap.c 418
8 cf_retap_packets file.c 2243
9 rtpstream_save rtp_stream.c 97
10 RtpStreamDialog::on_actionExportAsRtpDump_triggered rtp_stream_dialog.cpp 515
Bug: 16351
Change-Id: I54d37a2c97997395936df94ee5481b0d6d198aed
Reviewed-on: https://code.wireshark.org/review/35979
Petri-Dish: Alexis La Goutte <alexis.lagoutte@xxxxxxxxx>
Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
(cherry picked from commit eb0dcc984f49e58bad3312290b1ee965097bc222)
Reviewed-on: https://code.wireshark.org/review/35987
Actions performed:
from 744afc9 Fix indentation.
add 71f8b97 RTP: Fix rtpdump export
Summary of changes:
ui/qt/rtp_stream_dialog.cpp | 9 ++-------
1 file changed, 2 insertions(+), 7 deletions(-)