Commit Graph

31 Commits

Author SHA1 Message Date
Gerald Combs 34335a2bc1 Add a null pointer check. Fixes the current Buildbot menagerie failure.
svn path=/trunk/; revision=17797
2006-04-03 14:44:56 +00:00
Anders Broman 510b805e69 From Greg Morris:
Fixes bug 830
Adds a few more expert info details.
 

svn path=/trunk/; revision=17781
2006-04-02 19:37:12 +00:00
Jaap Keuter 87212ed160 Add missing preference storage.
svn path=/trunk/; revision=17639
2006-03-15 21:22:11 +00:00
Gerald Combs fa312091ce Throw an exception if we have an invalid offset. Fixes bug 585.
svn path=/trunk/; revision=16780
2005-12-13 17:00:03 +00:00
Anders Broman 9d646e76dc From Greg Morris:
ncp2222.py.diff
Fixed decoding of NCP 89,10 reply packet. Fixes malformed packet.
 
packet-ncp.c.diff
Adds NCP protocol preference options for enabling/disabling the echoing of data to the expert table.
Expert: EID to Name Lookups
Expert: NCP Connections
Expert: NCP Errors
Expert: Server Information
 
packet-ncp2222.inc.diff
Adds decoding of ZEN attribute values
zendmSearchType
zendmSearchOrder
Fixes blank name stored in EID to Name table if tuned name was returned in resolve name reply packet
Adds capability of reading hf values from NCP tree for exporting specific NCP info to expert table
Adds support for new NCP protocol preference options
Adds expert options for EID to Name Lookups, NCP connections, and Server Information
Change nwconnection from 16bit value to 32bit
 
Packet-ncp-int.h.diff
Fix function prototypes to match new 32bit nwconnection
Add global variables to allow for new NCP protocol preference options

svn path=/trunk/; revision=16575
2005-11-24 05:55:18 +00:00
Guy Harris 3bd8fdbca1 Get rid of an unused variable.
svn path=/trunk/; revision=16357
2005-10-29 21:43:00 +00:00
Guy Harris 3e0e3011e2 Don't run "process_ptvc_record()" on replies unless we're building a
protocol tree.

svn path=/trunk/; revision=16326
2005-10-26 23:10:20 +00:00
Guy Harris de27c8a831 Add a comment.
svn path=/trunk/; revision=16324
2005-10-26 20:21:28 +00:00
Guy Harris 36a2d54f78 Put the code to handle NDS ping replies and NDS replies into routines of
their own.

Do the tapping as early as possible, so it's done even if exceptions are
thrown - and do it regardless of whether the tree argument is null or
not, because a tap might be run without generating protocol trees.

Generate the expert info regardless of whether the tree arugment is null
or not, as that's also used with taps.

svn path=/trunk/; revision=16303
2005-10-25 06:20:46 +00:00
Guy Harris cd7fae828c Add #defines for NDS tag values, and use them in the definition of
nds_tags[] and in switch statements.  (Also, catch a case where we
didn't use NDS_PTYPE_ values.)

Use the existing #defines for MVTYPE_ values.

Make the handling of NDS_PTYPE_{IP,UDP,TCP} in one case match the
handling of the other cases (don't advance "ioffset", as it's advanced
by the length of the item).

When checking whether an NDS reply has a completion code, don't check
the fragment size, check the reassembled packet size.

Fix the handling of referrals in one case.

svn path=/trunk/; revision=16291
2005-10-24 02:49:32 +00:00
Guy Harris 8e35fd8078 Add #defines of NDS protocol types, and use them in the value_string and
switch statements.

Add some sanity checking for ptvcursor_advance calls.

proto_tree_add_item() works fine for IPv4 addresses; use it for them. 
(tvb_get_letohl() doesn't work fine for them, even though it appears to
do so on little-endian machines.)

Only set the "visible" flag on the protocol tree when we're sure we have
one, so that it gets set even if we create a temporary tree.

Catch exceptions when dissecting requests, and compute and save the
results of conditional tests before rethrowing the exception, so the
results get saved even if the dissection gets an exception.

svn path=/trunk/; revision=16288
2005-10-22 06:25:14 +00:00
Guy Harris f225f8819e At least on Tiger, multiple definitions of an external don't work.
svn path=/trunk/; revision=16272
2005-10-19 19:40:55 +00:00
Gilbert Ramirez ab3f0deefc From Greg Morris:
makefile.common.diff - epan directory
1. Adds new packet-ncp-sss.c and packet-ncp-sss.h for new Secret Store dissector
 
New Novell Secret Store Services dissector
packet-ncp-sss.c
packet-ncp-sss.h
 
ncp2222.py.diff
1. Adds a number of return values
2. Adds 64bit file size support
3. Add NCP 89,xx NCP's for UTF8 support
4. Fixes a number of field values for proper dissection
5. Adds support for Secret Store dissector
 
packet-ncp2222.inc.diff
1. Skwelches some compiler warnings
2. Redo of fix for bug 535 which original fix broke dissection of NDS verb 5
3. Adds support for Secret Store dissector
4. Adds expert data
5. Adds tap for service response time
6. Fixes dissection of stream attribute
7. Fixes defragmentation problem with more then 10 fragments
8. Fixes NDS dissection if reply buffer was less then 7
 
packet-ncp.c.diff
1. Adds tap data
2. Adds expert data
3. Fixes calculation for NCP connection number
4. Fixes malformed packet for destroy service connection
 
packet-ncp.c.diff
1. Adds tap data


svn path=/trunk/; revision=16266
2005-10-19 14:28:26 +00:00
Gerald Combs 2544a95b7f Break out of an infinite loop. Fixes bug 535. Remove an unused
function (which had a buffer overflow!)  Initialize a variable flagged
by Valgrind.

svn path=/trunk/; revision=16247
2005-10-16 20:40:27 +00:00
Ulf Lamping 6f43fbb2f0 EVERYTHING IN THE BUILDBOT IS GOING TO BE RED!!! Sorry!
I've done more than a day to change the timestamp resolution from microseconds to nanoseconds. As I really don't want to loose those changes, I'm going to check in the changes I've done so far. Hopefully someone else will give me a helping hand with the things left ...

What's done: I've changed the timestamp resolution from usec to nsec in almost any place in the sources. I've changed parts of the implementation in nstime.s/.h and a lot of places elsewhere.

As I don't understand the editcap source (well, I'm maybe just too tired right now), hopefully someone else might be able to fix this soon.

Doing all those changes, we get native nanosecond timestamp resolution in Ethereal. After fixing all the remaining issues, I'll take a look how to display this in a convenient way...

As I've also changed the wiretap timestamp resolution from usec to nsec we might want to change the wiretap version number...

svn path=/trunk/; revision=15520
2005-08-24 21:31:56 +00:00
Jörg Mayer 69dcb66cd9 More char -> const char warning fixes
svn path=/trunk/; revision=15218
2005-08-05 00:23:22 +00:00
Jörg Mayer eab0aeb4bb More char -> const char fixes
Declare some functions static

svn path=/trunk/; revision=15158
2005-07-30 16:34:38 +00:00
Guy Harris ab797734dd Get rid of the private "my_match_strval()" routine in many dissectors;
add a "match_strval_idx()" routine that does the same thing, and have
"match_strval()" call it.

Make those routines, and "val_to_str()", return a "const" pointer. 
Update dissectors as necessary to squelch compiler warnings produced by
that.

Use "val_to_str()" rather than using "match_strval()" and then, if the
result is null, substituting a specific string.  Clean up some other
"match_strval()"/"val_to_str()" usages.

Add a null pointer check in the NDPS dissector's "attribute_value()"
routine, as it's not clear that "global_attribute_name" won't be null at
that point.

Make some global variables in the AFS4INT dissector local.

Make some routines not used outside the module they're in static.

Make some tables "static const".

Clean up white space.

Fix Gerald's address in some files.

svn path=/trunk/; revision=14786
2005-06-26 19:56:52 +00:00
Gerald Combs 3a6c9ab134 Fix a format string vulneratility in the AFP dissector identified
by iDEFENSE.  Add constant format strings to proto_item_append_text()
in a bunch of other dissectors.  Copy a comment from proto.c to proto.h.

svn path=/trunk/; revision=14713
2005-06-20 19:35:58 +00:00
Gerald Combs b94621701c From Steve Grubb:
Fix bounds checking in lemon.c.  Fix up an if/else block in
packet-dcerpc-samr.c.  Fix hex printing in packet-gtp.c.


Fix other problems discovered by Steve:

Fix a bounds check in packet-isis-lsp.c.  Also, fix netmask printing
on non-little-endian machines.  Add a comment about adding a netmask
routine to to_str.c.  Add bounds checks to packet-ncp2222.inc.


All of these apply to bug 217.

svn path=/trunk/; revision=14627
2005-06-14 18:18:05 +00:00
Ulf Lamping fdb5b1af02 just to get things straight: a dissector should *never* do any g_assert() calls!
in a simple approach, I've replaced all g_assert() and g_assert_not_reached() calls by their exception throwing counterparts DISSECTOR_ASSERT() and DISSECTOR_ASSERT_NOT_REACHED()

this will replace application crash by showing a dissector bug, which is the desired behaviour

there were some g_assert calls in the protocol registering functions, which might not be acting as expected now, but to be able to simply search for g_assert in the future I've replaced that calls too

one g_assert remained, the one when someone throws an unknown exception "into" packet_frame.c, but IMHO this one should remain.

svn path=/trunk/; revision=14608
2005-06-10 18:28:22 +00:00
Ulf Lamping b88fc96be7 fix call to show_fragment_seq_tree() reported by buildbot
svn path=/trunk/; revision=14532
2005-06-02 22:11:22 +00:00
Gerald Combs c7dfef6321 Break out of another large-but-not-infinite loop. Fixes bug 143.
svn path=/trunk/; revision=14229
2005-04-29 14:25:07 +00:00
Gerald Combs e3a76369d1 Break out of a really large (but not infinite) loop.
svn path=/trunk/; revision=14226
2005-04-29 00:16:22 +00:00
Gerald Combs e1a597eccf Add some tvb_ensure_bytes_exist() calls. Fixes bug 101.
svn path=/trunk/; revision=14192
2005-04-26 20:55:26 +00:00
Gerald Combs dfc96b3d23 In get_string(), throw an exception if the string is too long to
process.  Hopefully this doesn't break anything.  Add a bunch of
tvb_ensure_bytes_exist() calls.  Fixes bugs 48 - 51.

svn path=/trunk/; revision=14159
2005-04-21 18:19:38 +00:00
Ronnie Sahlberg 3fbcc50513 From Greg Morris
Fix the breakage from my previous patch


svn path=/trunk/; revision=14022
2005-04-07 08:32:49 +00:00
Ronnie Sahlberg d430320b6a the netware dissector is the only dissector that accesses fvalue_ functions directly.
this made it crash for some packets when the filter speedup patch was enabled, since some of these finfo nodes did no longer exist.

as far as i can tell the only other instances of fvalue_ usage is from some tap based functions but all those do take care to make sure that the fields being accessed are all part of a filter, and will thus never be optimized away.

The patch does two things

1, refactor the code but do not change the logic to reduce the number of places where the external fvalue_ functions are called from.

2, add code to force the tree to be visible (and thus change the code) so that the speedup patch will not optimize these fiuelds away.




svn path=/trunk/; revision=14009
2005-04-04 04:14:41 +00:00
Anders Broman 851ecd2ac4 From Jon Ringle:
1) Added a setup_frame parameter to conversation_t
2) Used the conversation_t next to maintain a list of conversations with the 
same src/dest tuple but different setup_frame number.
3) Changed the signature of find_conversation() and conversation_new() to pass 
in the frame number.
4) Adjusted packet-sdp to select RTP conversation if both m=audio and m=image 
are present, and T.38 conversation if only m=image is present. I expect that 
RTP/T.38 dissecting to be better, but I don't have a way to generate T.38 
packets.

svn path=/trunk/; revision=13243
2005-02-02 20:07:03 +00:00
Guy Harris 5d1de30923 From Greg Morris:
1. Fix Fax Number NDS attribute.  This was causing malformed
	   packet message due to improper decoding. 

	2. Do not try to decode packet beyond connection status when
	   return value is non-zero (error condition).

svn path=/trunk/; revision=11836
2004-08-26 07:46:29 +00:00
Gilbert Ramirez 669db206cb Move dissectors to epan/dissectors directory.
Also move ncp222.py, x11-fields, process-x11-fields.pl,
make-reg-dotc, and make-reg-dotc.py.

Adjust #include lines in files that include packet-*.h
files.

svn path=/trunk/; revision=11410
2004-07-18 18:06:47 +00:00