diff options
| author | gram <gram@f5534014-38df-0310-8fa8-9805f1628bb7> | 2001-01-03 16:41:08 +0000 |
|---|---|---|
| committer | gram <gram@f5534014-38df-0310-8fa8-9805f1628bb7> | 2001-01-03 16:41:08 +0000 |
| commit | 7d286958f19942e52e7912f7eaebb1c8cbed804d (patch) | |
| tree | e67e6a292cf66edf79431f30778f239fc9468f4e /packet-hsrp.c | |
| parent | 0442a789c10c208756b38971f752eaf44ff37e03 (diff) | |
Ensure that all value_string arrays end in {0, NULL}. Dissectors got away
with not terminating their arrays because they knew the limits of the
value used to look up strings in the value_string array, but the
dfilter_expr_dlg does not know these limits and must rely on the terminating
{0, NULL} record.
Also, in SNA fixed a bug in which a field should have been defined as FT_UINT8
but was defined as FT_BOOLEAN.
In WTP, fixed a value string which had duplicate keys.
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2817 f5534014-38df-0310-8fa8-9805f1628bb7
Diffstat (limited to 'packet-hsrp.c')
| -rw-r--r-- | packet-hsrp.c | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/packet-hsrp.c b/packet-hsrp.c index 4980566960..ad18bb48ad 100644 --- a/packet-hsrp.c +++ b/packet-hsrp.c @@ -4,7 +4,7 @@ * * Heikki Vatiainen <hessu@cs.tut.fi> * - * $Id: packet-hsrp.c,v 1.13 2001/01/03 06:55:28 guy Exp $ + * $Id: packet-hsrp.c,v 1.14 2001/01/03 16:41:06 gram Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@zing.org> @@ -82,7 +82,8 @@ struct hsrp_packet { /* Multicast to 224.0.0.2, TTL 1, UDP, port 1985 * static const value_string hsrp_opcode_vals[] = { {HSRP_OPCODE_HELLO, "Hello"}, {HSRP_OPCODE_COUP, "Coup"}, - {HSRP_OPCODE_RESIGN, "Resign"} + {HSRP_OPCODE_RESIGN, "Resign"}, + {0, NULL}, }; #define HSRP_STATE_INITIAL 0 @@ -97,7 +98,8 @@ static const value_string hsrp_state_vals[] = { {HSRP_STATE_LISTEN, "Listen"}, {HSRP_STATE_SPEAK, "Speak"}, {HSRP_STATE_STANDBY, "Standby"}, - {HSRP_STATE_ACTIVE, "Active"} + {HSRP_STATE_ACTIVE, "Active"}, + {0, NULL}, }; static void |