Commit Graph

64 Commits

Author SHA1 Message Date
Guy Harris 490cadfcd0 In "nds_defrag()", handle "request_value->ncp_rec" being null.
In "dissect_nds_request()", insert the request information into the hash
table the first time we see the packet, regardless of whether we created
a new conversation or not.

svn path=/trunk/; revision=8541
2003-09-25 11:27:24 +00:00
Guy Harris e0ad895b14 Create a protocol tree if we don't have one, we're constructing the Info
column, and we need stuff from the protocol tree for the Info column.

Go back to the previous scheme for constructing the Info column; the
previous change fixes the problems for which the Info column changes
were fixes.

svn path=/trunk/; revision=8526
2003-09-24 03:34:00 +00:00
Guy Harris 88b1bc3fac From Greg Morris: get the request data before using it, if we don't
already have it.

svn path=/trunk/; revision=8523
2003-09-23 22:04:02 +00:00
Guy Harris 67c64a0c52 From Greg Morris: properly initialize request_value->info_string using
"strcpy()", not "strcat()".

svn path=/trunk/; revision=8518
2003-09-23 18:22:19 +00:00
Guy Harris d4a5694c99 From Greg Morris: update the Info column even if we're not building a
protocol tree.

svn path=/trunk/; revision=8502
2003-09-20 04:25:11 +00:00
Gilbert Ramirez e2d7335584 Fix signed/unsigned comparison warning.
svn path=/trunk/; revision=8300
2003-08-28 17:26:15 +00:00
Guy Harris 4901c45334 From Greg Morris:
1. Added support for defragmentation of NDS packets

	2. Moved the run_info_string logic outside of the
	   pinfo->fd->flags->visited because of the dissection of info
	   strings is not processed on subsequent decodes then the
	   find/string/info will not locate info string values.  By moving
	   this outside then when the find logic runs a tvb_dissect_new
	   then the info string gets processed again and all data is
	   visible.  Otherwise only info column data that is processed by
	   the NCP dissector is visible not any data from ncp2222.py
	   processed by the info string function.

svn path=/trunk/; revision=8257
2003-08-25 22:12:04 +00:00
Gilbert Ramirez c91b8e713f Fix signed/unsigned comparison warnings.
svn path=/trunk/; revision=8084
2003-07-25 04:11:51 +00:00
Guy Harris 34460af1ee Use "val_to_str()" to translate NDS verb numbers to strings.
svn path=/trunk/; revision=8060
2003-07-21 09:21:53 +00:00
Guy Harris 3b39429a6f From Greg Morris:
1. Some NCP's that displayed the file/directory path in the
	   summary window would display blank lines on Windows based
	   machines.  This was due to unicode or non-displayable
	   characters contained in the character string being added to
	   the column data.  I made a change to format/strip out
	   non-displayable characters prior to adding to the column
	   data.

	2. Moved ncp_req_hash_value struct to packet-ncp-int.h so that
	   I can use it within a future dissector (NMAS).

Clean up white space.

svn path=/trunk/; revision=7560
2003-04-25 20:36:29 +00:00
Guy Harris de205c4a3a Clean up the handling of the completion code.
svn path=/trunk/; revision=7424
2003-04-09 08:36:54 +00:00
Guy Harris b010dc03a3 From Greg Morris: update the introductory comment, fix the handling
of the completion code, and tweak the description of continuation
fragments.

svn path=/trunk/; revision=7416
2003-04-08 00:46:15 +00:00
Guy Harris 7005595a7f From Greg Morris: don't delete the EID hash table after the first pass,
as it's needed whenever a packet is dissected.

svn path=/trunk/; revision=7200
2003-02-25 21:28:04 +00:00
Guy Harris d902039842 Updates from Greg Morris:
1. Added a number of NCP Error return values.
	2. Added ability to pass byte strings to summary window.
	3. Modified file NCP's to reflect file handle in summary window.
	4. Changed all NCP's using QueueID from big endian to little endian.
	This is different from Novell's NCP documentation but matches same
	information obtained from NDS dissector.
	5. Added over 50 more NDS error return values.
	6. Partially fixed NDS Request verb 6 (Search Entries)

svn path=/trunk/; revision=7172
2003-02-19 21:47:46 +00:00
Guy Harris 086e99dff9 The hour and minute fields of times go from 0 to 23 and 0 to 59, so
adding 1 to them is incorrect (and cannot possibly be correct, as that'd
rule out 0 as valid values, meaning nothing can ever happen in the first
hour after midnight or in the first minute after the hour).

svn path=/trunk/; revision=7099
2003-02-08 03:06:06 +00:00
Ronnie Sahlberg b8fd3cc391 Keep track of and display the time between Call and Reply in NCP packets.
svn path=/trunk/; revision=6968
2003-01-22 00:45:30 +00:00
Guy Harris 5344a43575 Fix up the display of the "ncp.nds_time_filter" field.
svn path=/trunk/; revision=6852
2003-01-06 02:13:01 +00:00
Guy Harris 59932f2722 Don't cast away constness, and fix variable and structure member
qualifiers as necessary to ensure that we don't have to.

"strcmp()", "strcasecmp()", and "memcmp()" don't return booleans; don't
test their results as if they did.

Use "guint8", not "guchar", for a pointer to (one or more) 8-bit bytes.

Update Michael Tuexen's e-mail address.

svn path=/trunk/; revision=6726
2002-12-02 23:43:30 +00:00
Guy Harris d97d0803b3 Put in a note about the completion code not appearing to be present in
replies to continuation messages (or, at least, not present in the
obvious place).

svn path=/trunk/; revision=6470
2002-10-22 08:24:39 +00:00
Guy Harris c9900e6390 From Greg Morris:
Various NDS dissector enhancements including NDS verb 0x07, and
	proper ping flags for NDS ping version 10 request.

	Updated error codes for several NCP reply packets.

	NDPS enhancements.

svn path=/trunk/; revision=6463
2002-10-21 18:56:52 +00:00
Guy Harris 62e7954c94 Get rid of all places where "vstring" is set to point to a null string
constant, as that won't work with "get_string()" (it'll try to overwrite
the constant, and either crash, trash memory, or both).

svn path=/trunk/; revision=6460
2002-10-19 20:28:54 +00:00
Guy Harris b619b0410b From Greg Morris: put a buffer into "nw_uni_t", and make "get_string()"
fill that in.

svn path=/trunk/; revision=6456
2002-10-19 01:05:44 +00:00
Guy Harris 97222e340b Clean up white space.
Clean up the stuff to handle MVTYPE_PROC_ENTRY_SPECIFIERS a bit, and put
in comments explaining what's going on for that and MVTYPE_ATTR_REQUEST.

svn path=/trunk/; revision=6439
2002-10-16 23:24:21 +00:00
Guy Harris f27452006f Use #defines for multivalue types.
svn path=/trunk/; revision=6438
2002-10-16 23:04:53 +00:00
Guy Harris 84b245903c Make some counters 32-bit, as the upper limit on the loop in which
they're used is 32-bit.

Use "%u" rather than "%d" to print unsigned values.

svn path=/trunk/; revision=6437
2002-10-16 22:44:58 +00:00
Guy Harris 11b8825bcb Use #defines for vtype values, rather than numbers.
svn path=/trunk/; revision=6436
2002-10-16 22:20:30 +00:00
Guy Harris cd747aaeba Make arrays of "nds_val" structures rather than declaring a bunch of
individual structures, and initialize them, and put them into
the protocol tree, in a loop.

svn path=/trunk/; revision=6435
2002-10-16 21:57:24 +00:00
Guy Harris ddf3c6140b Fully initialize an ncp_req_hash_value after allocating it.
svn path=/trunk/; revision=6431
2002-10-15 17:31:20 +00:00
Guy Harris 01bd6435e4 We'll be building a protocol tree for all NDS Ping packets, so we can
just wrap the entire dissection in an "if", as was done before.
(Actually, we can just skip the "if" entirely, I guess....)

svn path=/trunk/; revision=6429
2002-10-15 07:47:57 +00:00
Guy Harris 35eba08c92 Set the "req_nds_flags" field for an NDS Ping request regardless of
whether we're building a protocol tree or not.

Set the separator if we put "Revision" into the summary.

Check for a remaining length >= 8, not > 8, to check for an NDS version
10 or later ping, and check the reported length, not the captued length.

svn path=/trunk/; revision=6428
2002-10-15 07:36:16 +00:00
Guy Harris 6af349418a Create the temporary tree if we're *not* building a protocol tree
already, not if we *are*.

svn path=/trunk/; revision=6414
2002-10-14 16:47:56 +00:00
Guy Harris e4b32038a7 Make the flag buffer in "dissect_ping_req()" large enough to hold the
longest possible flag string.

Don't create a temporary protocol tree unless we're not filling in a
protocol tree.

The NDS version isn't present in NDS pings prior to version 10; dissect
it only if it's present.

svn path=/trunk/; revision=6412
2002-10-11 21:52:11 +00:00
Guy Harris b348f66dd4 Note that "dissect_ping_req()" duplicates a bunch of stuff in
"dissect_ncp_request()".

Fix "dissect_ping_req()" to properly put the NDS version in a ping
request into the tree (although that field isn't present in the ping
requests I've seen, so perhaps it's only in NDS version 10 and later),
as big-endian (change FALSE to TRUE if it's little-endian).

Get rid of the unused "ncp_nds_verb" variable (set, but not used) in
"dissect_ncp_common()".

Call "dissect_ping_req()" from "dissect_ncp_common()" again.

svn path=/trunk/; revision=6411
2002-10-11 19:36:13 +00:00
Guy Harris 4f471acc34 Fix some spelling errors.
svn path=/trunk/; revision=6410
2002-10-10 23:52:51 +00:00
Guy Harris 15c6ff4f2f Rename "ncp.nds_string" to "ncp.nds_tree_name", as it's not a generic
string, it's a tree name (as the field's display name suggests).

Get rid of the unused "ncp.nds_string_value" field.

Fix up the names of some fields, and use "proto_tree_add_uint()" rather
than "proto_tree_add_uint_format()" to add them to the protocol tree.

svn path=/trunk/; revision=6408
2002-10-10 23:32:01 +00:00
Guy Harris ce4a4b3149 Fix up the handling of bitfields in NDS messages - get rid of extra
blanks, and don't put the list of flag values in if no bits are set.

svn path=/trunk/; revision=6407
2002-10-10 23:06:00 +00:00
Guy Harris ccab7497b2 "proto_tree_add_string_format()" requires that the string value be
passed as an argument before the format string; fix some calls to do so.

svn path=/trunk/; revision=6405
2002-10-10 21:24:59 +00:00
Guy Harris d6377805f5 When deciding whether to put "NDS" into the protocol field in the packet
list, check whether the protocol field is enabled, not whether the info
field is enabled.  Update the protocol field before trying to look at
the packet, so it gets updated even for short packets.

Only parse service reply packets as NDS packets; don't parse positive
ack packets that way.

svn path=/trunk/; revision=6404
2002-10-10 21:08:45 +00:00
Guy Harris b3af4098d2 Don't attach the string for an NDS error to the completion code - attach
the string for the completion code.  Do put the string for the NDS error
into the summary line, however.

Don't clear the NDS error code if we can't translate it to a string.

svn path=/trunk/; revision=6400
2002-10-10 03:58:47 +00:00
Guy Harris 9a4b466497 "#if 0" out the code in "get_string()", as it will probably cause
crashes on many platforms, and other random behavior on others.

Don't create a temporary tree with an NCP tree underneath it if we
already have an NCP tree; we don't want the results of a dissection to
be stuck in an invisible temporary tree if we're supposed to be
generating a protocol tree.

svn path=/trunk/; revision=6399
2002-10-10 03:25:47 +00:00
Guy Harris 9934126755 From cjs 2895: get rid of C++ comments, so IBM's Visual Age C compiler
can compile the code.

Note that Bill Fumerola rewrote the Cisco NetFlow dissector.

Update a bunch of addresses in the Ethereal man page, and put some
missing addresses in.

svn path=/trunk/; revision=6380
2002-10-08 19:19:00 +00:00
Guy Harris 488f1216ad Make some variables unsigned to remove some compiler warnings.
svn path=/trunk/; revision=6377
2002-10-08 08:58:55 +00:00
Jörg Mayer 011a6aafde Warning fixes for the ncp code by Greg Morris and J�rg Mayer.
svn path=/trunk/; revision=6338
2002-09-26 02:08:21 +00:00
Jörg Mayer 8c3a0d068f Greg Morris:
This update adds NCP 87/66.
Also add additional error codes, added hf_definition for super class
(NDS). Cleaned up some unused variables (Warnings when building
Ethereal). Fixed NCP type 0x9999 (server busy packets to echo
information to summary information), Added subtree format to NDS
dissector to cleanup display.

svn path=/trunk/; revision=6331
2002-09-25 00:37:01 +00:00
Gerald Combs a4cace5463 Fix some (but not all) compiler warnings.
svn path=/trunk/; revision=6317
2002-09-22 17:39:00 +00:00
Gerald Combs c9cdfff342 From Greg Morris: More NDS decodes.
Fix a typo in get_string().

svn path=/trunk/; revision=6313
2002-09-22 15:46:42 +00:00
Guy Harris 0aeae83ddc NDS fragment handles are 32 bits, according to Greg Morris.
svn path=/trunk/; revision=6084
2002-08-25 21:41:12 +00:00
Guy Harris ab89806eab Give the NDS Verb field a value_string table, so you can filter on it
without having to know the numerical values for the verbs.

Use that table to convert the verb value to a verb name.

Fix indentation.

svn path=/trunk/; revision=6075
2002-08-23 22:44:57 +00:00
Guy Harris 149f19d372 Note to the person who checked in the previous change: that's "already
have", not "don't have". :-)

svn path=/trunk/; revision=6074
2002-08-23 22:15:51 +00:00
Guy Harris 5357337904 In "dissect_nds_request()", if we already have a tree into which to put
items, don't create the temporary tree.

svn path=/trunk/; revision=6073
2002-08-23 22:08:50 +00:00