aboutsummaryrefslogtreecommitdiffstats
path: root/xdlc.c
Commit message (Collapse)AuthorAgeFilesLines
* Back out the previous change - the problem wasn't what I thought it was.Guy Harris2004-03-041-7/+8
| | | | | | | The problem is that two arguments to an "sprintf()" call were switched; un-switch them. svn path=/trunk/; revision=10296
* Don't show "func=" if the P/F bit isn't set.Guy Harris2004-03-031-7/+7
| | | | | | Don't put an extra space between "func=" and "P" or "F" if it is set. svn path=/trunk/; revision=10286
* From Jan Kiszka: compress the summary line display.Guy Harris2004-01-201-8/+9
| | | | svn path=/trunk/; revision=9758
* Add arguments to "dissect_xdlc_control()" to specify value_stringGuy Harris2004-01-181-4/+10
| | | | | | | | | | | | | | | | | | tables, for use in the top-level item for the control field, for unnumbered frame function codes for commands and responses. If the argument is null, default to the standard tables. Use "dissect_xdlc_control()" and the #defines from "xdlc.h" for IrDA. Use the reported length rather than the data length in the IrDA dissector when iterating over the data in an IrDA packet. Make "dissect_xid()" update the column information as well as the protocol tree. Put the slot number into the protocol tree only for command frames - "the slot number field is undefined in discovery XID response frames", to quote the IrLAP spec. svn path=/trunk/; revision=9713
* Have "dissect_xdlc_control()" take a pointer to a structure containingGuy Harris2004-01-031-176/+126
| | | | | | | | | | pointers to hf_ values, so the subfields of the control field are put into the protocol tree as filterable items. Change the protocols that use it appropriately. Export "dissect_xdlc_control()" to plugins. svn path=/trunk/; revision=9531
* Use "dissect_xdlc_control()" to dissect the control field of Frame RelayGuy Harris2003-09-021-8/+23
| | | | | | | | | | | packets that have one. Add an argument to "dissect_xdlc_control()" to indicate whether it should append the information to the Info field or just put it in the Info field. Use the #defines for the DLCI bitfields to extract the DLCI bits when constructing the DLCI. svn path=/trunk/; revision=8335
* Removed trailing whitespaces from .h and .c files using theJörg Mayer2002-08-281-5/+5
| | | | | | | winapi_cleanup tool written by Patrik Stridvall for the wine project. svn path=/trunk/; revision=6117
* Replace the types from sys/types.h and netinet/in.h by their glib.hJörg Mayer2002-08-021-6/+2
| | | | | | | | equivalents for the toplevel directory. The removal of winsock2.h will hopefully not cause any problems under MSVC++, as those files using struct timeval still include wtap.h, which still includes winsock2.h. svn path=/trunk/; revision=5932
* From Joerg Mayer: get rid of unused function arguments.Guy Harris2002-03-311-2/+2
| | | | svn path=/trunk/; revision=5053
* Include files from the "epan" directory and subdirectories thereof withGuy Harris2002-01-211-2/+2
| | | | | | | | | | | | | "epan/..." pathnames, so as to avoid collisions with header files in any of the directories in which we look (e.g., "proto.h", as some other package has its own "proto.h" file which it installs in the top-level include directory). Don't add "-I" flags to search "epan", as that's no longer necessary (and we want includes of "epan" headers to fail if the "epan/" is left out, so that we don't re-introduce includes lacking "epan/"). svn path=/trunk/; revision=4586
* Move the pointer to the "column_info" structure in the "frame_data"Guy Harris2001-12-101-9/+8
| | | | | | | | | | | | | | | structure to the "packet_info" structure; only stuff that's permanently stored with each frame should be in the "frame_data" structure, and the "column_info" structure is not guaranteed to hold the column values for that frame at all times - it was only in the "frame_data" structure so that it could be passed to dissectors, and, as all dissectors are now passed a pointer to a "packet_info" structure, it could just as well be put in the "packet_info" structure. That saves memory, by shrinking the "frame_data" structure (there's one of those per frame), and also lets us clean up the code a bit. svn path=/trunk/; revision=4370
* Convert LAPD and V.120 dissector to use tvbuff. Convert xdlc dissector-helper,Gilbert Ramirez2000-05-311-37/+37
| | | | | | too. svn path=/trunk/; revision=2030
* Add tvbuff class.Gilbert Ramirez2000-05-111-24/+24
| | | | | | | | | | | | | | | Add exceptions routines. Convert proto_tree_add_*() routines to require tvbuff_t* argument. Convert all dissectors to pass NULL argument ("NullTVB" macro == NULL) as the tvbuff_t* argument to proto_tree_add_*() routines. dissect_packet() creates a tvbuff_t, wraps the next dissect call in a TRY block, will print "Short Frame" on the proto_tree if a BoundsError exception is caught. The FDDI dissector is converted to use tvbuff's. svn path=/trunk/; revision=1939
* Fix some errors discovered by making GCC do format string/argumentGuy Harris2000-03-141-5/+5
| | | | | | | | | | | | | | | cross-checking, and by replacing "proto_tree_add_item_format()" by multiple routines to add items of various types. Make the arguments of "proto_tree_add_bytes_format()" and "proto_tree_add_string_format()" that specify the bytes or the string be "const" pointers, so that one can pass a "const" pointer without complaints from the compiler. Squelch a (bogus, but the compiler isn't in a position to know that) complaint about an uninitialized variable. svn path=/trunk/; revision=1716
* Break proto_tree_add_item_format() into multiple functions:Gilbert Ramirez2000-03-121-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | proto_tree_add_protocol_format() proto_tree_add_uint_format() proto_tree_add_ipxnet_format() proto_tree_add_ipv4_format() proto_tree_add_ipv6_format() proto_tree_add_bytes_format() proto_tree_add_string_format() proto_tree_add_ether_format() proto_tree_add_time_format() proto_tree_add_double_format() proto_tree_add_boolean_format() If using GCC 2.x, we can check the print-format against the variable args passed in. Regardless of compiler, we can now check at run-time that the field type passed into the function corresponds to what that function expects (FT_UINT, FT_BOOLEAN, etc.) Note that proto_tree_add_protocol_format() does not require a value field, since the value of a protocol is always NULL. It's more intuitive w/o the vestigial argument. Fixed a proto_tree_add_item_format-related bug in packet-isis-hello.c Fixed a variable usage bug in packet-v120.c. (ett_* was used instead of hf_*) Checked in Guy's fix for the function declearation for proto_tree_add_text() and proto_tree_add_notext(). svn path=/trunk/; revision=1713
* Fix Gerald's e-mail address.Guy Harris2000-01-071-2/+2
| | | | svn path=/trunk/; revision=1437
* Replace the ETT_ "enum" members, declared in "packet.h", withGuy Harris1999-11-161-6/+6
| | | | | | | | | | | | | | | | | dynamically-assigned "ett_" integer values, assigned by "proto_register_subtree_array()"; this: obviates the need to update "packet.h" whenever you add a new subtree type - you only have to add a call to "proto_register_subtree_array()" to a "register" routine and an array of pointers to "ett_", if they're not already there, and add a pointer to the new "ett_" variable to the array, if they are there; would allow run-time-loaded dissectors to allocate subtree types when they're loaded. svn path=/trunk/; revision=1043
* Have "get_xdlc_control()" and "dissect_xdlc_control()" return theGuy Harris1999-09-261-54/+13
| | | | | | | | | | | xDLC control field, so that its caller can not only determine from it whether the frame has a payload, but can also determine how long the control field is. Put macros in "xdlc.h" to determine both of those. Have "capture_llc()" and "dissect_llc()" use that information appropriately. svn path=/trunk/; revision=727
* Make Information Frame honor is_extended variable when reading Control.Gilbert Ramirez1999-09-261-2/+5
| | | | svn path=/trunk/; revision=725
* No, 0xCC is SNRME. (I *told* you I hated reversing bit strings....)Guy Harris1999-08-271-2/+1
| | | | svn path=/trunk/; revision=588
* TEST is 0xE0, not 0xC0; 0xC0 is SNRME. (I hate having to reverse bitGuy Harris1999-08-271-2/+4
| | | | | | strings....) Thanks to Tom Gallagher at Madge for pointing this out. svn path=/trunk/; revision=587
* Have "get_xdlc_control()" and "dissect_xdlc_control()" just return aGuy Harris1999-08-231-15/+44
| | | | | | Boolean indicating whether the frame has any payload to dissect or not. svn path=/trunk/; revision=556
* The only LLC frame types that should be dissected based on their SAP or,Guy Harris1999-08-231-27/+49
| | | | | | | | if the SAPs are SNAP, based on their ethertype are I frames and UI frames; others don't have payload to be dissected as belonging to other protocols. svn path=/trunk/; revision=555
* Get rid of the comments saying "make ["decode_numeric_bitfield()"] shiftGuy Harris1999-08-161-7/+1
| | | | | | the value appropriately!" - a change from Olivier Abad did so. svn path=/trunk/; revision=501
* Patch from Olivier Abad to make it correctly display non-extendedGuy Harris1999-08-051-3/+5
| | | | | | informatino frames. svn path=/trunk/; revision=443
* Add a "dissect_xdlc_control()" routine, to dissect the control field ofGuy Harris1999-08-041-0/+402
SDLC-derived protocols such as HDLC and derivatives of it such as LAPB, IEEE 802.2 LLC, and so on. Have the LLC and LAPB dissectors use it. Make "dissect_numeric_bitfield()" put the low-order bit of the bitfield in the low-order bit of an integer when printing it, so that the right value is printed. svn path=/trunk/; revision=434