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-srvloc.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-srvloc.c')
| -rw-r--r-- | packet-srvloc.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/packet-srvloc.c b/packet-srvloc.c index fcfdd4b974..e61b47c44c 100644 --- a/packet-srvloc.c +++ b/packet-srvloc.c @@ -6,7 +6,7 @@ * In particular I have not had an opportunity to see how it * responds to SRVLOC over TCP. * - * $Id: packet-srvloc.c,v 1.17 2001/01/03 06:55:33 guy Exp $ + * $Id: packet-srvloc.c,v 1.18 2001/01/03 16:41:07 gram Exp $ * * Ethereal - Network traffic analyzer * By Gerald Combs <gerald@zing.org> @@ -107,6 +107,7 @@ static const value_string srvloc_functions[] = { { DAADVERT, "DA Advertisement" }, { SRVTYPERQST, "Service Type Request" }, { SRVTYPERPLY, "Service Type Reply" }, + { 0, NULL } }; /* List to resolve flag values to names */ @@ -142,6 +143,7 @@ static const value_string srvloc_errs[] = { { CHRSET_NOT_UND, "Character set not understood" }, { AUTH_ABSENT, "Authentication absent" }, { AUTH_FAILED, "Authentication failed" }, + { 0, NULL } }; static int |