Commit graph

19 commits

Author SHA1 Message Date
Guy Harris
266b212aad The right length to use when scanning all the data in a packet is the
*reported* length, so that if the packet was cut short by a snapshot
length, we get an exception because we run past the end, and the packet
is marked as having been cut short.

Do not use DISSECTOR_ASSERT() to check for the validity of packet data;
dissectors should try to handle even malformed packets, they shouldn't
report malformed packets as dissector bugs (if DISSECTOR_ASSERT(), that
is *ipso facto* evidence of a dissector bug, *even if the packet is
malformed*).

Update the offset when scanning the packet even if we *aren't* building
the protocol tree.

svn path=/trunk/; revision=25068
2008-04-16 07:18:08 +00:00
Stig Bjørlykke
04eb913d71 Call DISSECTOR_ASSERT instead of g_assert in dissector code.
This fixes bug 2464.

svn path=/trunk/; revision=25039
2008-04-15 06:57:56 +00:00
Jeff Morriss
689e8bd855 From http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1539 :
Hexadecimal and octal are unsigned.  Don't let dissectors register signed
fields (FT_INT*) to be displayed in hexadecimal (including HEX_DEC and DEC_HEX)
or octal.  Fix dissectors that do that mostly by changing the fields to
unsigned though in PANA it appears the fields are meant to be signed so
change those fields to be displayed in decimal.

This fixes an assertion crash in hfinfo_numeric_format() if/when someone tries
to create a filter using one of these mixed signed/unsigned fields (because
that routine does not know how to present the user with a signed value in
hex).

Also add FT_*INT64 to the "make sure it's not BASE_NONE" check.

svn path=/trunk/; revision=24643
2008-03-15 22:41:57 +00:00
Jeff Morriss
bb323055b2 match_strval() may return NULL so don't blindly pass its return value into col_add_*() or proto_add_*(); use val_to_str() (returning "Unknown (%d)" if no match is found) instead.
svn path=/trunk/; revision=24557
2008-03-04 21:05:21 +00:00
Jeff Morriss
eb9620376a Fix bug 1908: strengthen the PANA new-style-dissector heuristics by checking that the length of the first AVP is less than the length of the message.
svn path=/trunk/; revision=23167
2007-10-12 21:31:58 +00:00
Jaap Keuter
a4f2082eb8 The reserved field is now 16 bits. Treat it as such.
svn path=/trunk/; revision=22585
2007-08-22 13:58:39 +00:00
Jaap Keuter
4533ff34a0 From Victor Fajardo:
Updated patches to support draft-ietf-pana-pana-18.txt.

svn path=/trunk/; revision=22583
2007-08-22 06:59:28 +00:00
Jaap Keuter
a9ed13c1d1 Fix reserved bits bitmask.
svn path=/trunk/; revision=22282
2007-07-10 05:15:45 +00:00
Jaap Keuter
0de9abe808 From Victor Fajardo:
Attached is a patch file for the PANA protocol dissector. The changes updates the packet-pana.c dissector to support draft-ietf-pana-pana-15.txt.

svn path=/trunk/; revision=22098
2007-06-14 07:09:07 +00:00
Stephen Fisher
937b925cf5 From Victor Fajardo:
The packet-pana.c dissector I have submitted previously [SVN rev 21447]
had a minor issue. The following are the fixes:
 
Change packet-pana.c, Line 823:

                       { "Ping","pana.flags.e",
 
to:
 
                       { "Error","pana.flags.e",
 

svn path=/trunk/; revision=21731
2007-05-08 19:33:24 +00:00
Jeff Morriss
ee73a2aff2 Don't use DISSECTOR_ASSERT to (essentially) verify packet data. Instead
add an expert info entry if the AVP length is 0 and just return after
dissecting that AVP.

svn path=/trunk/; revision=21465
2007-04-18 07:06:56 +00:00
Jeff Morriss
80c7330166 Replace g_assert() with DISSECTOR_ASSERT()
svn path=/trunk/; revision=21448
2007-04-16 09:03:11 +00:00
Jeff Morriss
aaf6cfb252 From Victor Fajardo: update the PANA dissector to draft-15a
svn path=/trunk/; revision=21447
2007-04-16 08:53:33 +00:00
Sebastien Tandel
68ad2f4b9b fix MacOSX gcc-3.3 warnings about unused tfs/value_string variables
most have been tagged unused (few have been deleted if dissector has not been
modified since a long time)

move packet-ssl-utils.c to DISSECTOR_SRC


svn path=/trunk/; revision=21431
2007-04-14 14:01:53 +00:00
Jeff Morriss
7e2cb04ebf Fix bug 1511:
http://bugs.wireshark.org/bugzilla/show_bug.cgi?id=1511

by replacing g_assert() with DISSECTOR_ASSERT().


svn path=/trunk/; revision=21366
2007-04-10 14:31:09 +00:00
Ronnie Sahlberg
8ce8e719e0 rename some structures and defines from the se_tree to the emem_tree prefix
svn path=/trunk/; revision=18894
2006-08-14 08:29:29 +00:00
Ronnie Sahlberg
b048bbcfe7 add request response tracking and response times
svn path=/trunk/; revision=18738
2006-07-14 23:04:35 +00:00
Ronnie Sahlberg
ce5366cc43 dont dissect reserved bits in a bitmap
svn path=/trunk/; revision=18734
2006-07-14 09:40:10 +00:00
Ronnie Sahlberg
48dab097e5 new protocol PANA from
Peter Racz



svn path=/trunk/; revision=18733
2006-07-14 09:17:11 +00:00