diff options
| author | Jörg Mayer <jmayer@loplof.de> | 2009-07-08 15:29:37 +0000 |
|---|---|---|
| committer | Jörg Mayer <jmayer@loplof.de> | 2009-07-08 15:29:37 +0000 |
| commit | c43391cd088ce7ac0ac9240704cf29b80ea70c41 (patch) | |
| tree | b78d75c5a58589d3a71fe3bc062fee359c33ae2a /epan/dissectors/packet-ip.c | |
| parent | 8daca54c04dcb86fa4ff4e3791dce4eef7503b12 (diff) | |
Currently, the expert code notifies if a packet is addressed
to 224.0.0.x with a TTL>1. Some protocols (notably VRRP and GLBP)
send out a TTL of 255 and thus cause a notice. That should be fixed,
so for now: Add a FIXME about that.
svn path=/trunk/; revision=29024
Diffstat (limited to 'epan/dissectors/packet-ip.c')
| -rw-r--r-- | epan/dissectors/packet-ip.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/epan/dissectors/packet-ip.c b/epan/dissectors/packet-ip.c index 0ebfe0a739..edfd8cae6a 100644 --- a/epan/dissectors/packet-ip.c +++ b/epan/dissectors/packet-ip.c @@ -1611,6 +1611,13 @@ dissect_ip(tvbuff_t *tvb, packet_info *pinfo, proto_tree *parent_tree) * * Flag a low TTL if the packet is not destined for a multicast address * (e.g. 224.0.0.0/4). + * + * FIXME: Add an exception list, as Some protocols seem to insist on + * doing differently: + * - IETF's VRRP (rfc3768) always uses 224.0.0.18 with 255 + * - Cisco's GLPB always uses 224.0.0.102 with 255 + * Even more, VRRP and GLBP should probably be flagged as an error, if + * seen with any TTL except 255. */ if (is_a_local_network_control_block_addr(dst32)) { if (iph->ip_ttl != 1) { |