IEEE 802.15.4: Fix two loops for undefined behavior at runtime

The errors were:
print-802_15_4.c:433:17: runtime error: unsigned integer overflow: 0 - 1
cannot be represented in type 'u_int' (aka 'unsigned int')
print-802_15_4.c:483:17: runtime error: unsigned integer overflow: 0 - 1
cannot be represented in type 'u_int' (aka 'unsigned int')

Merge branch 'kivinen-master'

Merge branch 'master' of https://github.com/kivinen/tcpdump into kivinen-master

Merge pull request #733 from marka63/dns-get-extended-rcode

Extract extend DNS rcode from OPT record.

Travis CI: Use ccache and parallel make.

Travis CI: Add debug infos

Add a folded 'cat Makefile' (without the mkdep part for autotools).
Add a folded 'cat config.h'.
Add a folded 'cat config.log' (for autotools).

The 'sleep 10' command is a workaround for a display problem on osx.

LMP: Fix a loop for undefined behavior at runtime

The error was:
print-lmp.c:1132:13: runtime error: unsigned integer overflow: 249 - 516
cannot be represented in type 'unsigned int'

Update the output of a test accordingly.

LMP: Rename a variable

Use 'length' as in most similar cases.

PPP: Update two error messages

Update the output of a test accordingly.

RIP: Update two error messages

Update the output of a test accordingly.

Bluetooth: Print the protocol name

Add the nd_print_protocol() function

Use the ndo->ndo_protocol field to print the protocol name.

Squelch a clang warning

The warning was:
./tcpdump.c:182:5: warning: no previous extern declaration for non-static
variable 'dflag' [-Wmissing-variable-declarations]

PPP: Fix two loops for undefined behavior at runtime

The errors were:
print-ppp.c:518:9: runtime error: unsigned integer overflow: 38 - 230
cannot be represented in type 'unsigned int'
print-ppp.c:1138:63: runtime error: unsigned integer overflow: 204 - 220
cannot be represented in type 'unsigned int'

Add a test case.

PPP: Fix inaccurate variable names

Clean up whitespaces

NFS: Fix an undefined behavior at runtime

The error was:
print-nfs.c:537:2: runtime error: unsigned integer overflow:
4294967295 + 3 cannot be represented in type 'unsigned int'
print-nfs.c:541:14: runtime error: unsigned integer overflow:
4294967295 + 3 cannot be represented in type 'unsigned int'

Add a test case.

CDP: Fix two loops for undefined behavior at runtime

The errors were:
print-cdp.c:363:13: runtime error: unsigned integer overflow: 0 - 1
cannot be represented in type 'u_int' (aka 'unsigned int')
print-cdp.c:375:13: runtime error: unsigned integer overflow: 0 - 1
cannot be represented in type 'u_int' (aka 'unsigned int')

BRCMTAG: Fix some warnings

The warnings were:
no previous extern declaration for non-static variable 'brcm_tag_te_values'
[-Wmissing-variable-declarations]
no previous extern declaration for non-static variable 'brcm_tag_rc_values'
[-Wmissing-variable-declarations]

ICMPv6: Add the IPv6-Only flag

This flag in defined in a draft which is close to getting published as
an IETF RFC: https://tools.ietf.org/html/draft-ietf-6man-ipv6only-flag-05.

SNMP: Fix an undefined behavior at runtime

The error was:
print-snmp.c:503:14: runtime error: unsigned integer overflow: 0 - 1
cannot be represented in type 'uint32_t' (aka 'unsigned int')

Travis CI: Print the devel libs in Linux builds

Travis CI: Fix the problem with folded outputs

Trailing newlines are now needed in travis_fold:start:* strings, because
of a Travis change.
Based on Travis support proposal.

IPX: Add a length check

This fix an undefined behavior at runtime.

The error was:
print-ipx.c:93:43: runtime error: unsigned integer overflow: 29 - 30
cannot be represented in type 'unsigned int'

Add a test case.

IS-IS: Fix inaccurate variable names

Add two TFTP tests

There is one pcap file with two outputs, with/without the '-T tftp' option.

TFTP: Print protocol name and length keyword

Update the output of a test accordingly.

Babel: Avoid an unsigned integer underflow

SMB: Avoid an unsigned integer underflow

RIP: Fix two loops for undefined behavior at runtime

1) RIPv2
The error was:
print-rip.c:386:9: runtime error: unsigned integer overflow: 16 - 20
cannot be represented in type 'unsigned int'

Without this change the unsigned integer variable 'len' is assigned a
very high value, because of underflow, and the loop continue incorrectly.

Add a test case.

2) RIPv1
Same bugfix, based on a code inspection, so comes without a test case.

RIP: Fix inaccurate variable names

Clean up whitespaces/indentation

ICMP: Fix an undefined behavior at runtime

The error was:
print-icmp.c:491:13: runtime error: unsigned integer overflow: 0 - 1
cannot be represented in type 'u_int' (aka 'unsigned int')

Use the ndo->ndo_protocol field instead of the protoname parameter

Moreover:
Fix a truncated string: "[!" is not the prefix for a truncated string.
Add two comments.
Clean up a whitespace.

Fix a typo

TCP: Fix an undefined behavior at runtime

The error was:
print-tcp.c:831:22: runtime error: unsigned integer overflow: 0 - 1
cannot be represented in type 'u_int' (aka 'unsigned int')

Merge pull request #734 from jkcko/master

Support DLT_IEEE802_15_4_TAP.

Clean up whitespaces/indentation

ISAKMP: Simplify an expression

Clean up whitespaces

Add two ndo_protocol reassignments after xxx_print() calls

Clean up whitespaces/indentation

IS-IS: Replace some ND_TTEST_*() calls with ND_TCHECK_*() calls

IEEE 802.11: Replace some ND_TTEST_*() calls with ND_TCHECK_*() calls

Moreover:
Use 'goto trunc' in some length tests.

Clean up whitespaces/indentation

RSVP: Replace a ND_TTEST_*() call with ND_TCHECK_*() call

GeoNet: Replace the ND_TTEST_*() calls with ND_TCHECK_*() calls

Moreover:
Add two comments

Clean up whitespaces

OLSR: Replace the ND_TTEST_*() calls with ND_TCHECK_*() calls

GRE: Replace the ND_TTEST_*() calls with ND_TCHECK_*() calls

Clean up whitespaces/indentation

Handle DLT_IEEE802_15_4_TAP.

https://github.com/jkcko/ieee802.15.4-tap

Add dns-badvers test case.

Add dns-badcookie test case.

Extract extend DNS rcode from OPT record and print it.

Add a test for NFS write verifier cookie in TESTLIST

This is a followup to commit 8b9f7d9d76.

Moreover:
Rename the .pcap file to .pcapng.

Remove some useless spaces

Remove a useless space

Change some pcap tests to test the printers again

The added sanity check was: packet length >= capture length.

Thus update packet length to be capture length if lower.

This change is a followup to commit 9e6ba479d8.

Merge pull request #722 from aieri/issue/695

Amend documentation about the use of the -C -G and -W switches.

Merge pull request #703 from nmap/update-gitignore

Add files generated by CMake on Windows to .gitignore

Merge pull request #728 from ffainelli/brcm-tag

Add support for dissecting Broadcom tags

Add Broadcom tags tests

Add 3 tests to test that we can properly dissect Broadcom tags. Two are
for DSA_TAG_BRCM where the tag is between Ethernet MAC SA and
Type/Length and one for DSA_TAG_BRCM_PREPEND where the tag is
encapsulating the Ethernet frame (thus requires -e to be shown).

Add support for decoding Broadcom Ethernet switches tags

This adds support for decoding the Broadcom Ethernet switches tags which
are are 4byte in length and are located between the Ethernet MAC SA and
the Type/Length field (DSA_TAG_BRCM) as well as the pre-pended Broadcom
tag (DSA_TAG_BRCM_PREPEND) which are located in front of the standard
Ethernet header.

This makes use of the recently introduced ether_print_hdr_len() to allow
specifying the non-standard Ethernet header length of 12 + 4 + 2 bytes.

Report 32-bit timestamps with uppermost bit set as "Error converting time"

In the ts_unix_print() function.

This change is a followup to commit 8e9fb1eb02.

Ethernet: Allow specifying non-standard Ethernet header length

A fair number of proprietary Ethernet switch tagging protocols, such as
Broadcom tags for instance, will place their tag between the MAC SA and
the Type/Length field.

Move the body of ether_print() into ether_print_hdr_len() and specify
the Ethernet header length as an argument to that function.
ether_print() calls ether_print_hdr_len() with a standard Ethernet
header lenght of 14 bytes, while other callers could specify an
arbitrary length. We still assume that the first Length/Type field to
parse is located 2 bytes before the end of that Ethernet header length.

This will be used in a subsequent commit to parse Broadcom tags.

Fix small misspellings

Update print-lldp.c

Small typo fix for `EVB Station` in Link Layer Discovery Protocol (LLDP) printer - `print-lldp.c`

Do AppVeyor builds on all branches.

Build the tcpdump-4.9 branch as well.

Handle rpcaps:// URLs as well, for rpcap-over-TLS.

Amend the documentation about the use of the -C -G and -W switches.

When using all three switches (-C -G and -W), the behavior is
practically identical to only using -C and -G, as the -W flag will only
affect the filename suffix.

Fixes #695

Squelch a narrowing warning.

Use a u_char variable for characters in the RST string; no need for it
to be an int.

Have all Internet-checksum computing routines return a uint16_t.

Those checksums are 16-bit; change the return types of those routines
appropriately.

Squelch some narrowing warnings.

Mostly, we just throw in casts of values already known to be in the
range of values supported by the lvalue's type to that type.

Also, for the IPX SAP table, we make the SAP value a uint16_t, as SAP
values are 16-bit.

Make a hash table index unsigned, Just Because.

Umm, wrong.

Add, and use, macros to do locale-independent case mapping.

This means we get the same behavior in Turkish locales (where, if we
aren't in the C locale, we might get lower-case "i" mapped to upper-case
"I with dot" and upper-case "I" mapped to lower-case "i without dot),
and may also suppress some shortening warnings from MSVC.

Revert "Try to squelch a warning about non-constant format strings."

This reverts commit 76ed0bb39212d964946a22380288c34adc09030f.

It doesn't help.

We may just want to suppress the warning, as in "trust us, we know what
we're doing", although it'd be nice if the format string passed to those
functions could be checked to make sure it is one that takes exactly one
integer argument.

Try to squelch a warning about non-constant format strings.

Declare the format-string argument to tok2strbuf(), tok2str(),
bittok2str(), and bittok2str_nosep() as such, to see whether that gets
MSVC not to complain - or if it just gets upset because there's no "..."
argument list for it.

Type and size cleanups.

Make the count of 2-octet values to dump an unsigned value.

Make the variables into which we fetch those octets unsigned as well.

Cast the result of the "is the character printable ASCII?  If so, it's
the character, otherwise it's '.'" to char, to squelch narrowing
warnings.

Squelch a warning.

We're returning the calculated checksum result modulo 65536, given that
Internet checksums are 16-bit; throw in a cast to let MSVC know that
this is intended.

Try to squelch another annoying repeated warning.

Build with the Npcap SDK as well as the WinPcap SDK.

Change tests/hoobr_bfd_print.pcap to test the BFD printer again.

Clamp the on-the-wire length values of the packets at 262144, so that we
hand the packets to the BFD printer, rather than just bailing out before
we ever get there.

Revert "Ethernet: Remove two useless tests"

This reverts commit b7fff7058282c332ba1d8d236c9864ab21b16adf (partially)

The sanity check in print.c/pretty_print_packet() function, packet length
(length) >= capture length (caplen), apply when the ether_print()
function is called by ether_if_print(), netanalyzer_if_print() or
netanalyzer_transparent_if_print().

But the ether_print() function is called in some other cases, thus not
sure length always >= caplen.

Fixed more warnings.

Fixed some warnings, added print-zep.c to CMakeLists

Removed some warnings

Updated 802.15.4 code

Handle radiotap+802.11 for DLT_LINUX_SLL2 as well.

Fix printing of Linux cooked captures with monitor-mode packets.

Apparently, if you have an interface in monitor mode, capturing on the
"any" device can get packets that have a SLL hatype of 803, which is the
ARPHRD_ value for radiotap, and with the payload containing a radiotap
header, followed by an 802.11 header, followed by the 802.11 payload.
Handle that.

ICMPv6: add a comment for the previous commit

[skip ci]

show ICMPv6 echo request ID

DCCP: Fix an undefined behavior at runtime

DCCP: Fix some undefined behaviors at runtime

The errors were like:
print-dccp.c:448:4: runtime error: unsigned integer overflow: 20 - 24
cannot be represented in type 'unsigned int'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior print-dccp.c:448:4

Squelch annoying bogus Clang warnings.

Require sys/capsicum.h to use Capsicum.

My PC-BSD 9.1 VM, at least, has sys/capability.h but not sys/capsicum.h;
we now use sys/capsicum.h, so require it to be present (which it is in
FreeBSD 10 and later).

Fix typo.

Require sys/capsicum.h to use Capsicum.

My PC-BSD 9.1 VM, at least, has sys/capability.h but not sys/capsicum.h;
we now use sys/capsicum.h, so require it to be present (which it is in
FreeBSD 10 and later).

Merge pull request #715 from rbgarga/fix_capsicum_header

Include correct capsicum header