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(-)