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-snmp.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-snmp.c')
| -rw-r--r-- | packet-snmp.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/packet-snmp.c b/packet-snmp.c index 5e6c9d5aa4..9ec3ba38e1 100644 --- a/packet-snmp.c +++ b/packet-snmp.c @@ -2,7 +2,7 @@ * Routines for SNMP (simple network management protocol) * D.Jorand (c) 1998 * - * $Id: packet-snmp.c,v 1.55 2001/01/03 06:55:33 guy Exp $ + * $Id: packet-snmp.c,v 1.56 2001/01/03 16:41:07 gram Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@zing.org> @@ -324,7 +324,8 @@ static const value_string smux_prio[] = { static const value_string smux_sout[] = { { SMUX_SOUT_COMMIT, "Commit" }, - { SMUX_SOUT_ROLLBACK, "Rollback" } + { SMUX_SOUT_ROLLBACK, "Rollback" }, + { 0, NULL } }; /* Error status values */ |