From 0ec5a271ea96e92d80060bded7245749a71e199d Mon Sep 17 00:00:00 2001 From: Gerald Combs Date: Mon, 6 Jun 2016 14:55:10 -0700 Subject: Qt: RTP audio stream fixups. Make sure audio_stream_ is non-NULL before we try to use it. Delete audio_stream_ more gracefully and add a note about mutexes on OS X and Windows. Bug: 12166 Change-Id: I12e76c49e631bc1de813c5c7d82c7d928c71237e Reviewed-on: https://code.wireshark.org/review/15759 Petri-Dish: Gerald Combs Tested-by: Petri Dish Buildbot Reviewed-by: Gerald Combs --- ui/qt/rtp_player_dialog.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'ui/qt/rtp_player_dialog.cpp') diff --git a/ui/qt/rtp_player_dialog.cpp b/ui/qt/rtp_player_dialog.cpp index 3dc9184312..161132973b 100644 --- a/ui/qt/rtp_player_dialog.cpp +++ b/ui/qt/rtp_player_dialog.cpp @@ -61,7 +61,7 @@ Q_DECLARE_METATYPE(QCPGraph *) // + dst port + ssrc. This means that we can have multiple rtp_stream_info // structs per RtpAudioStream. Should we make them 1:1 instead? -// Current RTP player bugs: +// Current and former RTP player bugs. Many have attachments that can be usef for testing. // Bug 3368 - The timestamp line in a RTP or RTCP packet display's "Not Representable" // Bug 3952 - VoIP Call RTP Player: audio played is corrupted when RFC2833 packets are present // Bug 4960 - RTP Player: Audio and visual feedback get rapidly out of sync @@ -72,6 +72,7 @@ Q_DECLARE_METATYPE(QCPGraph *) // Bug 10613 - RTP audio player crashes // Bug 11125 - RTP Player does not show progress in selected stream in Window 7 // Bug 11409 - Wireshark crashes when using RTP player +// Bug 12166 - RTP audio player crashes // XXX It looks like we duplicate some functionality here and in the RTP // analysis code, which has its own routines for writing audio data to a -- cgit v1.2.1