Fix bug 9835 disabled second media stream disables all media streams
authorHadriel Kaplan <hadrielk@yahoo.com>
Thu, 6 Mar 2014 09:22:31 +0000 (04:22 -0500)
committerAnders Broman <a.broman58@gmail.com>
Fri, 7 Mar 2014 05:03:57 +0000 (05:03 +0000)
commit058527f3f19a09c332b6877030b13dedf7414c1b
tree31a37c6e7fdfa27bc17f8d3159b43cb45ebd33a7
parent4fbcfc1289e6af3e090803b287fa508ece4b3822
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@gmail.com>
Reviewed-by: Evan Huus <eapache@gmail.com>
epan/dissectors/packet-sdp.c
epan/dissectors/packet-sip.c