cleaned up bgp_attr_print{}
  removed p,dat pointers and replaced through tptr,pptr where
    pptr is a pointer to the start of the attribute;
    tptr is a moving pointer that is incremented through the decoding cycle;

basic support for extended communtities

For packets with a bad ICMP checksum, show the bad checksum and what it
should have been.

For packets with a bad TCP checksum, show the value from the header, and
the value it should have had, as we do for packets with a bad IP
checksum, rather than showing the non-zero computed checksum.

Add a "in_cksum_shouldbe()" to compute, given an Internet checksum field
in a packet header and the computed checksum for the data that the
checksum covers, what the checksum field's value should have been.

Use that routine in the IP printer.

Set "off" before using it.

From Phil Wood <[email protected]>: fix the test for a community of "public"
to check the length of the community name before doing a length-limited
comparison.

better idendation and dedicated line for the common header;

use the new public print_unknown_data{} rather then the private
              isis_print_unknown_data{}

better support for unknown BGP attributes, messages, AFIs & SAFIs;
hexdump the data rather than ommitting them;

yet another hexdump routine;
print_unknown_data{} supports identation;

Update Kevin Steves' address.

Add items for SunATM.

Update and fix up the list of files and the note on Linux.

ETHERTYPE_IP will never show up as an NLPID in a Frame Relay packet, as
the NLPID is 1 byte and 0x800 doesn't fit in one byte.

Clean up "safeputchar()" a bit.

In "safeputchar()", use the result of extracting an unsigned character
from the argument, rather than the argument itself, when testing whether
it should be printed or not (the argument might well be a sign-extended
version of an 8-bit character, in which case it's < 0x80 as it's
negative, but it can't be safely handed to "isprint()").

the last commit contained a bogus commit message;
purpose of 1.105 was to print the ip header values (len, ttl, tos)
at the beginning of the line (where it belongs to) and not at the end;
it is only displayed using the -v(erbose) options;
the BGP output (where i am working now on) gets much more cleaner now;

split BGP output to multiple lines;
added support for Originator_ID, Cluster List Attributes;
reworked BGP Open Message Decoder do decode optional Capabilities;
fixed the NEXTHOP decoder of MP_REACH and MP_UNREACH to honor
 NH length based on AFI,SAFI

fix the MP_REACH_NLRI and MP_UNREACH_NLRI to bail out if a SAFI is unknown
rather than trying to decode it using decode_prefix4()

Make static the structure to which the test program checking what to use
for "inlines", so that we don't get tripped up by compilers complaining
that we're returning a pointer to an automatic variable.

"support" for the loopback ethertype 0x9000

Add SunATM support, based on code from Yen Yen Lim at North Dakota State
University.

Make "snap_print()" take an argument specifying the padding for bridged
LAN frames; it's 0 bytes for Frame Relay and 2 bytes for ATM.

From Paul S. Traina: FreeBSD's Frame Relay printer.

Make the code to print a SNAP-encapsulated packet a separate routine,
for use by the Frame Relay code.

Put in some casts to squelch compiler warnings.

Use EXTRACT_* to be a little more alignment-safe
Use ntohs() on the type code instead of defining the type numbers
 based on our endianness.

Get rid of no-longer-used "num_or_str()" function.

Remove print-mobility.c - I hadn't re-run configure.  Sorry for
 the braino.

Add print-mobility.c

Include the ICMP length

finally get rid of all the macros for codepoint to
string conversion and use tok2str() instead;

replace some macros for printing AFI and SAFI information with tok2str()

Don't try to print entries before the first one in subtypes.

Submitted by: Anonymous sourceforge user (req ID 576536)

some SAFIs for labeled BGP (code still missing)

Fix up AC_LBL_C_INLINE to put all of the test code into the "inlines"
section - the other section gets stuck in the middle of "main()", and,
although GCC lets you define functions inside functions, other C
compilers don't.

Pick a name more likely to be unique to use as the iterator variable in
AC_LBL_C_INLINE.

Make AC_LBL_C_INLINE print "no" rather than "not supported" if inline
isn't supported, to match what AC_C_INLINE does.

Fix indentation.

Put variables in quotes when testing them in "test" commands, so that
the test doesn't get a syntax error if the variable isn't set or is set
to a null string.

Some versions of the HP C compiler can handle inlines, but not if they
return a structure pointer.  Check whether the C compiler can handle
inline functions that return a structure pointer, not whether they can
handle inline functions that return an int, as at least some versions of
autoconf's AC_C_INLINE do.

From Kazushi Sugyo <[email protected]>:

Here is a patch for draft-ietf-mobileip-ipv6-17.
- print-mobility.c: Mobility header(MIPv6 defines a new IPv6 protocol)
  printing
- added Routing header type 2
- ICMPv6 Home Agent Address Discovery and Mobile Prefix Sol., Adv.
  message update

Document that -x prints padding too.

Give Andrew Hintz credit for the tcpdump.1 typo fix "-b" -> "-r".

Fix a typo.

avoid trigraph

regen from configure.in 1.150

dhcpv6 i-d revision 26, by jinmei@kame

__ss_len/family is now nonstandard.  standards were clarified to use
ss_len/family.

whitespace

whitespace cleanup

Get rid of unneeded includes of <netinet/in_systm.h> and <netinet/ip.h>.

Don't bother including <netinet/if_ether.h> if we're not using
"ether_ntohost()" - the only reason we include it is to get
"ether_ntohost()" declared.

We only include <net/if.h> to get "struct ifnet" in "struct arpcom"
defined on Solaris, so that we don't get errors when including
<netinet/if_ether.h>, so don't include that unless we're including
<netinet/if_ether.h>.

We only need incomplete definitions for "struct mbuf" and "struct
rtentry" if we're including <net/if.h>, so don't define them unless
we're including <netinet/if_ether.h>.

Get rid of unneeded incomplete definitions of "struct mbuf" and "struct
rtentry", and unneded includes of <sys/uio.h>, <sys/file.h>, and
<sys/ioctl.h>.

"gmt2local.c" doesn't need "struct timeval", so it doesn't need to
include <sys/time.h>.

support for

ES Neighbor(s)     TLV #3
Partition DIS      TLV #4
Prefix Neighbors   TLV #5
Prefix Neighbor(s)
 (variable length) TLV #7

clarified use of SYSTEM_ID_LEN, ETHER_ADDR_LEN, LSP_ID_LEN

plus some cleanups, boundary checks and modularization
 (isis_print_metric_block)

pedant

V3 FSINFO and PATHCONF requests take a file handle as an argument; parse
the request header and the file handle.

"parsestatus()" should only return NULL if the packet was truncated, so
that its callers can distinguish "packet truncated" from "NFS reply
status reports an error".

NFS-request-or-reply parsing routines that call "parsestatus()" should
not return 0 if "parsestatus()" returns a non-zero NFS reply status, so
that non-truncated packets with a non-zero NFS reply status aren't
printed as truncated NFS packets.

Fix some "TCHECK2()" calls to pass an object, not a pointer to an
object, as the first argument (the TCHECK/TTEST macros take objects, not
addresses, as the test argument).

NFS-request-or-reply parsing routines should return 0 for truncated
packets and 1 for non-truncated packets, not *vice versa*.

In TTEST2(), check to make sure the "l" argument isn't so large that
"snapend - l" underflows; this fixes a buffer overflow with malformed
NFS packets, and may fix other buffer overflows with malformed packets.

typo, from openbsd

Make "-x" and "-X" work with PPPoE interfaces; based on code from Darren
Reed, with additions to get the PPP header length from the PPP print
routine.

Only do "-x" printing in the top-level interface print routine; don't do
it in the routines, called by the top-level routines, to dissect
Ethernet, FDDI, or Token Ring packets, as those routines might also be
called for bridged frames over ATM, and the interface print routine for
ATM will also do "-x" printing.

Get rid of an extra newline.

Make static routines, and string buffers, not used outside
"print-isoclns.c".

Fix a typo.

Mention the SourceForge site.

when reporting a bad checksum, indicate what it should have been.

adjusted instructions to include :pserver: in CVS example.

tony has moved a while ago ;-)

cosmetic cleanup for the print_nsap routine

From Rick Watson <[email protected]>: use
unaligned-extraction macros to fetch 16-bit and 32-bit big-endian
quantities.

from Dirk Steinberg: check if the bottom of stack bit is set correct in
RFC3107 BGP labeled unicast NLRI messages

Explain, in full painful detail, what you need to do in order to do
*useful* packet capture on Digital/Tru64 UNIX (and add "Tru64 UNIX" to
the list of OS names).

From Phil Wood <[email protected]>: if we're exiting due to a "pcap_loop()"
error, report statistics and clean up first.

From Hannes Gredler: support BGP labeled unicast.

From Hannes Gredler: support the IS-IS buffersize TLV #14, and clean up
some names.

Fix a misspelling.

If we run out of data in the when processing tagged parameters, return
"success", not "failure" - there's no special "end of list" parameter.
Get rid of the check for a parameter tag of 0xff, for the same reason.

Take out an extra space between the "[" and the list of rates in a
"supported rates" tagged parameter, and don't print the "[" or the "
Mbit]", or the leading space if there are no supported rates (or there
is no "supported rates" parameter.

Use "fn_print" to print the SSID, in case there are non-printable
characters in it.

Show the channel number in decimal, not hex, as that's how it's given in
the IEEE 802.11 spec.

Show the supported rates in a probe response, if present.

Get rid of random extra trailing space in tagged parameter output.

Print the whole MPLS label stack.

Submitted by: Kaarthik Sivakumar <[email protected]>

Cosmetic TCHECK() change.

Handle old DR-priority and old Bidir-capable option values.

Fix spacing with -v

From Juergen Schoenwaelder <[email protected]> - fix for the
following problem:

According to the ASN.1 documents, OIDs of the form 2.100.3 are
correctly encoded as 0x0603813403.  tcpdump fails to properly
decode them - it displays the value 4.20.3 instead of 2.100.3.

Fix a formatto.

From Sami Farin <[email protected]>: put in a missing blank before "NBT
Packet".

From Hannes Gredler: support for the IP-Reach-subTLV #2 and
IDRP-info-TLV #131.

Squelch some compiler warnings caused by passing "char *"s rather than
"unsigned char *"s to "fn_print()".

Add a cast to squelch a compiler complaint - and remove a potential
problem, if the string in question probably contains any non-ASCII
characters.

Another bug fix from Gert Doering.

From Gert Doering <[email protected]>:

The VLAN ID calculation was wrong (either I had a bug in the
switch I tested the original code against, or I was just stupid)
- the "-1" offset doesn't belong there.

In addition, there are some new TLV values that Cisco has
started using.

Add some casts to squelch compiler complaints (and remove potential
problems, although the string in question probably won't contain any
non-ASCII characters).

Make the tag in "rfc1048_print()" a 16-bit variable, as 16-bit
quantities are stuffed into it.

Add an unsigned 8-bit variable as well, for 8-bit unsigned quantities.

Stuff unsigned 16-bit quantities into 16-bit variables.

Print unsigned values with "%u".

Cast the argument to "fn_printn()" to "u_char *", as that's the argument
type it expects.

Convert some format pointers to "char *" from "u_char *", and some data
pointers from "char *" to "u_char *", to eliminate signed vs. unsigned
complaints from some compilers.

Don't use "++"s in initialization, to avoid warnings from the Sun C
compiler.

Make some "u_char *"s "char *"s, to eliminate signed vs. unsigned
complaints from some compilers.

Constify.

Make "payloadPtr" a "u_char *" rather than a "char *", to eliminate some
signed vs. unsigned complaints from some compilers.

Make "ar_sha()", "ar_spa()", "ar_tha()", and "ar_tpa()" return "const
u_char *" rather than "const caddr_t", to avoid signed vs. unsigned
complaints from various compilers.

"espsecret_xform" and "espsecret_key" are static to "print-esp.c"; don't
declare them in "interface.h".

Regenerate.

"-A" flag to print packet data in ASCII, from Jørgen Thomsen
<[email protected]>.

Nobody seemed to know why we preferred sigset() to sigaction(); the
latter is a POSIX standard, and you don't have to include some
additional header file to get it declared, so we'll prefer the it.

Get rid of casts of pointers-to-const to pointers-to-nonconst.

Protect includes of <time.h> with "#ifdef TIME_WITH_SYS_TIME".