Ethereal-users: Re: [Ethereal-users] MGCP Decoding

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

From: Guy Harris <gharris@xxxxxxxxxxxx>
Date: Mon, 22 Jan 2001 00:55:48 -0800
> If it's truly not desired that an MGCP packet containing SDP data be
> listed as "MGCP/SDP" in the Protocol column, the MGCP dissector should
> set the column to "MGCP" before fetching *anything* from the packet, so
> that if it goes past the end of the packet the packet will still get
> correctly marked as MGCP, and should also do so after calling the SDP
> dissector.

I've checked in a change to the MGCP dissector to do exactly that.

If it is OK for an MGCP packet containing SDP data to be listed as
"MGCP/SDP" in the Protocol column - other protocols work that way, e.g. 
RTSP - then, as per my previous mail, the code to set the Protocol
column should be removed and the code to set the Info column should be
moved *before* the call to the SDP dissector, as the SDP dissector
appends to both of those columns, rather than overwriting them.