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-ncp.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-ncp.c')
| -rw-r--r-- | packet-ncp.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/packet-ncp.c b/packet-ncp.c index b1f8b910e5..e204ea35c0 100644 --- a/packet-ncp.c +++ b/packet-ncp.c @@ -3,7 +3,7 @@ * Gilbert Ramirez <gram@xiexie.org> * Modified to allow NCP over TCP/IP decodes by James Coe <jammer@cin.net> * - * $Id: packet-ncp.c,v 1.44 2001/01/03 07:53:43 guy Exp $ + * $Id: packet-ncp.c,v 1.45 2001/01/03 16:41:06 gram Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@zing.org> @@ -86,6 +86,7 @@ struct ncp_ip_rqhdr { static const value_string ncp_ip_signature[] = { { NCPIP_RQST, "Demand Transport (Request)" }, { NCPIP_RPLY, "Transport is NCP (Reply)" }, + { 0, NULL }, }; /* The information in this module comes from: |