Wireshark-commits: [Wireshark-commits] master-2.0 f6898f2: Don't use the non-existent decoded data
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: Mon, 26 Oct 2015 10:14:11 +0000 (UTC)
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=f6898f200396cc2fea45b476924375547a129835
Submitter: Guy Harris (guy@xxxxxxxxxxxx)
Changed: branch: master-2.0
Repository: wireshark

Commits:

f6898f2 by Guy Harris (guy@xxxxxxxxxxxx):

    Don't use the non-existent decoded data if decode_rtp_packet() returns 0.
    
    Just skip that packet.  Otherwise, it crashes.
    
    Fix file name in the introductory comment while we're at it.
    
    Change-Id: I286f4303a4ec152c0d00c5135395c1608bf2121a
    Reviewed-on: https://code.wireshark.org/review/11279
    Reviewed-by: Guy Harris <guy@xxxxxxxxxxxx>
    (cherry picked from commit 4ce9655658d9546ba7459aa80ff9a6be1381708e)
    Reviewed-on: https://code.wireshark.org/review/11280
    

Actions performed:

    from  b315d45   Allocate h245_labels when we initialize the h245dg tap.
    adds  f6898f2   Don't use the non-existent decoded data if decode_rtp_packet() returns 0.


Summary of changes:
 ui/qt/rtp_audio_stream.cpp |    7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)