Have separate "struct tok" PID-to-name tables for different OUIs; don't

Get rid of some extra blanks.

Base all interpretion of IE contents on the code set.

Get rid of an extra blank.

Don't dump core if a null "struct tok" array pointer is passed to
"tok2strbuf()".

bugfix: prevent infinite loop in ERO processing, print ERO subtype length

check against min. TLV length / get if logic right

bugfix: do not attempt to decode zero-length TLVs, only attempt to decode IS-REACH (varlength) TLVs if the TLV length indicates that there is at least a single byte on the wire

Don't exit just because an NSAP had a length of 0 or a too-long length;
just report it as an illegal length.

Check for some zero-length or too-short TLVs not already checked for.

Split a line to match the way it's split in libpcap's configure.in.

Albert Chin's patch to libpcap's configure.in:

AC_CHECK_DECLS will define the HAVE_DECL_xxx variable whether it
succeeds or fails, so you can't erase a "no" result and try
again with a different header.  Use AC_CHECK_DECL, which doesn't
define HAVE_DECL_xxx, and then explicitly define it based on
whether we found a declaration or not.

is required for tcpdump's configure.in as well.

Give a URL for the Zephyr protocol spec.

Squelch some unsigned vs. signed warnings.

Squelch a signed vs. unsigned warning.

Squelch some GCC 4.0 signed-vs-unsigned warnings.

Make the string pointer in "struct juniper_cookie_table" just a "char
*", so as to avoid GCC 4.0 warnings when initializing it from a string.

Declare "juniper_mfr_print()".

AC_TRY_COMPILE works only for code that fits inside "main()"; the test
for __attribute__ doesn't fit inside "main()" with GCC 4.0, as it
defines a function.

Put back AC_LBL_C_INLINE; it turns out that wasn't the cause of the
inline problem, and AC_LBL_C_INLINE checks for a case that some versions
of the HP C compiler don't handle, and only uses inline if that case
succeeds.

Get rid of an unused variable.

Get rid of, or use, unused variables.

Get rid of an unused variable.

Get rid of unused variables.

Check, and use, the length from the PAP header.

Get rid of an unused variable.  (It should be possible to tell the type
of address from the way it's printed.)

Add a comment asking about the address and mask length fields in
join/prune requests.

Get rid of an unused variable.

Attempt to squelch a warning from the SGI C compiler.

If the length is specified, check to make sure it doesn't go past the
end of the packet, and isn't shorter than the header length.

Control messages have to have lengths.

Get rid of an unused variable.

Get rid of an unused variable.

If we get an ELT other than a bit label in labellen(), log it.

Get rid of an unused variable.

#if out some other unused stuff, and note how it *should* be used.

Check for a negative "advance" value, which indicates an error.

Don't make the last statement in a loop a break statement, as that means
the body of the loop will be executed at most once.

When looping over the data in a TCP segment containing BGP packets, loop
until we reach the first of the end of the captured data or the end of
the segment, not just the end of the captured data.

Get rid of an unused variable (the length from the BGP header is passed
as the "length" argument).

Attempt to squelch a warning from the SGI C compiler.

refactor the juniper_parse header printer, add juniper FRF.16 dissector

Some systems (*cough*Irix*cough) not only don't declare
"ether_ntohost()" but don't even declare "struct ether_addr"; on those
systems, declare "struct ether_addr".

If we don't have ether_ntohost(), don't bother declaring it merely
because it's not declared in a system header file, as the reason it's
not declared is that it doesn't exist.

Put in some missing brackets.

Move some statements before the case's "break;" statement, so that
they're executed; if they shouldn't be executed, they shouldn't be in
the code.

"ipaddr_string()" takes, as an argument, a pointer to the bytes of an
IPv4 address, not an extracted IPv4 address value.

IRIX defines IPV6_VERSION in <netinet/in.h>, and we don't use it, so
remove the definition from ip6.h, to get rid of compiler warnings.

From Albert Chin:

  1. aclocal.m4 patch required by Tru64 UNIX which has
     inttypes.h in a non-standard location so gcc cannot find
     the PRI types. So, use <sys/bitypes.h> to get u_int#_t
     types for help detecting %llx, etc.
  2. Help gcc on aix find ether_ntohost declaration.

From Albert Chin:

  1. On AIX, AC_LBL_C_INLINE detected the compiler supported
     the inline keyword which is wrong. AC_C_INLINE from
     autoconf-2.59 worked.
  2. AC_CHECK_TYPE from autoconf-2.5x is no longer broken.
     Replaced AC_LBL_CHECK_TYPE with it, mainly to use
     <sys/bitypes.h> for Tru64 UNIX where some of the u_int#_t
     types are defined.
  3. Tru64 UNIX 4.0D doesn't support %llx; however, it does support %lx.
  4. Added <stdint.h> to interface.h for int#_t types on
     Tru64 UNIX 4.0D (required for missing/snprintf.c).
  5. Reworked includes in tcpdump-stdinc.h for int#_t types.

From Albert Chin: IBM's AIX C compiler complains about "&buf[sizeof
buf]", but doesn't complain about the equivalent "buf + sizeof(buf)".
(C89 allows a pointer into an array to point one element past the end of
the array, so both should, at least as I understand it, be legal, and we
decrement the resulting pointer before we use it, so both should be
safe.)

Declare "gndo" as "extern", so that it's only defined in tcpdump.c; on
some platforms, you get linker errors or, at least, warnings from
multiple definitions.

don't call bittok2str() twice in a printf() statement, fix typo (cc_values vs cv_values)

-bugfix: attempt to print ldp messages as long there is something on the wire
-improve support (add interface paramter TLV parser) for the Martini FEC TLV

backout redundant Sequence # printing

bugfix: don't forget to increment the tlv header size, misc. display cosmetics

Get rid of zero-length arrays in structures - they're not standard C89
and not all C compilers support them.

The minimum length of a SLARP packet is 14 bytes, not 18 bytes.  SLARP
requests don't have any data, apparently.

Document the "portrange" keyword.

Update the date stamp.

Make it clearer that a filter "ip" or "proto ip" or... refers to IPv4,
not IPv6.

From Noritoshi Demizu <[email protected]>: don't print "sack"
twice.

Boost size of buffers so that they can handle a 32-bit integer; from
OpenBSD.

Don't use "u_long", as its length is not guaranteed to be 4 bytes.  In
fact, don't even use "u_int32_t", as we're just pointing into a packet,
which is made up of bytes - use "u_char", and bump the pointer by 4 for
each item.

plumb in oui-name resolution

don't forget the boundary check, when dissecting an optional uptime parameter

More OUIs, end the oui_values table with the required {0, NULL} stopper.

Pull in some more SMI Network Management Private Enterprise Codes from
Ethereal.

add documentation for hierarchical VLAN filtering in libpcap 0.9

make the uptime-stamp printing optional as some implementations don't include that

fix compiler padding issue: struct cisco_slarp should be 18 bytes instead of 20

adjust test case to use -N and insert -t.

adjust #ifdef to work with NetBSD file as well, which
has slightly different multiple-include protection (alas)

use new ip_print_demux() routine in ISAKMP UDP-ESP encap
printer.

added ESP-in-UDP test cases.

added ESP-in-UDP test cases.

added ESP-in-UDP printer from RFC3948.

refactored ip_print() so that chained header parser (ESP/AH) can
more easily call the inner parts.

updates to test cases, "length" added.

make sure that pcap files are uu-encoded to avoid any
confusion.

move OUI #defines to the header file for other printer to share

move the SNAP OUI values out of the llc printer into
  the global oui.c file and make all SNAP printers use it

make the display output of the frame-relay SNAP printer consistent
  to the LLC SNAP printer (= print oui{id,name}, proto-id)

ignore output files.

tweaked copyright.

only attempt to print non-header data if there is something to print

Check for ether_ntohost() after checking for libpcap, so that we do so
after we've checked whether we need -lsocket -lnsl to get routines such
as ether_ntohost().

Don't check for netinet/ether.h unless we're going to use it.

Fix an error in the previous checkin.

Fix a typo.

Don't bother checking whether ether_ntohost() is declared if it exists
but is buggy so we won't be using it.

Do the same sort of checks whether ether_ntohost() is declared that we
do in libpcap for ether_hostton().  Include <netinet/ether.h> only if it
declares ether_ntohost().  If nothing declares it, declare it ourselves,
as we do in libpcap.

Don't cast the second argument to ether_ntohost() to a const pointer, as
some systems don't modify it but don't declare that argument as a const
pointer.  (This is similar to what we do on libpvap for
ether_hostton().)

Fix indentation.

When testing whether "inline" works, use the V_CCOPT flags, because some
of those might disable inlining; otherwise, the test for inlining will
succeed, but inlining won't work with the options we're using when
compiling.

Some UN*Xes need INET_ADDRSTRLEN defined as well, for "inet_ntop_v4()".

Protect some IPv6 code with #ifdef INET6/#endif.

Add declarations of __ntohl() and __ntohs() to squelch compiler warnings
on Solaris 7 x86 with GCC 2.8.1.

extra printing for IPv6 transcoded CLNP next-hops in the MP-REACH printer

-add support for CLNP and CLNP-VPN NLRIs
-treat the AFI/SAFI field as a composite 24-Bit entity
 and not as seperate 8/16-Bit entity

print the optional (payload) CLNP header in ERPDUs

add support for the CLNP 'Reason for Discard' Option

-add basic support for codeset shifting for IE printing in the frame-relay printer
-harden tok2str() and bittok2str() to catch NULL refs
-don't attempt to print a frame-relay IE if there is not enough bytes on the wire
 to print at least a full TLV

- bugfix: start CLNP checksumming at the packet start
- display the reqesting packet header of the Echo response payload

code cleanup:
  - remove the private print_nsap() and use isonsap_string() instead
  - pass on a length indicator to isonsap_string as CLNP and ES-IS,IS-IS TLVs
    and options store the length differently (semioctets vs. octets)
  - make the isonsap_string() output equal to the now abandoned print_nsap()
    printer which reflects representation of contemporary router software

CLNP updates:
  -add segment header support
  -add options processing
  -add some basic header sanity checking

From Gisle Vanem:

the _errno() stuff isn't needed for current versions of MinGW;

get rid of some definitions not needed with MSVC++ and other
non-MinGW32/non-Watcom compilers;

add IPv6 capability to inet_pton.c, courtesy of Paul Vixie;

add inline ntoh{ls}/hton{ls} functions for GCC/i386.

Squelch a (legitimate) compiler warning.

add support for Juniper ML-PPP DLT for printing captures
  taken on the Link-Service (LS) and Multi-Link (ML) PICs

add support for printing Multi-Link Frame Relay
  captured on Juniper ML-PIC and LS-PIC cards

-export the q933 printer and call it from the OSI printer
-display cosmetics:
 make the comma settings eflag compatible
 place a colon after printing the OSI proto-ID to indicate
  that we are crossing a layer boundary

cleanup: use (PPP_ADDRESS << 8 | PPP_CONTROL) rather than a private 0xff03