aboutsummaryrefslogtreecommitdiffstats
path: root/packet-bacnet.c
AgeCommit message (Collapse)AuthorFilesLines
2004-05-12Make the calls to build the protocol tree regardless of whether we'reguy1-172/+166
building one or not - we have to do some of that work anyway, so we know the offset of the payload and so that we call sub-dissectors, and the easiest way to do that is to do all the work (you make more procedure calls that way than you'd like, but the procedures at least return quickly when they find they've been passed a null pointer; hopefully that won't add too much CPU time to dissection of BACNET traces on the first pass). Don't use "proto_tree_add_uint_format()" in a case where it's not necessary - and fix the field being used not to have a bitmask, as it's not a bitfield. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@10861 f5534014-38df-0310-8fa8-9805f1628bb7
2003-01-25From Dave Richards: BACNET-over-ARCNET is BACNET-over-802.2-over-ARCNET,guy1-3/+1
not BACNET-directly-over-ARCNET. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@7000 f5534014-38df-0310-8fa8-9805f1628bb7
2003-01-23Support BACNET over ARCNET, under the assumption that no fragmentationguy1-1/+3
is done. Support "Novell EC" over ARCNET, under the assumption that it's just another way of sending IPX over ARCNET. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6988 f5534014-38df-0310-8fa8-9805f1628bb7
2002-11-16From Dave Richards: support BACNET over 802.2.guy1-1/+4
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@6644 f5534014-38df-0310-8fa8-9805f1628bb7
2002-08-28Removed trailing whitespaces from .h and .c files using thejmayer1-46/+46
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-02Replace the types from sys/types.h and netinet/in.h by their glib.hjmayer1-9/+1
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-07-17From Joerg Mayer:guy1-5/+1
dftest.c: Remove #if-0-ed includes packet-ieee80211.c, packet-wtls.c, packet-afp.c, packet-wsp.c, packet-wtp.c, ethereal_gen.py: Remove redundant include varargs (already in snprintf.h, and required only for snprintf.h) Remove unused include of snprintf.h from files not using "snprintf()". git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@5889 f5534014-38df-0310-8fa8-9805f1628bb7
2002-05-30Don't pass "tvb_reported_length_remaining(tvb, offset)" as the fourthguy1-2/+3
argument to "tvb_new_subset()" - just use -1 if the subset tvbuff is to run to the end of the parent tvbuff. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@5598 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-20Allow a length of -1 to be specified when adding FT_NONE and FT_PROTOCOLguy1-2/+2
items to the protocol tree; it's interpreted as "the rest of the data in the tvbuff". This can be used if 1) the item covers the entire packet or the remaining payload in the packet or 2) the item's length won't be known until it's dissected, and will be then set with "proto_item_set_len()" - if an exception is thrown in the dissection, it means the item ran *past* the end of the tvbuff, so saying it runs to the end of the tvbuff is reasonable. Convert a number of "proto_tree_add_XXX()" calls using "tvb_length_remaining()", values derived from the result of "tvb_length()", or 0 (in the case of items whose length is unknown) to use -1 instead (using 0 means that if an exception is thrown, selecting the item highlights nothing; using -1 means it highlights all the data for that item that's available). In some places where "tvb_length()" or "tvb_length_remaining()" was used to determine how large a packet is, use "tvb_reported_length()" or "tvb_reported_length_remaining()", instead - the first two calls indicate how much captured data was in the packet, the latter two calls indicate how large the packet actually was (and the fact that using the latter could cause BoundsError exceptions to be thrown is a feature - if such an exception is thrown, the frame really *was* short, and it should be tagged as such). Replace some "proto_tree_add_XXX()" calls with equivalent "proto_tree_add_item()" calls. Fix some indentation. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4578 f5534014-38df-0310-8fa8-9805f1628bb7
2001-12-10Move the pointer to the "column_info" structure in the "frame_data"guy1-5/+5
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-08Attach a descriptive name field type and base to dissector tables; thatguy1-9/+8
specifies how the selector values used as keys in those tables are to be displayed, and the title to use when displaying the table. Use that information in the code to display the initial and current entries of various dissector tables. Have the dissector for BACnet APDUs register itself by name, and have the BACnet NPDU dissector call it iff the BAC_CONTROL_NET bit isn't set, rather than doing it with a dissector table. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4358 f5534014-38df-0310-8fa8-9805f1628bb7
2001-12-03Make "dissector_add()", "dissector_delete()", and "dissector_change()"guy1-5/+8
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-26Moved from using dissect_data() to using call_dissector()hagbard1-3/+5
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@4269 f5534014-38df-0310-8fa8-9805f1628bb7
2001-06-18From Joerg Mayer: explicitly fill in all members of aguy1-28/+28
"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-05-30Get rid of some unnecessary settings of "pinfo->current_proto" - if theguy1-3/+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-04-20Fix Gerald's e-mail address - and fix it in "README.developer", soguy1-2/+2
people don't blindly insert the old address into new dissectors, which is probably how it got into the dissectors being fixed here. git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3348 f5534014-38df-0310-8fa8-9805f1628bb7
2001-03-31BACNET support, from Hartmut Mueller.guy1-0/+607
git-svn-id: http://anonsvn.wireshark.org/wireshark/trunk@3214 f5534014-38df-0310-8fa8-9805f1628bb7