aboutsummaryrefslogtreecommitdiffstats
path: root/packet-bgp.c
AgeCommit message (Collapse)AuthorFilesLines
2002-11-04From Hannes Gredler:guy1-9/+103
support dissecting of v6 VPN NLRIs; fix a little bug for displaying v4 nexthops using type 1 RD style [dissecting started at wrong offset]. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6561 f5534014-38df-0310-8fa8-9805f1628bb7
2002-10-15From Hannes Gredler: dissect the graceful restart capability in BGP OPENguy1-3/+59
messages. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6430 f5534014-38df-0310-8fa8-9805f1628bb7
2002-10-15Fix more typing issues. Pull the marker, length, and type individuallygerald1-33/+37
instead of copying them into a struct. Remove the "bgp" struct, as it is no longer used. Gracefuly (more than before, at least) handle a zero packet length. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6421 f5534014-38df-0310-8fa8-9805f1628bb7
2002-10-15Fix signed/unsigned type mismatches, as pointed out by Silvio.gerald1-22/+20
Add BGP output to randpkt. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6420 f5534014-38df-0310-8fa8-9805f1628bb7
2002-10-10Properly check the return value of decode_prefix4(). This prevents angerald1-1/+15
infinite loop caused by a malformed packet. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6392 f5534014-38df-0310-8fa8-9805f1628bb7
2002-08-28Removed trailing whitespaces from .h and .c files using thejmayer1-34/+34
winapi_cleanup tool written by Patrik Stridvall for the wine project. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6117 f5534014-38df-0310-8fa8-9805f1628bb7
2002-08-27From Hannes Gredler:guy1-26/+43
open up a subtree when decoding l2info extd communities; some bugfixes in the extd community dissector. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6098 f5534014-38df-0310-8fa8-9805f1628bb7
2002-08-27From Hannes Gredler:guy1-47/+129
more complete support for L2VPNs as described in draft-kompella-ppvpn-l2vpn; fix a segfault in the extd_community decoder; more consistent SAFI strings (tcpdump); more robust V6 decoding (the assumption that v6 may come only in unlabeled form is wrong :-|) git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6093 f5534014-38df-0310-8fa8-9805f1628bb7
2002-08-24Properly check for an address family that's neither AFNUM_INET norguy1-3/+13
AFNUM_INET6. Properly check for "mp_addr_to_str()" indicating that it doesn't know the address family being parsed. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6082 f5534014-38df-0310-8fa8-9805f1628bb7
2002-08-24From Hannes Gredler:guy1-168/+207
Basic support for AFI & Extd communities in draft-kompella-ppvpn-l2vpn More robust handling for unknown AFIs in BGP MP(UN)REACH NLRIs Fix typos. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6079 f5534014-38df-0310-8fa8-9805f1628bb7
2002-08-15From Jian Yu:guy1-11/+128
a small fix for displaying unrecognized capabilities; ORF support (draft-ietf-idr-route-filter-06.txt). git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@5998 f5534014-38df-0310-8fa8-9805f1628bb7
2002-08-03We don't use anything from <arpa/inet.h>, so we don't need to includeguy1-5/+1
it. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@5938 f5534014-38df-0310-8fa8-9805f1628bb7
2002-08-02Replace the types from sys/types.h and netinet/in.h by their glib.hjmayer1-17/+9
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. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@5932 f5534014-38df-0310-8fa8-9805f1628bb7
2002-05-21Fixes to BGP problems, from Dirk Steinberg.guy1-3/+15
Add some additional bounds checking to "decode_MPLS_stack()" so as not to overflow the buffer handed to it. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@5518 f5534014-38df-0310-8fa8-9805f1628bb7
2002-05-21From Hannes Gredler: fix misuse of "&&" when "&" was intended.guy1-3/+3
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@5517 f5534014-38df-0310-8fa8-9805f1628bb7
2002-05-20From Hannes Gredler: fix to label stack handling for withdrawn routes.guy1-1/+9
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@5511 f5534014-38df-0310-8fa8-9805f1628bb7
2002-05-15From Hannes Gredler: labeled unicast support for BGP.guy1-1/+26
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@5481 f5534014-38df-0310-8fa8-9805f1628bb7
2002-02-14From Laurent Rabret: print the correct length in the Extendedguy1-5/+4
Communities attribute in a BGP Update message. Also, get rid of an extra space before a colon in the display for that attribute, which isn't in other attributes. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4732 f5534014-38df-0310-8fa8-9805f1628bb7
2002-01-30Just use "val_to_str()" to generate names for communities whose valuesguy1-21/+16
are in the reserved region (which also fixes a bug where we weren't printing the value for the NO_ADVERTISE community correctly). git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4638 f5534014-38df-0310-8fa8-9805f1628bb7
2002-01-21Include files from the "epan" directory and subdirectories thereof withguy1-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/"). git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4586 f5534014-38df-0310-8fa8-9805f1628bb7
2002-01-17Do desegmentation.guy1-33/+41
Collapse the two loops through the TCP segment into one. Use "tvb_reported_length()", not "tvb_length()", to find out how big the TCP segment is ("tvb_length()" says how much of it was captured, not how much of it there is). git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4557 f5534014-38df-0310-8fa8-9805f1628bb7
2002-01-15Correctly handle BGP attributes with lists of entries when the attributeguy1-36/+39
has the "extended length" flag set - the starting offset of the list depends on whether the "extended length" bit was set or not. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4544 f5534014-38df-0310-8fa8-9805f1628bb7
2001-12-10Move the pointer to the "column_info" structure in the "frame_data"guy1-8/+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. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4370 f5534014-38df-0310-8fa8-9805f1628bb7
2001-12-03Make "dissector_add()", "dissector_delete()", and "dissector_change()"guy1-2/+5
take a dissector handle as an argument, rather than a pointer to a dissector function and a protocol ID. Associate dissector handles with dissector table entries. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4308 f5534014-38df-0310-8fa8-9805f1628bb7
2001-11-03Cooperative Route Filtering Capability support, and Route Refreshguy1-4/+76
Message bug fix, in BGP, from Motonori Shindo. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4142 f5534014-38df-0310-8fa8-9805f1628bb7
2001-09-13Support for dissecting multiple capabilities, from Endoh Akira.guy1-116/+115
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3930 f5534014-38df-0310-8fa8-9805f1628bb7
2001-07-21There is really no need to have the BGP dissector and the LDP dissectorguy1-26/+6
have two independent "value_string" tables mapping RFC 1700 address family numbers to names, nor is there any need to have the BGP dissector and the PIM dissector have two independent sets of #defines for RFC 1700 address family numbers; put a single "value_string" table in "afn.c" and put a declaration of it, and #defines for the address family numbers, into "afn.h", and have the dissectors use that. Move the #define for PGM into "ipproto.h", and add an entry for it in the "value_string" table in "ipproto.c". Have the PGM dissector use the standard Ethereal mechanisms for resolving addresses, and have it use "value_string" tables for mapping option types, the OPX bits, and packet types to strings. Use "bytes_to_str()" to turn byte arrays into strings of hex digits. Pass the packet type string to "dissect_pgmopts()" as an argument, rather than making it a global. Don't use "proto_tree_add_XXX_format" routines if you can possibly just use "proto_tree_add_XXX"; give various fields the correct radix and type, and VALS() strings if necessary, to make that happen (and to make filtering on them more pleasant). Put the type, length, and total length of the options into the protocol tree as separate fields. Don't have separate type, length, and OPX fields for every type of option; one field will suffice. Don't format a string with "sprintf()" and then pass that string to "col_add_fstr()" with a format of "%s" and the string as an argument - "col_add_fstr()" can format strings itself (that's what the "f" stands for). Don't byte-swap and then un-byte-swap IPv4 address fields in the header, just leave them network byte order to start with. Use the correct fields for "proto_tree_add_XXX", rather than using the same field multiple times. Quit early if an address family identifier isn't AFNUM_INET, as that means the structure we use to dissect the header doesn't match the actual header. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3761 f5534014-38df-0310-8fa8-9805f1628bb7
2001-07-09Get rid of the unused zero-length "bgpext_osptf_rtype_metric" array -guy1-3/+1
not all compilers let you define an array with no explicit length and no initializers. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3670 f5534014-38df-0310-8fa8-9805f1628bb7
2001-07-08Support for draft-rosen-vpn-ospf-bgp-mpls and for additional BGPguy1-22/+60
extended communities, from Aamer Akhter. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3667 f5534014-38df-0310-8fa8-9805f1628bb7
2001-07-03And explain why the fact that RFC 2858 says you can't do what RFC 2545guy1-2/+8
suggests, and that RFC 2858 obsoletes RFC 2283 which says you can, doesn't matter - Ethereal's job isn't to enforce protocol standards or to refuse to dissect stuff that doesn't conform to the final version of standards; if it can dissect stuff that's now illegal but that wasn't illegal in the past, and do so without causing problems when dissecting currently legal stuff, it should so so, so that if you have captures that include now-illegal stuff (perhaps from old devices that haven't been upgraded, or from old captures), you can still see what was happening. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3644 f5534014-38df-0310-8fa8-9805f1628bb7
2001-07-03Explain why we dissect more than one address in the MP_REACH_NLRIguy1-1/+7
attribute. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3643 f5534014-38df-0310-8fa8-9805f1628bb7
2001-06-18From Joerg Mayer: explicitly fill in all members of aguy1-2/+2
"header_field_info" structure, including the ones that are later set by the routines to register fields. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3561 f5534014-38df-0310-8fa8-9805f1628bb7
2001-06-15There can be more than one next hop network address in theguy1-9/+19
BGPTYPE_MP_REACH_NLRI stuff; dissect all of them. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3553 f5534014-38df-0310-8fa8-9805f1628bb7
2001-06-15If "buf" is a "char *", "sizeof(buf)" evaluates to the size of aguy1-2/+2
"char *" variable, not to the size of the buffer to which it points. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3552 f5534014-38df-0310-8fa8-9805f1628bb7
2001-06-10MP-BGP message support, from Thierry Stagiaire.guy1-241/+460
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3534 f5534014-38df-0310-8fa8-9805f1628bb7
2001-05-30Get rid of some unnecessary settings of "pinfo->current_proto" - if theguy1-2/+1
dissector is called only through a handle or dissector table, the code that handles those calls sets it for you. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3480 f5534014-38df-0310-8fa8-9805f1628bb7
2001-05-16Support for Cisco-proprietary capabilities in BGP, fromguy1-1/+5
<bgp4news@yahoo.com>. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3416 f5534014-38df-0310-8fa8-9805f1628bb7
2001-04-23Get rid of some unnecessary includes.guy1-2/+2
Fix up Gerald's e-mail address. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3371 f5534014-38df-0310-8fa8-9805f1628bb7
2001-04-17Tvbuffified by Heikki Vatiainen.guy1-327/+315
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3317 f5534014-38df-0310-8fa8-9805f1628bb7
2001-01-22Remove more "CHECK_DISPLAY_AS_DATA()" calls and "pinfo->current_proto ="guy1-3/+3
statements. Move the setting of the Protocol column in various dissectors before anything is fetched from the packet, and also clear the Info column at that point in those and some other dissectors, so that if an exception is thrown, the columns don't reflect the previous protocol. Make the IP dissector static, as it's called only via dissector tables or dissector handles. Also make the "dissect the TOS field as the DiffServ DS field" flag static, as it's not referred to outside of "packet-ip.c". In the NCP dissector, refer to the port type through "pinfo" rather than through the global "pi", as it's a tvbuffified dissector. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2929 f5534014-38df-0310-8fa8-9805f1628bb7
2001-01-09Add an additional "protocol index" argument to "{old_}dissector_add()",guy1-2/+2
"{old_}heur_dissector_add()", "{old_}conv_dissector_add()", and "register_dissector()", so that an entry in those tables has associated with it the protocol index of the protocol the dissector handles (or -1, if there is no protocol index for it). This is for future use in a number of places. (Arguably, "proto_register_protocol()" should take a dissector pointer as an argument, but 1) it'd have to handle both regular and heuristic dissectors; 2) making it take either a "dissector_t" or a union of that and a "heur_dissector_t" introduces some painful header-file interdependencies so I'm punting on that for now. As with other Ethereal internal APIs, these APIs are subject to change in the future, at least until Ethereal 1.0 comes out....) git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2849 f5534014-38df-0310-8fa8-9805f1628bb7
2001-01-03Have "proto_register_protocol()" build a list of data structures forguy1-2/+3
protocols, in addition to adding structures to the list of filterable fields. Give it an extra argument that specifies a "short name" for the protocol, for use in such places as pinfo->current_proto; the dialog box for constructing filters; the preferences tab for the protocol; and so on (although we're not yet using it in all those places). Make the preference name that appears in the preferences file and the command line for the DIAMETER protocol "diameter", not "Diameter"; the convention is that the name in question be all-lower-case. Make some routines and variables that aren't exported static. Update a comment in the ICP dissector to make it clear that the dissector won't see fragments other than the first fragment of a fragmented datagram. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2810 f5534014-38df-0310-8fa8-9805f1628bb7
2000-12-28>This patch adds a missing capabilities NOTIFICATION message, and support foritojun1-3/+4
>RFC2385 (Protection of BGP Sessions via the TCP MD5 Signature Option). From: Greg Hankins <gregh@twoguys.org> git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2791 f5534014-38df-0310-8fa8-9805f1628bb7
2000-12-25bgp route refresh/MP capability option.itojun1-17/+219
Greg Hankins <gregh@twoguys.org> git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2780 f5534014-38df-0310-8fa8-9805f1628bb7
2000-11-19For each column, have both a buffer into which strings for that columnguy1-2/+2
can be put, and a pointer to the string for the column, which might or might not point to that buffer. Add a routine "col_set_str()", which sets the string for the column to the string passed to it as an argument; it should only be handed a static string (a string constant would be ideal). It doesn't do any copying, so it's faster than "col_add_str()". Make the routines that append to columns check whether the pointer to the string for the column points to the buffer for the column and, if not, copy the string for the column to the buffer for the column so that you can append to it (so you can use "col_set_str()" and then use "col_append_str()" or "col_append_fstr()"). Convert a bunch of "col_add_str()" calls that take a string constant as an argument to "col_set_str()" calls. Convert some "col_add_fstr()" calls that take a string constant as the only argument - i.e., the format string doesn't have any "%" slots into which to put strings for subsequent arguments to "col_set_str()" calls (those calls are just like "col_add_str()" calls). Replace an END_OF_FRAME reference in a tvbuffified dissector with a "tvb_length(tvb)" call. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2670 f5534014-38df-0310-8fa8-9805f1628bb7
2000-08-20- proto_bgp was not added in protocol tree, so "bgp" was notdeniel1-6/+17
working as display filter... - add display filters. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2310 f5534014-38df-0310-8fa8-9805f1628bb7
2000-08-13Add the "Edit:Protocols..." feature which currently only implementsdeniel1-1/+3
the following: It is now possible to enable/disable a particular protocol decoding (i.e. the protocol dissector is void or not). When a protocol is disabled, it is displayed as Data and of course, all linked sub-protocols are disabled as well. Disabling a protocol could be interesting: - in case of buggy dissectors - in case of wrong heuristics - for performance reasons - to decode the data as another protocol (TODO) Currently (if I am not wrong), all dissectors but NFS can be disabled (and dissectors that do not register protocols :-) I do not like the way the RPC sub-dissectors are disabled (in the sub-dissectors) since this could be done in the RPC dissector itself, knowing the sub-protocol hfinfo entry (this is why, I've not modified the NFS one yet). Two functions are added in proto.c : gboolean proto_is_protocol_enabled(int n); void proto_set_decoding(int n, gboolean enabled); and two MACROs which can be used in dissectors: OLD_CHECK_DISPLAY_AS_DATA(index, pd, offset, fd, tree) CHECK_DISPLAY_AS_DATA(index, tvb, pinfo, tree) See also the XXX in proto_dlg.c and proto.c around the new functions. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2267 f5534014-38df-0310-8fa8-9805f1628bb7
2000-08-11Miscellaneous code cleaningdeniel1-12/+8
- add <stdarg.h> or <varargs.h> in snprintf.h and remove those inclusions in the other #ifdef NEED_SNPRINTF_H codes - remove the check of multiple inclusions in source (.c) code (there is a bit loss of _cpp_ performance, but I prefer the gain of code reading and maintenance; and nowadays, disk caches and VM are correctly optimized ;-). - protect all (well almost) header files against multiple inclusions - add header (i.e. GPL license) in some include files - reorganize a bit the way header files are included: First: #include <system_include_files> #include <external_package_include_files (e.g. gtk, glib etc.)> Then #include "ethereal_include_files" with the correct HAVE_XXX or NEED_XXX protections. - add some HAVE_XXX checks before including some system header files - add the same HAVE_XXX in wiretap as in ethereal Please forgive me, if I break something (I've only compiled and regression tested on Linux). git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2254 f5534014-38df-0310-8fa8-9805f1628bb7
2000-08-07Allow either old-style (pre-tvbuff) or new-style (tvbuffified)guy1-2/+2
dissectors to be registered as dissectors for particular ports, registered as heuristic dissectors, and registered as dissectors for conversations, and have routines to be used both by old-style and new-style dissectors to call registered dissectors. Have the code that calls those dissectors translate the arguments as necessary. (For conversation dissectors, replace "find_conversation_dissector()", which just returns a pointer to the dissector, with "old_try_conversation_dissector()" and "try_conversation_dissector()", which actually call the dissector, so that there's a single place at which we can do that translation. Also make "dissector_lookup()" static and, instead of calling it and, if it returns a non-null pointer, calling that dissector, just use "old_dissector_try_port()" or "dissector_try_port()", for the same reason.) This allows some dissectors that took old-style arguments and immediately translated them to new-style arguments to just take new-style arguments; make them do so. It also allows some new-style dissectors not to have to translate arguments before calling routines to look up and call dissectors; make them not do so. Get rid of checks for too-short frames in new-style dissectors - the tvbuff code does those checks for you. Give the routines to register old-style dissectors, and to call dissectors from old-style dissectors, names beginning with "old_", with the routines for new-style dissectors not having the "old_". Update the dissectors that use those routines appropriately. Rename "dissect_data()" to "old_dissect_data()", and "dissect_data_tvb()" to "dissect_data()". git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@2218 f5534014-38df-0310-8fa8-9805f1628bb7
2000-05-11Add tvbuff class.gram1-87/+87
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. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@1939 f5534014-38df-0310-8fa8-9805f1628bb7