https://bugs.wireshark.org/bugzilla/show_bug.cgi?id=5208

Peter Wu <pe...@lekensteyn.nl> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |pe...@lekensteyn.nl

--- Comment #18 from Peter Wu <pe...@lekensteyn.nl> ---
I'm currently having a look at the SDP dissector which seems to assume that
port zero means "no media". But that is only true for responses in the
Offer/Answer model. For RTSP the value is merely an optional recommendation
which can be set to zero (see comment 4).

And from the SDP Offer/Answer model RFC we find where the above "ignore media
with port zero" comes from (https://tools.ietf.org/html/rfc3264#section-5.1):

   [..] sendrecv is the default. [..]
   For recvonly and sendrecv streams, the port number and address in the
   offer indicate where the offerer would like to receive the media
   stream. [..]
   A port number of zero in the offer indicates that the
   stream is offered but MUST NOT be used.  This has no useful semantics
   in an initial offer, but is allowed for reasons of completeness,
   since the answer can contain a zero port indicating a rejected stream
   (Section 6).  Furthermore, existing streams can be terminated by
   setting the port to zero (Section 8).  In general, a port number of
   zero indicates that the media stream is not wanted.

I'll first try to refactor the SDP dissector to reduce duplication in
setup_sdp_transport and dissect_sdp. Maybe the SDP dissector should provide a
means to export the parsed SDP to the parent dissector (SDP, RTSP, etc.). RTSP
would then mainly be interested in the media and should probably not register
any recommended ports since the client may ignore them (instead, rely on the
DESCRIBE response).

-- 
You are receiving this mail because:
You are watching all bug changes.
___________________________________________________________________________
Sent via:    Wireshark-bugs mailing list <wireshark-bugs@wireshark.org>
Archives:    https://www.wireshark.org/lists/wireshark-bugs
Unsubscribe: https://www.wireshark.org/mailman/options/wireshark-bugs
             mailto:wireshark-bugs-requ...@wireshark.org?subject=unsubscribe

Reply via email to