aboutsummaryrefslogtreecommitdiffstats
path: root/epan/dissectors/packet-pppoe.c
AgeCommit message (Collapse)AuthorFilesLines
2007-10-23Apply the small performance enhancment patches for:Anders Broman1-2/+2
- if offset is 0, tvb_length is the same as tvb_length_remaining, just faster. Replace - col_append_fstr() with faster col_append_str() - col_add_str() with col_set_str() when it's safe svn path=/trunk/; revision=23252
2007-09-02tvb_reported_length() returns a gint, so assign its value to a gint.Guy Harris1-1/+2
svn path=/trunk/; revision=22773
2007-09-02The actual length of the PPPoE payload is returned byGuy Harris1-9/+27
tvb_reported_length_remaining(), not by tvb_length_remaining() - tvb_length_remaining() shows only the amount of *captured* data remaining, but the capture might have been done with a snapshot length that cut the packet data short. The payload length from the PPPoE header could legitimately be different from the actual length of the PPPoE payload if there's not enough PPPoE payload to avoid padding at the E(thernet) level. Only complain if there shouldn't have been any padding. Report an "expert" warning if the payload length looks wrong. Update a comment to reflect current reality (as of many many years ago, when we went all-tvbuff). svn path=/trunk/; revision=22770
2007-08-15- s/ntohs/g_ntohsJeff Morriss1-5/+5
- s/ntohl/g_ntohl - s/free/g_free - Change some tvb_get_string()+g_free()'s into tvb_get_ephemeral_string() - Change some tvb_fake_unicode()+g_free()'s into tvb_get_ephemeral_faked_unicode() - Change some tvb_get_string() calls that were clearly memory leaks (like atoi(tvb_get_string(...))) into tvb_get_ephemeral_string() svn path=/trunk/; revision=22515
2006-12-22Report inaccurate PPPoE Session payload length when detected to aid in theStephen Fisher1-12/+16
troubleshooting of malformed packets. svn path=/trunk/; revision=20203
2006-05-21change a whole bunch of ethereal into wiresharkRonnie Sahlberg1-2/+2
svn path=/trunk/; revision=18196
2005-12-07From Martin M various updates and fixesRonnie Sahlberg1-2/+5
Hi, Some tiddly changes: pppoe- don't create an empty discovery tags tree when the payload length is 0 chap- make chap.value work as a filterable field rtcp- append the packet type to the protocol tree name  svn path=/trunk/; revision=16712
2005-11-28Don't show the tag twice.Guy Harris1-2/+5
svn path=/trunk/; revision=16612
2005-11-28Host-Uniq is binary, not text.Guy Harris1-3/+3
FT_BYTES items don't have a base - the bytes are always dumped in hex - so use BASE_NONE with them. svn path=/trunk/; revision=16611
2005-11-25From Martin Mathieson:Anders Broman1-140/+321
Here is a patch to the PPPoED and PPPoES dissectors. - All fields are now filterable - General prettification - I've added a preference to PPoED to set whether or not tag values and data lengths should be shown in the tree (default is OFF) svn path=/trunk/; revision=16582
2005-09-28Have show_exception() take a "const char *" as its exception message,Guy Harris1-0/+1
and not free the string to which it points. Pass to REPORT_DISSECTOR_BUG() strings allocated with ep_strdup_printf(), so that they're freed automatically. svn path=/trunk/; revision=16039
2005-09-17Move a pile of protocol-related headers from the top-level sourceGuy Harris1-1/+1
directory to the epan directory. Some of them should perhaps ultimately be moved to epan/dissectors, if they pertain only to stuff exported by a particular dissector. Fix Gerald's e-mail address in files we're moving. svn path=/trunk/; revision=15844
2005-06-26Get rid of the private "my_match_strval()" routine in many dissectors;Guy Harris1-2/+2
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-10just to get things straight: a dissector should *never* do any g_assert() calls!Ulf Lamping1-2/+2
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
2004-07-18Move dissectors to epan/dissectors directory.Gilbert Ramirez1-0/+309
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
generated by cgit v1.2.3 (git 2.25.1) at 2024-05-22 02:59:39 +0000