Ethereal-cvs: [Ethereal-cvs] cvs commit: ethereal packet-rtsp.c packet-sdp.c

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Guy Harris <guy@xxxxxxxxxxxxxxxxxxx>
Date: Sun, 12 Nov 2000 15:23:54 -0600 (CST)
guy         2000/11/12 15:23:53 CST

  Modified files:
    .                    packet-rtsp.c packet-sdp.c 
  Log:
  As RFC 2327 says, "SDP is purely a format for session description - it
  does not incorporate a transport protocol, and is intended to use
  different transport protocols as appropriate including the Session
  Announcement Protocol [4], Session Initiation Protocol [11], Real- Time
  Streaming Protocol [12], electronic mail using the MIME extensions, and
  the Hypertext Transport Protocol."
  
  As such, it shouldn't set the protocol column to SDP, as that means
  the protocol column won't indicate what the transport protocol was;
  instead, it should append "/SDP" to the protocol column - RTSP was,
  after calling "dissect_sdp()", setting the protocol column to
  "RTSP/SDP", and this change means that all protocols using SDP will have
  the protocol column set in that fashion, and that the RTSP dissector
  doesn't have to explicitly set the protocol column to cause that to
  happen.
  
  Revision  Changes    Path
  1.24      +1 -3      ethereal/packet-rtsp.c
  1.13      +2 -2      ethereal/packet-sdp.c