Fix a typo in a comment

[skip ci]

Revert "autotools: don't link with -lcrypto twice."

This reverts commit 036fa9c7b8be4bda360a1075427b3b2420a92986.

That doesn't work.  We'd have to figure out what -I flag to use with
that version of libcrypto and set the CFLAGS appropriately, and set
HAVE_LIBCRYPTO to yes.

If you have more than one non-system version of a library, hilarity is
likely to ensure.  Don't do that.

autotools: don't link with -lcrypto twice.

In case some other library with which we're linking - for example, a
version of libpcap that has rpcap-over-TLS support? - is causing us to
link with libcrypto, don't look for libcrypto ourselves.

Update the GitHub issue template

doc: Actualize README.solaris.md. [skip ci]

Remove old OpenIndiana release, add the most recent OmniOS release.

tests: Add two ARCnet packet captures. [skip appveyor]

The capture files were produced using an improvised ARCnet+Ethernet+LTE
network at Museum Of Communication in Burntisland.

ARCnet node 0xBE (the IPv4 host) is a Debian 3.0 PC with kernel 2.4.18
and a UM9065L-based ARCnet interface only.  ARCnet node 0x50 (the IPv4
router) is a Debian 6.0 PC with kernel 2.6.32, a COM20020-based ARCnet
interface and an Ethernet uplink to an LTE router (the latter also
appears in the captures as the DNS server).

The host pings the router and then makes an HTTP request to a web server
on the Internet, there is some ARP exchange as well.  One capture
represents this being done using RFC 1051 mode and another -- RFC 1201.

CI: Add warning exemptions for Sun C (suncc-5.15) on Solaris 10

CI: Add warning exemptions for Sun C (suncc-5.14) on Solaris 10

Autoconf: Print the GNU C Library version when needed in autogen.sh

CHANGES: Move change(s) backported to 4.99

[skip ci]

Fix a typo in a comment

Avoid -E and -M options inconsistencies with no libcrypto

The inconsistencies were:
-E
tcpdump: option requires an argument -- 'E'
-E xxx
tcpdump: WARNING: crypto code not compiled in

-M
tcpdump: option requires an argument -- 'M'
-M xxx
tcpdump: WARNING: crypto code not compiled in

CHANGES: Add change(s) backported to 4.99

[skip ci]

Print "invalid option -- z" when it is

It can only be used if defined(HAVE_FORK) || defined(HAVE_VFORK).
e.g. not on Windows systems.

Update print_usage().

Update the man page.

tcpdump.c: Avoid including two times "netdissect-stdinc.h"

Suppress bogus C4738 from MSVC.

Our trick to extract a big-endian IEEE floating-point number from a
packet confuses MSVC into thinking we're doing something we're not
doing, or maybe just that we might do so somewhere that it can't see.
Suppress the resulting warning.

This should fix issue #1318.

Rename a #define that's also define by Illumos.

Remove "\n" in error() calls: It is added in the function

CHANGES: Add change(s) backported to 4.99

[skip ci]

Add error messages about -C, -G and -z options usage

Also:
Replace warning by error messages about -A, -x[x] and -x[x] usage for
consistency with those on -f, -r and -V.
Update an error message about -f.

CHANGES: Add change(s) backported to 4.99

[skip ci]

Update the -z option processing

It can only be used if defined(HAVE_FORK) || defined(HAVE_VFORK).

This avoid:
tcpdump.c:3140:31: warning: unused parameter 'filename'
  [-Wunused-parameter] compress_savefile(const char *filename)

CHANGES: Add change(s) backported to 4.99

[skip ci]

Use __func__ in some error messages in compress_savefile()

tcpdump.c: Fix a warning when HAVE_FORK and HAVE_VFORK are not defined

The warning was:
tcpdump.c:226:13: warning: unused function 'child_cleanup'
  [-Wunused-function] static void child_cleanup(int);

Autoconf: Avoid incorrectly include the libpcap's config.h

Fix the problem when configure like following is done:
./configure CFLAGS='-I../libpcap -I.'
This gives:
CCOPT =  -I../libpcap -I. -W -Wall [...]

See also the GitHub bug report #1313.

In Makefile.in, there is:
INCLS = -I. @V_INCLS@

Put $(INCLS) first in FULL_CFLAGS.

Require config.h to be from the tcpdump build.

This way the build will reliably fail if it happened to include the
header from some other build (e.g. libpcap).  See GitHub bug report
#1313.

Cirrus CI: Bump FreeBSD up to 13.5.

Remove pcap-missing.h.

Complete commit d5b1a16: pcap_list_datalinks() has been available since
libpcap 0.8.

CHANGES: Add change(s) backported to 4.99

[skip ci]

NTP: Update a field name with the RFC 5905 name

s/Originate Timestamp/Origin Timestamp/
s/Originator Timestamp/Origin Timestamp/

Originate Timestamp: Obsoleted.
Originator Timestamp: Never used.

CHANGES: Add/move change(s) backported to 4.99

[skip ci]

CHANGES: Add change(s) backported to 4.99

[skip ci]

NTP: Fix p_ntp_time_fmt() using epoch 1/epoch 2 convention (RFC 4330)

Moreover:
Update the error message, replace
"[Time is too large to fit into a time_t]"
by
"[timestamp overflow]".
Same as in ts_date_hmsfrac_print() and ts_unix_print().

Fix indentation.

Update 5 test outputs and processings, based on skip_time_t_not (64).

Use a less cryptic message for packet timestamp overflow

In ts_date_hmsfrac_print() and ts_unix_print() replace
"[timestamp < 1970-01-01 00:00:00 UTC]"
by
"[timestamp overflow]".

(Wikipedia "Year 2038 problem" uses "overflow".)

DHCP: Add support for option 108 "IPv6-Only Preferred" (RFC 8925)

Add a test file with DHCP option 108 in both the request and the reply,
provided by Bill.

DHCP: Fix printing IP address pairs options

'p' is used by the options Policy Filter (21) and Static Route (33).
They have a minimum length of 8 and the length MUST be a multiple of 8.
Check that.

Add a test file for option 33 (lengths: 8, 16, 24, 3, 0), build with
scapy.

DHCP: Use more nd_print_invalid()

Remove "ERROR: " in the error messages. Use instead nd_print_invalid().

Update some error messages.

CHANGES: Add changes backported to 4.99

[skip ci]

DHCP: Fix printing boolean options

For 'B', the options 19, 20, 27, 29, 30, 31, 34, 36, 39 and 116 that
use it are only 1 octet "0/1" boolean. No need for a while loop.

Print the data and an error message if the length is not 1.

DHCP: Fix typos in some error messages

ICMPv6: Recognise ND option 38 (PREF64)

This option, defined in RFC 8781, allows a router administrator to pass
NAT64 prefix information for the network to end hosts together with
other routing and prefix information in the RA message, getting rid
of the need for DNS64 service in the network.

As of today the option is widely supported by software routers,
including radvd, BIRD and systemd-networkd.

We implement the printer and test some valid and broken option contents.

DHCP: Fix printing for option 116 "Auto-Configure" (RFC 2563)

DHCP: Fix print format for client-last-transaction-time option

Option 91 time (RFC 4388) is unsigned, thus use 'l'.

DHCP: Fix a comment about formats for (un)signed longs/shorts data

l - unsigned longs (32 bits)
L - longs (32 bits)
s - unsigned shorts (16 bits)

[skip ci]

CMake: Print the time_t size (32-bit/64-bit)

Autoconf: Use AC_SYS_YEAR2038_RECOMMENDED when possible

On Linux, if the environment variable BUILD_YEAR2038=yes, Autoconf
version >= 2.72 and GNU C Library version >= 2.34, uncomment
AC_SYS_YEAR2038_RECOMMENDED to ensure time_t is Y2038-safe.

Tested with: BUILD_YEAR2038=yes MATRIX_CMAKE=no ./build_matrix.sh
on linux-armv7l because currently CMake build don't have year 2038 support.

ICMPv6: Recognise ND option 14 (Nonce)

This option has been observed to be included in neighbour solicitations
sent by Linux kernel when `conf/*/enhanced_dad` is enabled on the
relevant network interface. It looks like it appeared in the linked
kernel commit.

In accordance with RFC 7527, a nonce can help distinguish valid DAD NS
messages in the presence of Ethernet loopbacks: after a DAD probe is
sent on the network, if an NS is received with the same nonce it is
considered looped back and ignored.

We implement the printer for this option and add a simple test with 3
different verbosity levels.

Also, introduce a helper function to util-print.c which prints
arbitrary data from the dissected packet as space-separated hexadecimal
octets, e. g. "77 c1 ad e9 17 70". Use it in the new printer.

Link: https://github.com/torvalds/linux/commit/adc176c5472214971d77c1a61c83db9b01e9cdc7
Link: https://www.iana.org/assignments/icmpv6-parameters/icmpv6-parameters.xhtml#icmpv6-parameters-5

tcpdump(1): Describe the -i flag better. [skip ci]

CHANGES: Add a change backported to 4.99

[skip ci]

Fix "ip broadcast" netmask byte order with the -f flag.

Let's suppose the interface eth0 has one IPv4 address with a /24
netmask.  Without -f tcpdump leaves the netmask variable set to 0, which
regardless of the host endianness causes "ip broadcast" to match
destination hosts 0.0.0.0 and 255.255.255.255:

# tcpdump -i eth0 -d 'ip broadcast'
(000) ldh      [12]
(001) jeq      #0x800           jt 2 jf 6
(002) ld       [30]
(003) jeq      #0x0             jt 5 jf 4
(004) jeq      #0xffffffff      jt 5 jf 6
(005) ret      #262144
(006) ret      #0

With -f tcpdump calls pcap_lookupnet(), which correctly sets the netmask
to 0xFFFFFF00 (in network byte order).  Then pcap_compile() receives the
same value, but it expects it to be in host byte order, so on a
little-endian host the resulting filter program incorrectly tests for a
0x00FFFFFF netmask:

# tcpdump -i eth0 -f -d 'ip broadcast'
(000) ldh      [12]
(001) jeq      #0x800           jt 2 jf 7
(002) ld       [30]
(003) jset     #0xff000000      jt 4 jf 6
(004) and      #0xff000000
(005) jeq      #0xff000000      jt 6 jf 7
(006) ret      #262144
(007) ret      #0

Add two missing ntohl() wrappers to make it right:
# tcpdump -i eno1 -f -d 'ip broadcast'
(000) ldh      [12]
(001) jeq      #0x800           jt 2 jf 7
(002) ld       [30]
(003) jset     #0xff            jt 4 jf 6
(004) and      #0xff
(005) jeq      #0xff            jt 6 jf 7
(006) ret      #262144
(007) ret      #0

Audit the init_print() code path and do not change anything because
there the byte order is already correct.  Add comments to spell the byte
order in every case and update the -f flag description in the man page.
See also libpcap commit 1e54958.

Autoconf, CMake: Check for libcap-ng on Linux only.

Autoconf, CMake: Check for Capsicum on FreeBSD only.

Autoconf: Check for Homebrew on Linux and macOS only.

Same as in libpcap.

TESTrun: Copy some recent improvements from libpcap.

Add a help message and the same command-line options as in libpcap
(--passed, --one, --list, --config).  Add a licence boilerplate.  Move
some code around to make it cleaner.  To check for Windows, test for
"msys" only.  Use diff on all platforms.  Drop TESTRUN_PERL.

Autoconf, CMake: Define PACKAGE_URL.

Make CMake PACKAGE_xxxxx definitions in config.h reproduce Autoconf in
the same go.

Fix comments about bpf_dump() in Autoconf and CMake.

ICMPv6: Add printing for Node Information Queries Qtype "IPv4 Addresses"

Add two test files.

ICMPv6: Use a variable with a better name, simplifie some codes

For Node Information Queries Qtype "Node Addresses".

Add two test files.

ICMPv6: Replace custom code with bittok2str_nosep() calls

Moreover:
Remove a "just experimental. not in spec" flag ("anycast").
Replace a comment "undefined for query" by a check.

RESP: Replace a fn_print_char() call by a ND_PRINT() call

fn_print_char() is used to filtering out the non-printable characters.
Unnecessary here.

ICMPv6: Delete some remains from drafts for Node Information Queries

These drafts date from 1999-2002.

ICMPv6: Fix duplicate type printing for Node Information Queries

Delete some duplicate macros, keeping macros with names similar to the
RFC 4620 names.

Update the outputs of two tests.

ICMPv6: Delete some duplicate macros

Keep macros with names similar to IANA names.

Refine bpf_dump(). [skip appveyor]

Constify the temporary pointer, as was done in libpcap in 2008.  Remove
the BDEBUG conditional block becase in libpcap that is a part of the
optimizer debug feature, but in tcpdump it is dead code.

IPv6: Print 'No Next Header' when relevant

This avoids to print 'next-header unknown (59)' when it's the first
header.

Add a test file.

ISAKMP: Avoid reinventing nd_printjn()

FR: Avoid reinventing nd_printjnp()

LDP: Avoid reinventing nd_printjn()

PPP: Avoid reinventing nd_printjn()

RSVP: Avoid reinventing nd_printjn()

ISAKMP: Avoid reinventing nd_printjn()

L2TP: Avoid reinventing nd_printjn()

TCP: Update the Reset segment processing

Don't use "[!RST" for truncation, use standard [|tcp].

Replace the nd_printn() call by a nd_printjn() call.

Moreover:
Add a space (style).

SMB: Avoid reinventing nd_printjn()

DNS: Use ND_TCHECK_LEN() instead of a custom bounds check

This avoids an invalid-pointer-pair AddressSanitizer error.

Add a test file.

The error was:
reading from file ppp_ip_udp_dns.pcap, link-type PPP_SERIAL(PPP over
  serial), snapshot length 2098998
=================================================================
==258093==ERROR: AddressSanitizer: invalid-pointer-pair: 0x61300000c086
  0x61300000019e
    #0 0x78274f in ns_rprint [...]/print-domain.c:728:9

Address 0x61300000c086 is a wild pointer inside of access range of size
  0x000000000001.
0x61300000019e is located 0 bytes after 350-byte region
  [0x613000000040,0x61300000019e)
allocated by thread T0 here:
    #0 0x4b997e in malloc ([...]/tcpdump+0x4b997e)
    #1 0x60220b in ppp_hdlc [...]/print-ppp.c:1367:16

SUMMARY: AddressSanitizer: invalid-pointer-pair
  [...]/print-domain.c:728:9 in ns_rprint

Use more the ND_TCHECK_n() macros

Reimplement the tests similarly to libpcap. [skip appveyor]

Import TESTlib.pm, TESTst.pm and TESTmt.pm from libpcap.  In TESTrun use
strict and warnings, also address all Perl issues that manifest because
of that, remove old code that the imported files make redundant and
clean the rest up.  Ibid., remove the core dump file before every test
and apply the "only this one test" mode even if the test does not come
from the TESTLIST file; focus on tcpdump specifics and have the imported
files handle all test/result logistics.

The latter among other things places all temporary test files in a
proper temporary directory rather than the source tree, which in turn
eliminates the tests/NEW and tests/DIFF temporary directories, which
also includes *.out.raw.stderr files; the .passed and .failed files have
been gone since commit b82970c in 2020.  This way, the tests/.gitignore
file no longer has a purpose, so remove it and the associated exemption
for TEST_DIST in Makefile.  Remove failure-outputs.txt from the
top-level .gitignore as well.  Prune "make distclean" as well.

Merge tests/*.tests into TESTrun: these are a part of the source tree
rather than volatile external data, so instead of implementing the
required run-time logistics just place the Perl data structures in the
only Perl script that uses them.  Convert "config_set" and
"config_unset" using "skip" and equivalent Perl expressions.

Since these changes rewrite most of TESTrun, reindent and reformat it to
match the code style of the new files.

Make all test names unique in TESTLIST.

PTP: Refine timestamp printing.

In ptp_print_timestamp_identity() and ptp_print_announce_msg() use
ptp_print_timestamp() to deduplicate code.  In ptp_print_timestamp()
change the format to SECONDS.NANOSECONDS and indicate an invalid
nanoseconds value.  Update the tests.  This resolves GH issue #1260.

Report invalid microseconds as "us", not "ms".

In timeval-operations.h for microseconds and nanoseconds define both the
maximum number of units per second and the string to use for reporting
an invalid value.  Use the new macros in ts_frac_print() and update a
test.

For consistency in print-arista.c instead of MAX_VALID_NS and
BOGUS_NS_STR use the macros from timeval-operations.h.

Update the GitHub issue template for the new workflow

Summary:
Create the .github/ISSUE_TEMPLATE directory
Rename the file ISSUE_TEMPLATE to .github/ISSUE_TEMPLATE/issue_template.md
Update this file with some keywords (name, about, ...)
Add a .github/ISSUE_TEMPLATE/config.yml file containing
"blank_issues_enabled: false".

[skip ci]

LISP: Add a length check

Verify we have at least the header length.

Print the protocol name with nd_print_protocol_caps() before any check.
Update the lisp_type[] structure accordingly.

Add a test file with an invalid length.

ESP: Delete a useless custom bounds check

The two GET_BE_U_4() perform the bounds checks.

Print the protocol name, with nd_print_protocol_caps(), before any
check.

Add a test file with a truncated ESP packet.

CHANGES: Add changes backported to 4.99

[skip ci]

ISO: avoid undefined behavior and integer overflow in the fletcher checksum calculation

The fletcher checksum calculation would sometimes left-shift
a negative number, which is an undefined operation.  Rework the
code to avoid this.

checksum.c:186:20: runtime error: left shift of negative value -36
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior checksum.c:186:20

Unlike some checksum routines that use the defined semantics of
2's-complement unsigned overflow to their advantage, this one
gets the wrong value if it is allowed to overflow, due to the
use of mod-255.

Convert c1 to uint64_t to avoid overflow.

checksum.c:163:16: runtime error: unsigned integer overflow: NNN + NNN
  cannot be represented in type 'unsigned int'

Use integers during subtraction to avoid implicit conversion to unsigned
when calculating both x and y

checksum.c:172:18: runtime error: unsigned integer overflow: NNN - NNN
  cannot be represented in type 'unsigned int'
checksum.c:172:9: runtime error: implicit conversion from type
  'unsigned int' of value NNN (32-bit, unsigned) to type 'int' changed
  the value to -NNN (32-bit, signed)
checksum.c:173:12: runtime error: unsigned integer overflow: NNN - NNN
  cannot be represented in type 'unsigned int'
checksum.c:173:9: runtime error: implicit conversion from type
  'unsigned int' of value NNN (32-bit, unsigned) to type 'int' changed
  the value to -NNN (32-bit, signed)

NFS: Delete dead code

Since  commit f2eaaa35725c6cb52e9e3947769fc6e7f2579dee in year 2000
the typedef struct my_fsid has a field 'char Opaque_Handle[2 * 32 + 1];'

Thus sizeof(*fsidp) is > 14.
Delete the else block as dead code and the two useless if.

This should remove two Visual Studio warnings:
parsenfsfh.c(318,25): warning C4127: conditional expression is constant
parsenfsfh.c(319,22): warning C4127: conditional expression is constant

BOOTP: Use an uint16_t variable to get the result of a GET_BE_U_2()

This will fix this Visual Studio warning:
print-bootp.c(1009,18): warning C4242: =: conversion from uint16_t to
uint8_t, possible loss of data

Fix comments. [skip ci]

Yes, the offending set of *BSDs (FreeBSD, DragonFly BSD, and
CupertinoBSD) all do define IPPROTO_PIGP.  And the #defines are
IPPROTO_xxx, not IP_PROTO_xxx.

Add macOS to the list of OSes that define IPPROTO_IGRP as 88. [skip ci]

Add comments about the IGRP/EIGRP protocol number and #define mess.

The current IANA protocol number assignments page lists 9 as "any
private interior gateway (used by Cisco for their IGRP)" and 88 as
"EIGRP" from Cisco.

Recent FreeBSD and DragonFly BSD <netinet/in.h> headers define
IP_PROTO_IGRP as 9 and IP_PROTO_IGRP as 88.  We define IP_PROTO_PIGP
("private interior gateway protocol") as 9 and IP_PROTO_EIGRP as 88;
those names* better match what the current protocol number assignments
say.

tcpdump: show error message for PCAP_ERROR_CAPTURE_NOTSUP.

If an attempt to open a capture device failed because capturing isn't
supported on that device, check whether an error message with a further
explanation was provided and, if so, report it.

CHANGES: Add changes backported to 4.99

[skip ci]

frag6: Add a bounds check in non-verbose mode

Define ND_LONGJMP_FROM_TCHECK.

Rename a test and its output, with "-v" in the names.
Add a test case (same pcap printed without "-v").

IP: Use ND_TTEST_LEN() instead of a custom bounds check

This avoids an invalid-pointer-pair AddressSanitizer error with options
-fsanitize=address -fsanitize=pointer-compare
and
ASAN_OPTIONS=detect_invalid_pointer_pairs=1

The error was:
    ip_printroute_asan                      : TEST FAILED[...]

reading from file ip_printroute_asan.pcap, link-type EN10MB (Ethernet),
  snapshot length 60
=================================================================
==ERROR: AddressSanitizer: invalid-pointer-pair: 0x60600000006a
  0x60600000005c
    #0 0x558ce1cddb0b in ip_print
      /home/cpe/soft-origin/tcpdump/my-tcpdump/print-ip.c:429:55

0x60600000006a is located 14 bytes after 60-byte region
  [0x606000000020,0x60600000005c)
allocated by thread T0 here:
    #0 0x558ce1c5390e in malloc
      (/home/cpe/soft-origin/tcpdump/my-tcpdump/tcpdump+0x1fd90e)
    #1 0x558ce1efbba3 in pcap_check_header
      /home/cpe/soft-origin/tcpdump/my-libpcap/sf-pcap.c:480:14

0x60600000005c is located 0 bytes after 60-byte region
  [0x606000000020,0x60600000005c)
allocated by thread T0 here:
    #0 0x558ce1c5390e in malloc
      (/home/cpe/soft-origin/tcpdump/my-tcpdump/tcpdump+0x1fd90e)
    #1 0x558ce1efbba3 in pcap_check_header
      /home/cpe/soft-origin/tcpdump/my-libpcap/sf-pcap.c:480:14

SUMMARY: AddressSanitizer: invalid-pointer-pair
  /home/cpe/soft-origin/tcpdump/my-tcpdump/print-ip.c:429:55 in ip_print

Point to the correct documentation for installation on Windows. [skip ci]

CHANGES: Add a change backported to 4.99

[skip ci]

Add sub-second packet timestamp checks for invalid micro/nano

Now prints e.g.:
    2  17:16:10.1000000 (invalid ms) IP [...]
    3  17:16:10.2147483648 (invalid ms) IP [...]
or
    2  17:16:10.1000000000 (invalid ns) IP [...]
    3  17:16:10.2147483648 (invalid ns) IP [...]

Add two test files.

tests: Fix some tests with invalid microsecond packet timestamps

If the timestamp for packets are given in seconds and microseconds,
a microsecond value > 999999 is invalid. Set it to 999999.

IKEv1: Fix a typo in a comment

[skip ci]