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-rx.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-rx.c')
| -rw-r--r-- | packet-rx.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/packet-rx.c b/packet-rx.c index 544ad2cd33..92c79339ac 100644 --- a/packet-rx.c +++ b/packet-rx.c @@ -4,7 +4,7 @@ * Based on routines from tcpdump patches by * Ken Hornstein <kenh@cmf.nrl.navy.mil> * - * $Id: packet-rx.c,v 1.16 2001/01/03 06:55:31 guy Exp $ + * $Id: packet-rx.c,v 1.17 2001/01/03 16:41:07 gram Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@zing.org> @@ -71,7 +71,8 @@ static const value_string rx_flags[] = { { RX_REQUEST_ACK, "req-ack" }, { RX_LAST_PACKET, "last-pckt" }, { RX_MORE_PACKETS, "more-pckts" }, - { RX_FREE_PACKET, "free-pckt" } + { RX_FREE_PACKET, "free-pckt" }, + { 0, NULL } }; static int proto_rx = -1; |