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

(cherry picked from commit b0f1d71891088c708ee98f94d0e26140b0eb3c68)

CHANGES: Move change(s) backported to 4.99

(cherry picked from commit 2db7c32de1a1c3e7f940e8275a285bbf44c658ad)

[skip ci]

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.

(backported from commit 9beea81f673bd6267a536d9132ca06a3d20a0235)

Fix a typo in a comment

(cherry picked from commit 232b861896457ec92c8919ce3ac0d84084c76501)

[skip ci]

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

(backported from commit 9b3eb17a6942b746b6d1ed7e763e8d22632b6c46)

CHANGES: Add change(s) backported to 4.99

(cherry picked from commit 1e5d31794035611f059bc5d118d9e68d400830b9)

[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.

(backported from commit 43064ae5a5e9f882b62d7270bfff675f742702e4)

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

(cherry picked from commit cd2208037c766bb724ad8fd519c1b72d90f65bec)

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

(cherry picked from commit c602726f3a91f2c344bde10c6e2f3c6149a15adc)

CHANGES: Add change(s) backported to 4.99

(cherry picked from commit 9676f2ddf3ba55d7032ab6c1ede971031e59122e)

[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.

(cherry picked from commit 6ee6645b108a5285689672aa3257c207e6d769fa)

CHANGES: Add change(s) backported to 4.99

(cherry picked from commit b87e23ef01053a65fc310111631a6ddbfdc38bdf)

[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)

(backported from commit 8fac227c17aa5f1c09f35626738cd68de03843b8)

Use __func__ in some error messages in compress_savefile()

(cherry picked from commit 21cf6d8a4e3c82d83eef820801e69b0328acc431)

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);

(cherry picked from commit 56a7c968aa852a689d891c5bbd648c009c24fa2e)

CHANGES: Add change(s) backported to 4.99

(cherry picked from commit c6aa6c2229cab8c977d68705a0ae7824ca515a94)

[skip ci]

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.

(cherry picked from commit ccee9549f927eb13f869ee0db10c53c445191b8f)

CHANGES: Add change(s) backported to 4.99

(cherry picked from commit adbf6336e7c078b896f7aaed1b7d89e263fa18cf)

[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.

(backported from commit e06a30339fb3f4eb9a6c5dc503dcd8fbaa5034c6)

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

(cherry picked from commit e4e3caaca6aa2c0b0d5154b7c9aecba41059d744)

[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).

(backported from commit 801a7c1d641766e5117715d28e2aa8639becadd4)

Use C99 macros to define 64-bit constants and maximum 64-bit values.

We require C99 support, so don't roll our own versions.

(cherry picked from commit dcb1b03409b91aa3c68156817b953781f35bc3a2)

Improve the handling of size suffixes for -C.

Support upper-case suffixes (K/M/G) as well as lower-case suffixes.

Make sure nothing *follows* the suffix.

We don't need to check for the suffix before parsing the number; the
parsing routines stop if they see a non-digit character.

(cherry picked from commit 5ff873f0d277e10ad9deffa92b00b38cbfef4914)

optional unit suffix on -C

(cherry picked from commit 81e2588d2ba6290cc189923287f1c042275a0bc5)

ZEP: use the exissting NTP time formatting code.

The strftime() format for that case is slightly different, so add a new
p_ntp_time_fmt() routine that takes a strftime() format, and use that in
ZEP.

That way, we can benefit from any handling of times not fitting in a
time_t being done by the NTP time printing code.

(cherry picked from commit e57fe28eb2a42698ca948488ed2dcb8fad9d35e5)

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.

(cherry picked from commit 09512cfc01958a37f67951bf7b6e00d52ed1e12e)

Add some const qualifiers

Moreover:
Fix indentation.

(cherry picked from commit 857656d29151f99a1c5de50a0c7574c3cf786241)

Spell "indent" where it belongs.

(cherry picked from commit 032d569e681df53b6e0b03fc20bdb1970faafa20)

Refine print_unknown_data().

Use __func__ and ND_MIN() for simplicity.

(cherry picked from commit 2e6a03c3f1c64ee0455b191cc58b78ea984aa1cb)

Text protocols: Use u_int as return type for fetch_token()

It was 'int' but
1) fetch_token() return 0 or idx which is an 'u_int'.
2) fetch_token() is called with:
   idx = fetch_token(...);
   with idx an 'u_int'.

(cherry picked from commit a6b1a1a514c89a6a8cbf8e9cdcfd7e86c3de108e)

CHANGES: Add change(s) backported to 4.99

(cherry picked from commit f2c28cc36ad57e60ef8fc844c7affeb5ce093ecd)

[skip ci]

Fix incompatible pointer types with time functions calls on Windows

On Windows in a struct timestamp, tv_sec is a long not a 64-bit time_t.

The problem shows:
listening on \Device\NPF_Loopback, link-type NULL (BSD loopback),
snapshot length 262144 bytes
    1  [localtime() or gmtime() couldn't convert the date and time].052255
       IP 10.0.0.10 > 224.0.0.251: igmp v2 report 224.0.0.251
    2  [localtime() or gmtime() couldn't convert the date and time].792000
       IP 10.0.0.10.138 > 10.0.0.255.138: NBT UDP PACKET(138)

The warnings with clang-cl were:
util-print.c(253,18): warning: incompatible pointer types passing
  'const long *' to parameter of type 'const time_t *'
      (aka 'const long long *') [-Wincompatible-pointer-types]
  253 |                 tm = localtime(&tv->tv_sec);
      |                                ^~~~~~~~~~~

util-print.c(255,15): warning: incompatible pointer types passing
  'const long *' to parameter of type 'const time_t *'
      (aka 'const long long *') [-Wincompatible-pointer-types]
  255 |                 tm = gmtime(&tv->tv_sec);
      |                             ^~~~~~~~~~~

(cherry picked from commit dab871e8be7a30e511487ef0a5d401ab5657d314)

LSP ping: Use 'TimeStamp Sent' and 'TimeStamp Received' from the RFC

s/Sender Timestamp/TimeStamp Sent/
s/Receiver Timestamp/TimeStamp Received/

Fix the indentation in the output.

(cherry picked from commit 10b47a83006ea1ba2ce1fa507d05b523dd7256c6)

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".)

(cherry picked from commit 3026b252ffa687822d637c6b9e77b52d33b8f9d3)

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.

(cherry picked from commit a4508ee9abc27eb8e08191a206fb57edd8996273)

DHCP: Use more nd_print_invalid()

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

Update some error messages.

(cherry picked from commit 60237c7f7e9b0ed63d80ebeb281e85d102f345c3)

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

(cherry picked from commit e80cc07aa0f7523c1aa94cea5f2c38c70c06b33f)

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 an error message if the length is not 1.

(backported from commit 711571721fd3f13f1082e4e803c4066e1a51c540)

CHANGES: Add changes backported to 4.99

(cherry picked from commit 95c9f8f25d3e2ce85486524cd385fd6ef0093bcf)

[skip ci]

DHCP: Fix typos in some error messages

(cherry picked from commit a691f1a961ee78b59bcd06cf4f4abf9d2120273c)

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

(cherry picked from commit 52f6a29fa43417b129280b9f6b7d59461434acdf)

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.

(backported from commit 7cd08fe7c4a07df9028084612f81f90bd1b8a412)

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

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

(cherry picked from commit 7328aad0c945e4aac6ee1e79b596f616474860c4)

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)

(cherry picked from commit 575575a8a8a86a271d38ec7ea467e05eae832545)

[skip ci]

CHANGES: Add a change backported to 4.99

(cherry picked from commit 2f339a70dc2c4c17385bad97467be526aad4f7f5)

[skip ci]

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

(backported from commit 2d2951af231e06e36cdaead4ab8fdcde847da85f)

Cirrus CI: Set LANG in coverity_task same as in linux_task

To avoid these warnings:

perl: warning: Setting locale failed.
perl: warning: Please check that your locale settings:
        LANGUAGE = (unset),
        LC_ALL = (unset),
        LANG = "en_US.UTF-8"
    are supported and installed on your system.
perl: warning: Falling back to the standard locale ("C").
(cherry picked from commit c226248a44756cc8899564498405f1adcdc69826)

Cirrus CI: Bump Ubuntu up to 24.04. [skip appveyor]

Also lose stale MAKEFLAGS after commit 778e4bf.

(cherry picked from commit 9ef0559cc9aa935c1d55510775387947d63430ac)

Cirrus CI: Bump FreeBSD up to 13.4. [skip appveyor]

(cherry picked from commit 063b2e704a249c50f02766f140dcb11e4e525630)

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

Add two test files.

(cherry picked from commit 62078833231c635fcf296cdadb9fe0477a2d2acc)

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

For Node Information Queries Qtype "Node Addresses".

Add two test files.

(cherry picked from commit 806745e10e5ad2c963792a481c88fe2ee039bb00)

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.

(cherry picked from commit 805c9491c3712a5dc5f7af30a5da5f52c1194d36)

ICMPv6: Delete some remains from drafts for Node Information Queries

These drafts date from 1999-2002.

(cherry picked from commit efce7b624c234bf60eee169d70ed1d1cecc18bcc)

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.

(cherry picked from commit 901b8017ac7f9362fdbb83d2407677448fdc2d98)

ICMPv6: Delete some duplicate macros

Keep macros with names similar to IANA names.

(cherry picked from commit 434d67af40e92abafb32b36d7dcfbc4b1d8413bd)

CHANGES: Add changes backported to 4.99

(cherry picked from commit 6cd500c8dafe5a9bbf3e56312493709e8b118726)

[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)

(backported from commit c5b54bfbd68b03f7997feaa277db30d399975a4d)

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

(cherry picked from commit 431bdb5c365360d61968e7fa02322825b1da4208)

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

(cherry picked from commit 805fd42b376aea1da2ec4314c13337d45680bbfb)

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.

(cherry picked from commit 4b7d5a255b22717afb6052645343d59669811fd6)

CHANGES: Add changes backported to 4.99

(cherry picked from commit 9781ab8f8cf7a1055eb60b8c4d0c9abe066201eb)

[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").

(cherry picked from commit 67deae111e0d6c4df9dc4fb6a35213286815611f)

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

(cherry picked from commit 873b872c8e07c1565080c6ab09da6ee92b9909a6)

CHANGES: Add a change backported to 4.99

(cherry picked from commit 6549da061ce94b5f2c6e2902c79b84a16ee7971d)

[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.

(cherry picked from commit f08d60d42d60adce6e5b374882fc2035972730f4)

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.

(backported from commit 7e68ce3fa8ab1e7edd71c5653b3ca7908a35a14f)

IKEv1: Fix a typo in a comment

(cherry picked from commit d45c1a87ca9c70696ae7ba195b7e78973ba99cf4)

[skip ci]

CHANGES: switch to sentence case for some pflog entries. [skip ci]

tests: Remove debugging leftovers from TESTrun.

(cherry picked from commit 61861f30e082d269fe698ab9dfe056ff4ed2a19e)

CHANGES: changes backported to 4.99. [skip ci]

(cherry picked from commit 70dcdd6417c0fa3fd6893e1902d1717ae9a4e6d4)

Remove unused missing/snprintf.c. [skip ci]

The file could be compiled on demand using "make snprintf.o", although
with many warnings, but tcpdump build would never use the file
regardless of the variety of the build.

Some of the code there has been commented out since commit 45aef49 in
2001, and the rest became unused after commit 1ed63b5 in 2019.

(cherry picked from commit 0d397bff479c600af5a602ee56cdefaa2d84e9ce)

autotools, CMake: fix issues with snprintf test and sanitizers.

Avoid trying to cast negative values to unsigned types, or doing shifts
of signed types, in order not to have the test program fail if we're
building with undefined-behavior sanitizers enabled.

See the-tcpdump-group/libpcap#1396 for the equivalent libpcap issue.

(cherry picked from commit b7bd627c96f7a70446e4837ca7d5acd6d8122ce2)

CMake: check whether check_c_source_runs() works.

That's the simplest way to check whether we can use
check_c_source_runs() to test for a suitable snprintf; it's easier than
trying to find out the target instruction set architecture and comparing
it with the host instruction set architecture, as CMake doesn't provide
any mechanism to provide the target instruction set architecture, on all
platforms, in a form that can be compared with the host instruction set
architecture, and even if the target is different, we may be able to run
code for that instruction set architecture if, for example, it's a
32-bit version of the instruction set architecture on which the build is
being done, or if there's a binary emulator.

(cherry picked from commit 7fe2a0ec262c2c1e7db829b4e13b70c64b0f1da9)

CMake: Fix the comment about versions. [skip ci]

(cherry picked from commit 40b8db437e559753032d3493b3e69e13603ea3fe)

Fix a typo in a comment

(cherry picked from commit 49fa8ff39769041ed5b659724f2f74810a5c4c03)

cmake: treat building for another ISA as a cross-compile.

CMake appears to have the notion that a build is only a cross-compile if
the targt *operating system* is different.  This is an incorrect notion,
as even if the target is the *same* OS but a different instruction set,
you may not be able to do tests that involve compiling and running a
program.

Check whether CMAKE_GENERATOR_PLATFORM is set and has a value different
from that of CMAKE_HOST_SYSTEM_PROCESSOR and, if that's the case, set
CMAKE_CROSSCOMPILING to TRUE.

This comes from libpcap, where the equivalent change fixed issue
the-tcpdump-group/libpcap#1352.

(A different strategy may be necessary for cross-builds with UNIX
toolchains.)

(cherry picked from commit 2dd45475f02529c658aeb5a691dfb0c78e129aa1)

CMake: Skip snprintf(3) tests when cross-compiling.

Same as in libpcap.

(cherry picked from commit 2ec7e78bacc1d211af8765af4b4675eb6f1532ef)

Simplify the checks for snprintf().

Do not check that the function is just available: first, it is in C99,
so the check almost certainly is a waste of time; second, the source
requires the function unconditionally; third, the subsequent "snprintf()
is suitable" check implies the "snprintf() is available" part anyway.

(cherry picked from commit e1bcb7941d916285e5e502ff84dff5a15ec0b498)

CI: install coreutils with Homebrew on macOS. [skip appveyor]

We need it because we do an install of libpcap, including "make check",
and "make check" requires Homebrew's coreutils in order to get the
timeout command.

(cherry picked from commit 673a2fedf68a562ccf4612b467f373168b85552b)

CHANGES: Add a change backported to 4.99

(cherry picked from commit c50f3934dacc4bd3fe16a900d4c63c05acb50ce3)

[skip ci]

Warn that options -A, -x[x] and -X[X] are mutually exclusive

(cherry picked from commit 18b9876f6211ba416a7a1755e4252c720d7c2b40)

[skip ci]

man: Add details on the relationships between -A, -X[X] and -x[x] options

(cherry picked from commit 98b96cb74c3841e6e4eabf619cf682b4945468ad)

[skip ci]

LISP: Add three test cases (existing pcap printed without "-v")

Rename three test and three test outputs, with "-v" in the names.

(cherry picked from commit ec5170b2658af4b5f9bfa8a85ed92ce20ac3594a)

CHANGES: add changes backported to 4.99. [skip ci]

(cherry picked from commit f60ddc5b3e6efd44dcdf60ee02a8c57bfd132c17)

CMake: fix to support CMake 3.31.

Expand the "still support old versions of CMake, for the benefit of
people with older versions, but avoid deprecation warnings/errors"
section to handle the deprecation of pre-3.10 versions by 3.31.

(cherry picked from commit 199bb821ba57097b8629a0d992779483b424e7a4)

ipv6: set the snapshot length for the payload correctly.

Cut off the snapshot length to the end of the IP payload or the end of
the data in which it's contained, whichever comes first.

(cherry picked from commit 6bc9e72adcb9acd46720b9e8f0e1d41bbde1bdb9)

Cirrus CI: Update the macOS task image

Based on https://cirrus-ci.org/guide/macOS/.

This will avoid the warning:
Only ghcr.io/cirruslabs/macos-runner:sonoma is allowed. Automatically
upgraded.

(cherry picked from commit 3b11c29db440d55f7720bef9f7faff562a7e1f39)

CHANGES: Add two changes backported to 4.99

(cherry picked from commit c0677574af4bb64f3c16d3c14d19c73db95a2aa4)

[skip ci]

ptp: Add test for management packets

Test setup:
ip link add link vethA type veth peer name vethB
ip link set dev vethA up
ip link set dev vethB up
ptp4l -2 -P -S -i vethA &
tcpdump -i vethA 'ether[14] = 0xD' -w ptp_management.pcap &

Tcpdump filtering on only management packets (0xD) to skip capturing
other traffic that is already covered by other tests.

Sending management requests (which ptp4l responds to):
pmc -b 3 -2 -i vethB 'get current_data_set'
pmc -b 3 -2 -i vethB 'get port_data_set'
pmc -b 3 -2 -i vethB 'get default_data_set'
pmc -b 3 -2 -i vethB 'get parent_data_set'
pmc -b 3 -2 -i vethB 'get clock_description'

Sidenote: the management packets contains a TLV at the end and I
included 5 of them (out of the total ~50 listed in the
standard). Tcpdump currently does not print the TLV.

Signed-off-by: Casper Andersson <[email protected]>
(cherry picked from commit c98b4d0b3aa0a66f413ef3d86d726421a46fac5a)

ptp: Fix management packet fields

`bp` was modified inside the function but the change was not
reflected back outside, resulting in the fields afterwards accessing
the wrong part of the packet.

Signed-off-by: Casper Andersson <[email protected]>
(cherry picked from commit 487405e871739842bc46caaba27f93419f2722ed)

PTP: Remove spaces before colons in output

Update four test outputs accordingly.

(cherry picked from commit dd2c53b069d92bc8e36be5df10642010fc6f34d9)

CHANGES: Add a change backported to 4.99

(cherry picked from commit 40c29b66d7473224e7f02ba6aa107c9610879cb0)

[skip ci]

TCP: Fix Reset segment processing

When a received RST segment contains data, this is diagnostic data that
explains the cause of the RST.

So we have to stop processing, even if we don't print the data (no "-v").

Rename two test outputs, with "-v" in the names.
Add two test cases (existing pcap printed without "-v").

(cherry picked from commit 7f9a48502c3003fd80e0be79d80f572bf667216a)

TCP: Add a test with data in the RST segment, truncated example

(backported from commit d4fe5e32fdd85c6a4f623c05f40795f5fb7b8f07)

TCP: Add a test with data in the RST segment

(backported from commit 383670e8f3f5b51d1017d8c1a678aa7eaf1f2b4d)

IPv6: Fix inconsistencies in the printing of some option names

1) Hop-by-Hop Options Header

$ ./tcpdump --skip 1 -c1 -tnv -r tests/icmpv6.pcap
Before:
IP6 (hlim 1, next-header Options (0), payload length 36)
  fe80::215:17ff:fecc:e546 > ff02::16: HBH (rtalert: 0x0000) (padn)
  [icmp6 sum ok] ICMP6, [...]

There is "next-header Options" but the next header is printed "HBH".

After:
IP6 (hlim 1, next-header HBH (0), payload length 36)
  fe80::215:17ff:fecc:e546 > ff02::16: HBH (rtalert: 0x0000) (padn)
  [icmp6 sum ok] ICMP6, [...]

2) Destination Options Header

$ ./tcpdump  -tnv -r tests/erspan-type-iii-pb-1.pcap
Before:
IP6 (class 0x30, flowlabel 0x00001, hlim 1, next-header unknown (60),
  payload length 288) 4120:7467:1700:4200:143:100:7f01:400e >
  4591:bfd7:cd87:d7:68:38:101:e800: DSTOPT (padn) [...]

There is "next-header unknown" but next header is printed "DSTOPT".

After:
IP6 (class 0x30, flowlabel 0x00001, hlim 1, next-header DSTOPT (60),
  payload length 288) 4120:7467:1700:4200:143:100:7f01:400e >
  4591:bfd7:cd87:d7:68:38:101:e800: DSTOPT (padn) [...]

Moreover:
Update a ndo_protocol (s/hbhopt/hbh/).
Use nd_print_protocol_caps().

Update some test outputs accordingly.

(backported from commit 803b7028e04b757c5351a300a905e326c7d5a200)

CHANGES: add a change backported to 4.99. [skip ci]

(cherry picked from commit 32e1896329e45bc22afaa60301a28e6341b7fccd)

ip: set the snapshot length for the payload correctly.

Cut off the snapshot length to the end of the IP payload or the end of
the data in which it's contained, whichever comes first.

(cherry picked from commit a4221536d52271098acd24aad11fc6657afa76b8)

Don't hardwire the Windows system directory path.

Instead, fetch it with GetSystemDirectoryW() and append "\Npcap" to it.

(cherry picked from commit d9355ee3f173c08e971096dfd95fa1d63f6ebaf6)

Appveyor: Run "git show --oneline -s"

It is used to identify the git HEAD.

(cherry picked from commit fb3da6112098f6752149363c53732212a7ead1de)

[skip ci]