summaryrefslogtreecommitdiff
path: root/print-isoclns.c
Commit message (Collapse)AuthorAgeFilesLines
* ISOCLNS: Fix a segmentation faultFrancois-Xavier Le Bail2015-11-191-19/+23
| | | | | Moreover: Avoid two consecutive truncation messages.
* ISOCLNS: Fix two segmentation faultsFrancois-Xavier Le Bail2015-11-191-33/+45
| | | | | | | | Moreover: Add/fix a lot of bounds and length checks. Add and use tstr[] string. Fix some typos. Update the output of a test according to these changes.
* ISOCLNS: Fix some warningsFrancois-Xavier Le Bail2015-11-191-5/+6
| | | | | | | | | | | The warnings were: ./print-isoclns.c: In function 'osi_print_cksum': ./print-isoclns.c:3107:23: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] || length > ndo->ndo_snaplen ^ ./print-isoclns.c:3108:32: warning: comparison between signed and unsigned integer expressions [-Wsign-compare] || checksum_offset > ndo->ndo_snaplen ^
* Add a missing space to a printout.Guy Harris2015-09-021-1/+1
|
* Not all C compilers accept C++/C99 comments by default.Guy Harris2015-04-141-3/+7
| | | | Use "#if 0" instead to comment out code.
* protected osi_print_cksum is more general fashionMichael Richardson2015-03-091-6/+9
|
* some patches to deal with kday pcap filesMichael Richardson2015-03-091-2/+11
|
* clean K&R style up in function declarations a bitDenis Ovsienko2015-03-051-10/+10
| | | | The function body should have its opening brace on the next line.
* Don't run past the snaplength when printing a packet with a too-short LI.Guy Harris2015-03-011-1/+1
| | | | Fixes GitHub issue #437.
* Squelch a Coverity warning.Guy Harris2015-02-061-2/+1
| | | | | For unknown PDU types, we return 0 whether print_unknown_data() succeeds or fails; don't bother checking whether it succeeds, Just Do It.
* Leave it up to ip6_print() to handle non-IPv6-capable systems.Guy Harris2014-10-011-2/+0
| | | | | | | | | | | | Always define and declare ip6_print(), always compile print-ip6.c, and always call it if we recognize a payload as IPv6. If INET6 isn't defined, ip6_print() will just print the length and note that printing isn't supported. That way, we don't do weird dissection of IPv6 packets on systems without IPv6 support, due to, for example, ethertype_print() returning 0 ("not dissected") for IPv6 packets on those systems (IPv6-over-Frame Relay was dissected weirdly due to this).
* u_intN_t is dead, long live uintN_t.Guy Harris2014-04-231-106/+106
| | | | | | | | | And, as we require at least autoconf 2.61, and as autoconf 2.61 and later have AC_TYPE_UINTn_T and AC_TYPE_INTn_T macros, we use them to define the uintN_t and intN_t macros if the system doesn't define them for us. This lets us get rid of bitypes.h as well.
* Netdissectify the to-name resolution routines.Guy Harris2014-04-041-12/+12
| | | | | | | | Have them take a netdissect_options * argument, and get the "no name resolution" flag from it. Move the declaration of dnaddr_string to addrtoname.h, along with the other XXX-to-string routines.
* NDOize some generic codeDenis Ovsienko2014-04-031-1/+1
|
* NDOize LLDP, PPP and RSVP decodersDenis Ovsienko2014-04-021-3/+3
|
* improve some recent conversionsDenis Ovsienko2014-03-291-2/+2
|
* NDOize ISO CLNS decoderDenis Ovsienko2014-03-281-603/+599
|
* NDOize Frame Relay, LMP and RADIUS decodersDenis Ovsienko2014-03-181-1/+1
|
* Fix a compiler warning.Guy Harris2014-02-011-1/+2
| | | | | It's not necessarily safe to do pre-increment or post-increment in an argument to an EXTRACT_ macro.
* remove tcpdump's own CVS keywordsDenis Ovsienko2014-01-031-5/+0
| | | | | | Remove lots of $Header's and a few $Id's that all belong to the former CVS repository of tcpdump itself. These keywords have been frozen since the migration to git in late 2008.
* ndo-ize print-ascii: hex_print_with_offset()Michael Richardson2014-01-011-25/+25
|
* whitespace changesMichael Richardson2014-01-011-59/+59
|
* justify declarations of struct tok arraysDenis Ovsienko2013-09-241-33/+33
| | | | | | Make sure all of them are declared const and most of them -- static. Proper declaration of token arrays is a common review point for new code that is based on existing decoders. Thus fix the issue at its root.
* Fixes-for-test-resultsAjith Adapa2013-02-071-40/+48
|
* Fix some errors.Guy Harris2012-06-071-3/+3
| | | | | | | | | isis_print_mcid() returns no value and nobody expects it to; declare it void, not int. Fix a typo in a case statement ("defualt" rather than "default"). Fix indentation.
* Added support for SPBM Service Identifier and Unicast Address subtlvAjith Adapa2012-05-221-12/+45
|
* Added support for SPBAjith Adapa2012-05-221-0/+309
|
* Get rid of an unused variable.Guy Harris2011-12-081-4/+0
|
* Clean up option-walking code.Guy Harris2011-10-131-4/+7
| | | | | | | | | Loop as long as the remaining option list length is not zero, even if that means we try to process the remaining options if the remaining length is 1, so that if the option length is bogus, we'll report it. Check for a valid ESIS_OPTION_ES_CONF_TIME length - it's supposed to be 2.
* From Minto Jeyananth <minto@juniper.net>: add support for IS-IS Generic ↵Hannes Gredler2011-03-021-1/+12
| | | | Cryptographic Authentication as per RFC 5310
* From Minto Jeyananth <minto@juniper.net>: fix the IS-IS MT flag tokentableHannes Gredler2011-03-021-2/+2
|
* Convert the IPv6 printer to use netdissect.Guy Harris2010-11-071-1/+1
|
* bugfix: the IS-IS Multi-topology TLV MTID is prepended once per TLV and not ↵hannes2008-08-161-7/+6
| | | | once per MT-IS neighbor
* Add HMAC-MD5 verification support for IS-IShannes2008-08-161-4/+25
|
* bugfix: the Multi-topology header is just at the beginning of the TLV and ↵hannes2007-03-021-13/+15
| | | | not before every prefix
* from Jacek Tobiasz <Jacek.Tobiasz@atm.com.pl>:hannes2007-02-221-2/+2
| | | | display the IS-IS istype field as L2 and not L1L2.
* print shouldbe checksum for broken OSI checksum, remove osi_cksum()hannes2006-09-251-40/+28
|
* do not bail if we hit a zero-length TLV, continue to read the next TLV ↵hannes2006-06-161-2/+2
| | | | header instead.
* From Don Ebright: fix a place where we were referring to AF_INET6 evenguy2006-05-121-3/+6
| | | | when INET6 wasn't defined.
* fixup typo from last commithannes2006-04-141-2/+2
|
* add support for link attribute subTLV, update GMPLS printer to latest spec ↵hannes2006-04-101-12/+48
| | | | (rfc4205)
* bugfix: wrong offset calculation for restart holdtimerhannes2006-04-061-2/+2
|
* move AF specific stuff to dedicated fileshannes2006-02-211-16/+13
|
* Get rid of an #ifdef - to zero something out with memset, just useguy2005-11-121-6/+2
| | | | "sizeof something" as the size argument.
* From Markus Schoepflin: don't use "struct in6_addr" if INET6 isn'tguy2005-10-161-1/+21
| | | | | | | defined, because, in that case, "struct in6_addr" probably isn't defined, either. Check for too-large bit lengths in TLVs.
* retain the old diffserv-te codepoints for a transition periodhannes2005-09-201-2/+5
|
* But the PDU length checks back, so "tmp" doesn't go negative.guy2005-09-051-11/+16
| | | | | | | At least as I read RFC 3847, you won't get a system ID without the remaining hold time field; after printing the flags field, check whether there's anything else after the flag field and, if not, quit, otherwise print the remaining hold time field and, if present, the system ID field.
* rework the restart signaling TLV printer:hannes2005-09-051-15/+22
| | | | | -accept TLV length of 1 as this is now perfectly valid as per rfc 3847 -only attempt to print a trailing system-ID and not a node-ID
* update diffserv-TE codepoints as per rfc4124hannes2005-08-231-12/+12
|
* code cosmetics: add a few TLV_MINLEN #defineshannes2005-07-141-13/+22
|