Wireshark · Wireshark-commits: [Wireshark-commits] master 058527f: Fix bug 9835 disabled second media stream disables all media str

[Wireshark code review <code-review-do-not-reply@xxxxxxxxxxxxx>]

URL: https://code.wireshark.org/review/gitweb?p=wireshark.git;a=commit;h=058527f3f19a09c332b6877030b13dedf7414c1b
Submitter: Anders Broman (a.broman58@xxxxxxxxx)
Changed: branch: master
Repository: wireshark

Commits:

058527f by Hadriel Kaplan (hadrielk@xxxxxxxxx):

    Fix bug 9835 disabled second media stream disables all media streams
    
    When a single media line is rejected in an SDP answer, for example a second
    'm=video' line, wireshark disables ALL media sessions, instead of just that
    one.  But per the RFCs, all it should do is disable just the one RTP media
    session the m= line represents. This commit fixes that, so that a disabled
    media session (one with a m= port of 0) in the SDP answer only disables its
    associated/paired media stream in the offer.
    
    Change-Id: I9bd0d3fc88b8eaa55207c9bf3f3e37da7746fd14
    Reviewed-on: https://code.wireshark.org/review/526
    Reviewed-by: Anders Broman <a.broman58@xxxxxxxxx>
    Reviewed-by: Evan Huus <eapache@xxxxxxxxx>
    

Actions performed:

    from  4fbcfc1   (Benign) Don't use "address-of" operator on arg of VALS() (not that it makes any actual difference)..
    adds  058527f   Fix bug 9835 disabled second media stream disables all media streams


Summary of changes:
 epan/dissectors/packet-sdp.c |   40 +++++++++++++++++++++++++---------------
 epan/dissectors/packet-sip.c |    2 ++
 2 files changed, 27 insertions(+), 15 deletions(-)