Wireshark-commits: [Wireshark-commits] master-3.2 67c7761: RTCP: contain BYE packet dissection to i

From: Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>
Date: Sun, 22 Mar 2020 21:37:03 +0000
URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=67c7761c1fe2fa67d39a5727b698707c05e021b9
Submitter: "Peter Wu <peter@xxxxxxxxxxxxx>"
Changed: branch: master-3.2
Repository: wireshark

Commits:

67c7761 by Jaap Keuter (jaap.keuter@xxxxxxxxx):

    RTCP: contain BYE packet dissection to its stated length
    
    The Goodbye RTCP packet dissection has to derive whether the optional
    "Reason for leaving" string is present or not. This has to be derived
    from the length. When put into a compound RTCP packet, the length
    derivation from the TVB length does not work, because another RTCP
    packet may follow in this compound RTCP packet.
    
    With this change the stated length of this RTCP packet is passed to
    the RTCP BYE packet dissection function in order to make a proper
    length determination and not overrun into the next RTCP packet in the
    compound RTCP packet.
    
    Bug: 16434
    Change-Id: Iab0fdd52c745028a9928bbef6c731ff649213277
    Reviewed-on: https://code.wireshark.org/review/36532
    Petri-Dish: Anders Broman <a.broman58@xxxxxxxxx>
    Tested-by: Petri Dish Buildbot
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    (cherry picked from commit f998e785d5ead49c57ef282765519d4ae75e056e)
    Reviewed-on: https://code.wireshark.org/review/36535
    Reviewed-by: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Petri-Dish: Jaap Keuter <jaap.keuter@xxxxxxxxx>
    Reviewed-by: Peter Wu <peter@xxxxxxxxxxxxx>
    

Actions performed:

    from  acb73c7   [Automatic update for 2020-03-22]
     add  67c7761   RTCP: contain BYE packet dissection to its stated length


Summary of changes:
 epan/dissectors/packet-rtcp.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)