[bugzilla-daemon@xxxxxxxxxxxxx]

Pascal Quantin changed bug 10898

What	Removed	Added
CC		pascal.quantin@gmail.com

Comment # 2 on bug 10898 from Pascal Quantin

This is packet 3.

The problem comes form the fact that MEGACO dissector calls SDP dissector, and
that SDP dissector assumes that a port will always be a numerical value (which
seems logical at first glance).

So this code found in packet-sdp.c:
        /* XXX Remember Port */
        proto_tree_add_uint(sdp_media_tree, hf_media_port, tvb, offset,
tokenlen,
                            atoi((char*)tvb_get_string_enc(wmem_packet_scope(),
tvb, offset, tokenlen, ENC_UTF_8|ENC_NA)));

Does not handle (legitimately) a $ character. RFC 4566 says:
   <port> is the transport port to which the media stream is sent.  The
      meaning of the transport port depends on the network being used as
      specified in the relevant "c=" field, and on the transport
      protocol defined in the <proto> sub-field of the media field.
      Other ports used by the media application (such as the RTP Control
      Protocol (RTCP) port [19]) MAY be derived algorithmically from the
      base media port or MAY be specified in a separate attribute (for
      example, "a=rtcp:" as defined in [22]).

What does a $ character means in MEGACO case?

---

      You are receiving this mail because:

• You are watching all bug changes.