Address Michael's comments.

Be more portable when opening stdin.

Refactor reading from VFile into a function.

Properly print when dlt changes.

When we open a new file we need to get new printinfo.

DLT change detection.

If when writing an output file, error if the DLT of subsequent files is not
the same as the original.

Add multi-file support.

Merge remote branch 'github/master'

Merge pull request #20 from infrastation/master

another Babel update

As we're always printing "LLDP, length XXX", don't do it for the system name.

Add "radius" as an option for -T.

This allows tcpdump to handle RADIUS running on non-standard ports.

Submitted-By: [email protected]
Man page also updated by me.

Note that we now document "-T carp".

Document "-T carp".

Update for changes in the 4.3 branch.

LLDP: print packet protocol at all verbosity levels

The LLDP printer doesn't show the packet protocol unless -v is used,
which results in pretty useless output lines where only the timestamp is
present. Make sure we include the default protocol+length output even in
default mode.

update Babel decoder for DLen in HD TLV

This change reflects a new Digest Length field added to Hash Digest TLV
in the current draft of Babel authentication spec. babel_print_v2() is
updated to fetch and validate the new field. Test files too.

Merge pull request #18 from infrastation/master

fix printing of infinite lifetime in ICMPv6

fix printing of infinite lifetime in ICMPv6

icmp6_opt_print() in the case of ND_OPT_PREFIX_INFORMATION always
appended "s" (seconds) to the returned string result of get_lifetime(),
which could be either a printed number or the "infinite" word. This
change leaves it up to get_lifetime() to decide if "s" is necessary.
This also improves the ND_OPT_ROUTE_INFO case, where route information
lifetime has the same type and semantics.

Merge pull request #17 from infrastation/master

RIPv1/v2 updates

fix Babel type 11 TLV length check

RFC6126 4.3. TLV Format: "If the body is longer than the expected length
of a given type of TLV, any extra data MUST be silently ignored."

decode RIPv2 authentication up to RFC4822

This change addresses a few issues in rip_entry_print_v2() and
rip_print():

1. In the case of Simple Password (RFC2453) authentication the last
(16th) character of a password was never printed. Other password
characters were printed regardless of existing isprint() test.

2. In the case of Cryptographic (RFC4822) authentication there were no
details available for fixed-size auth header and variable-size auth
trailer.

3. Depending on particular hash function used, a normal authentication
trailer "RTE" may be 20 or more bytes long. Iteration over packet RTEs
should stop once a trailer is decoded. Exact number of RTEs in a message
cannot be told from message size any more.

Test cases are added for Request and Response messages with Simple
Password, Keyed-MD5, HMAC-SHA-1, HMAC-SHA-256, HMAC-SHA-384 and
HMAC-SHA-512 authentication modes. Earlier test case is updated to match
new "number of routes" output format.

fix RIP Request/full table decoding issues

RIP Request and Response messages have the same structure. Update a
switch block in rip_print() to proceed with decoding of both.

A Request message may contain an AFI 0 RTE standing for a full table
request, normally sent on a router start. Update rip_entry_print_v1()
and rip_entry_print_v2() to treat IPv4 and AFI 0 as two valid,
distinguishable cases.

No declarations in the middle of a block - not all C compilers support that.

Fix "no IPv6" code path.

Get rid of unused (but set) variable.

Check the return value of some print routines.

Some routines return -1 on error; bail and return -1 if they do.

Add some attribute/TLV length checks.

Make sure we don't run past the end of a BGP attribute or LDP TLV when
dissecting the attribute/TLV.

Make some of the code do a bit more of a "step the pointer through the
data"-style dissection; that was done while debugging the changes in
question.  It also fixes up some code to not check for more data than
should actually be there.

Update references to RFC 4906 from the draft, and note that RFC 4447
replaces it.

0x%02d is silly - 0x means "hex", so it should be 0x%02x.

Found by Xavier Heiny.

"lmp.new" isn't a checked-in file; don't make it part of the distribution.

Fix an error and a warning.

Use a separate variable to index through the hash digest, not the
variable used to index through the packet.

Make some variables holding one-byte quantities u_int rather than u_char
while we're at it - not much use in *requiring* them to fit into 8 bits
(the code might be better if we don't, and that eliminates a warning).

Fix some errors.

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.

Fix printing of 64-bit quantities.

Do *NOT* assume that "%l[doxu]x" - or "%ll[doxu]" - is the way to print
a 64-bit quantity; on UN*X, it might be a "long" or a "long long",
depending on whether you're on a 32-bit or 64-bit platform and, on
Windows with MSVC++, it's not a long (even in 64-bit mode) and doesn't
use "%ll[doxu]", either.  Instead, use PRI[doxu]64; that's what C99
defines, and what we define ourselves if the C environment doesn't
define it.

Merge pull request #16 from infrastation/master

add decoding of Babel authentication TLVs

add decoding of Babel authentication TLVs

This change extends existing Babel protocol decoder to recognise
experimental authentication TLVs (types 11 and 12) and includes
a respective test case for the new code.

Merge pull request #15 from ajithadapa/master

Added spb.out and spb.pcap

Adding the test files for SPB

Merge pull request #14 from ajithadapa/master

Support for SPB related SUB-TLVs

Added test files for spb.

spelling fixes

Merge pull request #12 from msekletar/master

Dropping root privileges

Dropping root priviledges using Linux Capabilities

Added support for SPBM Service Identifier and Unicast Address subtlv

Added support for SPB

Merge remote-tracking branch 'infrastation/master'

Merge pull request #11 from yekm/master

Minor manpage change

minor manpage fix

add DNSSL (RFC6106) support

The most notable difference between RFC5006 and RFC6106 is the addition
of DNSSL RA option. This commit adds DNSSL handling code to make tcpdump
fully RFC6106-aware. This code has been tested against RA packets
generated by Quagga and radvd.

added missing test files to releasetar

do not ship bittypes.h

do not convert protocol to names if -n flag

minor whitespace changes

updated platforms

VERSION updated back to GIT

updated test case outputs

prep for 4.3.0 release

fixes from Jamal Hadi Salim <[email protected]> for forces: SPARSE data (per RFC 5810)

whitespace changes in print-ip.c

added icmpv6 test case

error in passed calculation

Give more information on -l, and note that -U works without -w.

Note that -l, on Windows (i.e., in WinDump), is unbuffered, not
line-buffered, and describe -U as an alternative (that doesn't have that
problem on Windows).

Note that -U does packet-buffering without -w.

Fix up the formatting of the example command lines with -l.

Discuss buffering when describing the -w flag.

Mention that the output of tcpdump -w is buffered, and note that this
means packets won't necessarily show up in the output as soon as they're
received, so programs reading that output won't see them immediately,
and note that the -U flag forces "packet buffering" so that a reader
will see the packets as soon as they're received.

Fix the other typo, so setting CPPFLAGS in the environment works correctly.

Based-On-Patch-From: Simon Ruderich <[email protected]>

Ask for the libpcap Makefile while we're at it.

...so that we can see what's being used to turn grammar.y into
grammar.c.

While we're at it, say "config.log file from the libpcap source
directory", in the hopes of making it more likely that we'll get it in
addition to the tcpdump config.log.

Fix typo, so setting CPPFLAGS in the environment works correctly.

Based-On-Patch-From: Simon Ruderich <[email protected]>

Ask for more information if we don't find pcap_loop.

I give up.

People keep reporting that the configure process for tcpdump fails to
find pcap_loop, and the config.log file they send us says there's no
pcap_parse in libpcap, which suggests that something went wrong in the
build process for libpcap; perhaps they don't have Bison and the
configure script got confused and failed to cause the parser to be named
"pcap_parse", or something such as that, or perhaps Bison was recently
"improved" in a fashion that breaks that, but I've never been able to
reproduce this on any of the Linux distribution installations to which
*I* have access.

I therefore ask them to send the config.log output and make output for
libpcap; *not one of the reporters of this problem* has bothered to send
that information, so we're stuck.  Perhaps they don't care enough (in
which case, why did they bother asking us about it?), or perhaps they're
annoyed that we asked them a further question rather than Just Fixing
The Problem(TM) (in which case, all I have to say is "welcome to the
Wonderful World Of Computer Software(TM) - get used to it").

So let's just ask for all that information.  (I would not be surprised
if this doesn't suffice and that they *still* just send us the tcpdump
config.log output, but at least I'll be able to tell them that they
should have Read The Fine Error Message(TM).)

Fix printing of BGP optional headers.

Anonymously contributed patch; tested with Cisco and Arista routers by
the patch contributor.

Back out DLT_PFSYNC support.

Unfortunately, the DLT_PFSYNC support depends on header files included
from the pfctl command's source tree, and trying to arrange to find that
would be too much trouble.

Add DLT_PFSYNC support.

From FreeBSD PR bin/124825: tcpdump(8) does not support pfsync(4) data,
which in turn was ported over from OpenBSD.  We already have CARP
support, so we did not port that part over.

Put the addresses early in the message format, and handle short messages.

Putting the addresses after "TIPC vX.Y" matches other protocols, such as
IPv4/IPv6.

Not all messages have full headers.

Also, constify some parameters and variables.

Print only one line in non-verbose mode.

Also, don't print extra blank lines in verbose mode.

TIPC support.

Reviewed-and-much-modified-by: Guy Harris <[email protected]>

Print a space after the options if there are any options.

Get rid of extra blank after unknown options.

Reviewed-By: Guy Harris <[email protected]>

Use expr instead of bashisms.

Reviewed-By: Guy Harris <[email protected]>

Merge remote branch 'github/master'

Merge pull request #7 from clifffrey/fix-lldp-network-policy

Fix LLDP Network Policy bit definitions.

Merge pull request #6 from fenner/master

IGMPv3 Max Response Time is in units of 0.1 second.

Merge pull request #5 from kmaehashi/feature-SIGUSR1

Use SIGUSR1 to request info when SIGINFO is not defined

added test packet for icmpv6 advertisement interval fix

Fix LLDP Network Policy bit definitions.

Previous commit accidentally used 6 seconds cutoff.

60 seconds == 600 in units of 0.1, oops.

IGMPv3's Max Response Time is in units of 0.1 second.

If it's less than 60 seconds, print it as %.1fs, to be able to
accurately represent small values including the tenths-of-a-second.
Only use relts_print() when it is 60 seconds or more.

tcpdump now accepts SIGUSR1 as a signal to print the statistics on systems that does not support SIGINFO

Merge remote branch 'github/master'

Merge branch 'master' of git+ssh://bpf.tcpdump.org/tcpdump/master/git/tcpdump

permit -n flag to affect print-ip for protocol numbers

Merge pull request #4 from infrastation/master

ND_OPT_ADVINTERVAL is in milliseconds, not seconds

ND_OPT_ADVINTERVAL is in milliseconds, not seconds

RFC 6275 7.3. New Advertisement Interval Option Format

   Advertisement Interval

      32-bit unsigned integer.  The maximum time, in milliseconds,
      between successive unsolicited Router Advertisement messages sent
      by this router on this network interface.

Note the length checks in the Babel printer.

Add length checks, use EXTRACT_16BITS().

Use EXTRACT_16BITS() rather than a hand-rolled macro to extract
big-endian 16-bit quantities from the packet.

When processing the message, check against the body length *and* the UDP
payload length, as well as against the raw frame length.

Add the new PPPoE test files.

Merge branch 'master' of git+ssh://bpf.tcpdump.org/tcpdump/master/git/tcpdump

Merge pull request #3 from bodgit/master

Test case for PPPoE & RFC 4638 Payload Tag

Add simple PPPoE test case

Add changes in 4.2.1.

Merge remote branch 'github/master'

Merge pull request #2 from bodgit/master

Teach PPPoE parser about RFC 4638

Teach PPPoE parser about RFC 4638

Fix a typo (from the Red Hat tcpdump package) and use .LP.

Get rid of an unused variable.

Fix a bunch of "sizeof(sizeof(XXX))".

In some places, there was one too many levels of sizeof() -
sizeof(sizeof(XXX)) is sizeof(size_t), but we wanted the size of type
XXX.

Reviewed-By: Guy Harris <[email protected]>

Add a CARP dissector and a command-line option to dissect proto 112 as CARP.

CARP and VRRP both use IP protocol number 112, so there needs to be a -T
flag to specify that protocol 112 be dissected as CARP rather than VRRP.

Also update the man page.

Redo length checks in ospf6_print_lsa().

Check to make sure we haven't run past the end of the LSA by doing
length checks - and be a bit fussier about length checks.  Do more
end-of-packet checks as well.