Wireshark-commits: [Wireshark-commits] master 74247f9: Fix RTP player crash on invalid streams
Date Index · Thread Index · Other Months · All Mailing Lists
Date Prev · Date Next · Thread Prev · Thread Next
From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 22 Nov 2015 11:25:14 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=74247f96a998797e933f09f566f60406b9ba92b4
Submitter: Peter Wu (peter@xxxxxxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

74247f9 by Peter Wu (peter@xxxxxxxxxxxxx):

    Fix RTP player crash on invalid streams
    
    On Linux with pulseaudio, the RTP player can crash when an invalid RTP
    stream is played. Prevent that by detecting when stream playback fails.
    
    Since the stateChanged signal receiver is registered on the same
    thread, it is guaranteed that any outputStateChanged calls happen before
    returning from audio_output_->start().
    
    GTK+ not have this issue, its player simply does not show the decoded
    stream at all.
    
    Change-Id: I51a91a7f410ef3d46551bc8df0049542efbb806f
    Reviewed-on: https://code.wireshark.org/review/11997
    Petri-Dish: Peter Wu <peter@xxxxxxxxxxxxx>
    Tested-by: Petri Dish Buildbot <buildbot-no-reply@xxxxxxxxxxxxx>
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  26366ef   SCSI: Fix mixed up SCSI senddiag PF values
    adds  74247f9   Fix RTP player crash on invalid streams


Summary of changes:
 ui/qt/rtp_audio_stream.cpp |    5 +++++
 1 file changed, 5 insertions(+)