print-ppp: variable 'length' is updated and the new value is never used

NDOize IP mobile, MSDP and NULL decoders

VRRP: top off the previous change

Update the test cases and squelch a compiler warning.

Add support for VRRPv3 (IPv4 only).

VRRP: add a sample capture and two test cases

The sample capture consists of VRRPv2/IPv4, VRRPv3/IPv4 and VRRPv3/IPv6
packets. It was produced using 7 MikroTik devices running RouterOS 6.10
and configured for the same set of virtual addresses with different
priorities per device. The devices were powered on one after another
with a ~30 seconds delay in the order of VRRP priority ascending. The
test cases decode the capture with and without "-v".

Fix commit 833425ab: babel uses 1/100th of seconds, not 1/10th

Also update affected testcases.

ZMTP: fix an MSVC compiler warning

The prototype and implementation of 'zmtp1_print_datagram()' is slightly
different; the prototype has a 'const u_int len' as the last parameter.
Whereas the implementation has no 'const'. Hence I get a warning from
MSVC.

Use a common style for all option parsers.

Report "length bogus, should be ..." for invalid lengths for all
options.  That's not a fatal error - the next option can be parsed - so
don't return 0 in that case.

Don't put a colon after the option length unless there's something
following it.

Do the TCHECKs for particular CCP options in the code to handle the
option, as is done elsewhere.

Print the values for the CCP options on the same line, as is done
elsewhere.

Print some PPP CCP options; taken from issue #214, with changes.

Report the CCP protocol type as CCP.

Decode options for BSD, MRVCA, and DEFLATE compression.

Add some explanatory comments, and add some length checks, while we're
at it.

issues/286: get rid of Useless "no IPv4 address assigned" warning

NDOize HSRP, IGRP and UDLD decoders

remove unused print-netbios.c

The file had no purpose as the draft code in it wasn't compiled.

update Gisle's e-mail address per his request

Travis script: set silent operation for make

This change allows to see better the possible warnings

NDOize CIP, IPComp and IPFC decoders

Also remove some unneeded includes while at it.

don't include pcap.h needlessly

Both interface.h and netdissect.h include <pcap.h>, thus most files
should not include it regardless if these need it or not. The only
exceptions so far remain:
* addrtoname.c
* missing/datalinks.c
* missing/dlnames.c
* tcpdump.c

Travis script: refactor env section

Ask for more warnings during a Travis CI build

.travis.yml: add 'touch .devel configure'

NDOize AH, BEEP and DTP decoders

Also remove some unnecessary passages.

revert the latest change

./configure fails during the "BUILD_LIBPCAP=true BUILD_IPV6=true"
scenarios (both GCC and Clang):

Makefile:8411: warning: overriding commands for target `configure'
Makefile:455: warning: ignoring old commands for target `configure'
Makefile:8416: warning: overriding commands for target `stamp-h.in'
Makefile:460: warning: ignoring old commands for target `stamp-h.in'
Makefile:8421: warning: overriding commands for target `stamp-h'
Makefile:465: warning: ignoring old commands for target `stamp-h'
Makefile:8424: warning: overriding commands for target `Makefile'
Makefile:468: warning: ignoring old commands for target `Makefile'
Makefile:8427: warning: overriding commands for target `config.status'
Makefile:471: warning: ignoring old commands for target `config.status'
[...]
./config.status: line 1103: syntax error near unexpected token `done'
./config.status: line 1103: `done # for ac_tag'

I don't have the expertise to resolve it well right now, keep Travis CI
working meanwihle.

request more warnings during a Travis build

dismiss decode_prefix.h

The only purpose of that header file was to provide two declarations
that now fit interface.h same well.

refine the previous change

This time eliminate a few instances of the code initializing struct tok
with negative values and squelch the warnings like below:

warning: initializer will be sign-extended: -1

base struct tok on unsigned rather than signed int

This resolves a few Solaris-specific warnings emitted by
/opt/solarisstudio12.3/bin/cc like below:

warning: initializer does not fit or is out of range: 0xffffffff

(also happened for 0x80000000)

NFS: unify TCP and UDP decoding, fix a bug

Change the UDP NFS printing format to match the current format of TCP
NFS printing (this complements commits f3051bc and 29d83db).

Update the man page to use the new format (this complements commit
2c86c75).

Restore printing of NFS packet length, which was broken in commit
68fe98a.

CARP: NDOize

don't include addrtoname.h needlessly

Do a bunch more length checking.

Make sure we don't run past the end of the packet or, when processing
the packet header, run past the end of the header.

make OpenSSL-specific tests conditional

Merge pull request #366 from fxlb/dccpv6

fix partial checksum errors in DCCP decoder, IPv6 case

fix partial checksum errors in DCCP decoder, IPv6 case

Merge git://github.com/the-tcpdump-group/tcpdump

Merge pull request #364 from fxlb/dccp

DCCP tests: more verbosity (from -v to -vv)

Fetch the sequence number the way we fetch the acknowledgment number.

Fetch 24 bits if the X bit isn't set, 48 bits if it is, using the
appropriate EXTRACT_ macros.

We do this with "struct dccp_hdr" being a header structure with a 24-bit
sequence number and "struct dccp_hdr_ext" being a header structure with
a 48-bit sequence number.

Use an 8-octet array when the ACK field is always 8 octets.

Get rid of "struct dccp_hdr_ack_bits" while we're at it.

Get rid of unused function.

DCCP tests: more verbosity (from -v to -vv)

Add {40,48,56}-bit big-endian extract macros and use them for DCCP ACKs.

Add macros to, given an octet pointer, extract 40-bit, 48-bit, and
56-bit big-endian numbers from the location pointed to by that pointer,
and use them when extracting ACK numbers from DCCP packets.  This fixes
problems on big-endian(!) machines.

Mark structures with UNALIGNED.

Maybe this will fix the crashes that appear to be occurring on the
opencsw.org buildbot; it's building with Sun/Oracle C, not GCC, but it's
at least worth a try.

Merge pull request #363 from fxlb/master

fix partial checksum errors in DCCP decoder, IPv4 case

fix partial checksum errors in DCCP decoder, IPv4 case

only include udp.h when necessary

print-pptp: add a missing break

print-pptp: pptp_result_code_print(): badly placed break

RRCP: remove extra return

print-mobile: mobile_print(): variable 'cp' is assigned a value that is never used

print-forces: print_metailv(): unused parameter 'len'

print-802_11: cleanup handle_deauth()

- variable 'offset' is assigned a value that is never used
- parameter 'length' is updated and the new value is never used

print-icmp6: remove useless break

Clean up the TLV processing loop.

We want to process TLVs until we run out of data, so make the main loop
condition "length > 0".  Break out of the loop if we see an
NFULA_PAYLOAD TLV.

Add in a bunch of length checks, and make those and existing length
checks report "[|nflog]".

With -A and -AA, don't send CRs to the standard output.

They don't belong on the ends of lines on UN*X, and the standard I/O
library will give us one at the end of the line on Windows so they're
not needed there.  In the middle of a line, just print a ".".

Use the new libpcap <pcap/nflog.h> for NFLOG definitions and declarations.

Use the header file for information about LINKTYPE_NFLOG files.

Don't build the NFLOG printer if we don't have that header file.

Fix some checks for pcap/XXX.h header files to look for
"tcpdump-stdinc.h" rather than <tcpdump-stdinc.h>, so that we don't fail
to find that header.

Do our own isascii(), isprint(), isgraph(), and toascii().

We do *not* want the behavior of isprint() and isgraph() to be
locale-dependent - we want both of them to return "true" only for ASCII
characters.

We have to do our own isascii() and toascii() on non-UN*X systems
anyway, so let's just do all of them ourselves.

Don't assume snprintf() returns the number of characters formatted.

On some UN*Xes (such as some versions of HP-UX), it doesn't.

Fix a compiler warning.

It's not necessarily safe to do pre-increment or post-increment in an
argument to an EXTRACT_ macro.

Don't use the __attribute__((packed)) on most platforms.

It won't necessarily work with non-GCC-compatible compilers, so use it
only on GCC-compatible compilers.

Even with some GCC-compatible compilers (such as, err, umm, GCC), it
doesn't do the right thing on some platforms, e.g. 64-bit SPARC, where
the compiler generates code that assumes alignment even when using it.
The only platforms I know of where an unaligned big-endian load can be
done better than by loading bytes and shifting-and-ORing them together
are MIPS and possibly Alpha, so only do it there.

Not all OSes have a diff that supports "diff -u".

justify some syntax in Travis spec

missing/inet_ntop.c fix (GH #358)

The extra 'return (NULL)' is dead code.

fix more Solaris testing issues

echo doesn't support "-n"
grep doesn't support "-q" and "-e"

the bugfix missing from the previous commit

fix some shell mistakes after a Solaris run

It worked on Linux and FreeBSD but OpenCSW build produced the following:

./lmp-v.sh: !: not found
./TESTrun.sh: local: not found
./TESTrun.sh: test: unknown operator ==

LMPv1: make the "-v" test GCC-specific (GH #333)

test scripts overhaul

Rewrite TESTonce to do only one thing (run a test with given parameters)
but do it well. Split TESTrun.sh into functions and extend it to do only
a specific test if requested. Justify format of the test results and
move most of the test results printing from TESTrun.sh to TESTonce.

The interval in an AODV HELLO extension is not aligned on a 4-byte boundary.

As with memcpy, so with memcmp.

More UNALIGNED_MEM{CPY,CMP} on IP addresses.

Another case where UNALIGNED_MEMCPY() is probably necessary.

It was necessary in other cases where we copied IP addresses from the IP
header, so it's probably necessary here as well.

No need for casting back and forth.

That might also convince the compiler that it can "helpfully" optimize
the copy into something that assumes 4-byte alignment, which we don't
want it to do on platforms where the optimized code will trap if the IP
header *isn't* aligned on a 4-byte boundary, given that there's no
guarantee that it *is* aligned on a 4-byte boundary.

Fix typo.

Only do the unaligned_mem{cpy,cmp} hack if necessary.

If the processor does unaligned accesses, it's not necessary.

No need to declare unaligned_mem{cpy,cmp} in netdissect.h *and* interface.h.

More possibly-unaligned memcpy()s and assignments - use unaligned_memcpy().

Check for compiling for IPv6; don't check whether we can create an IPv6 socket.

Don't check whether we can create a TCP-over-IPv6 socket; that appears
to fail in some cases where tcpdump can be built with IPv6 support.
Instead, just check whether AF_INET6 and struct in6_addr are defined.

We don't use missing/sockstorage.h, so get rid of it.

We're not using struct sockaddr_storage any more, so don't check for it.

Rework the SA list code.

This simplifies the code, gets rid of some type-punning about which GCC
warns, and reduces the size of the list entries.

We no longer use struct sockaddr_storage, so we don't need to define it.

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

Again, it's necessary because Git seems to think it's necessary.

We don't use PACKED, so get rid of it.

clean LIBNETDISSECT_SRC definition up

Use one file per line syntax and sort the files by name.

Use unaligned_memcmp() to compare with IPv{4,6} addresses in a packet.

They are, after all, not guaranteed to be aligned.

Use EXTRACT_nBITS even when just testing against zero.

*All* references to possibly-unaligned multi-byte fields have to be done
in an unaligned-safe fashion.

Use AC_CHECK_TOOL to find ar.

On some platforms, such as at least some versions of HP-UX, AR isn't
automatically set by make.

Another length check, also found by the Clang Static Analyzer.

Add some length checks for byte-escaped PPP.

Thanks and a tip of the Hatlo hat to the Clang Static Analyzer for
catching these.

Fix some more unaligned accesses.

Rework the cookie cache.

This simplifies the code, gets rid of some type-punning about which GCC
warns, and reduces the size of the cache entries.

Merge git://github.com/the-tcpdump-group/tcpdump

"Please enter a commit message to explain why this merge is necessary" -
because Git appears to think it is, for some reason, otherwise it
wouldn't have told me to enter that message.

Merge pull request #356 from msekletar/master

print-forces: remove unused variable

print-forces: remove unused variable

memcmp() doesn't modify either of its arguments.

Revert print-tcp memcpy() changes, and use unaligned_memcpy() instead.

That should prevent optimizing the memcpy into code that assumes
alignment.

Add unaligned_memcmp(), and use it, as well.

Get rid of form feed in source.

Move safememcpy() to util.c so it doesn't get inlined.

It appears that some C compilers will inline safememcpy() *and* will, as
a result, optimize to assume alignment it if it's passed a
putatively-aligned pointer. As the pointers in question are not
guaranteed to be aligned, that can cause crashes on, for example, SPARC.

Also, rename the function to unaligned_memcpy(), to clarify what's
"safe" about it, and change some direct memcpy() calls to use it as
well.

Fix a bug the previous change made a bit more obvious.

Don't use a u_int16_t * to extract data from packets.

That convinces some compilers that the data is 16-bit aligned, but
there's no such guarantee of alignment.

Add changes for 4.5.1 and post-4.5.1 changes.

Point to IP addresses with "void *"s.

This keeps GCC from assuming the IP addresses are aligned and generating
32-bit load/store pairs on SPARC; those require alignment and cause
tcpdump to crash.

Mark the TCP header structure as unaligned.