RADIUS: Fixup the previous commit.

Use GET_ macros and C99 uint types. Use nd_printn() to print RADIUS
strings, which are not NUL-terminated (see RFC 2865 page 25). Use
p_ntp_time() to print NTP timestamps. In print_attr_location_data() print
the Location field as hex and add a comment to explain why. In
print_basic_location_policy_rules() display any non-zero MBZ bits in the
Flags field. Make output format more consistent. Update the test case.

RADIUS: Add attributes from RFC5580. (GH #706)

Rename min() and max() to ND_MIN() and ND_MAX(). [skip ci]

As discussed on tcpdump-workers, it was a place for a potential clash
with non-macros.

Fixup some letter case in the man page. [skip ci]

Cirrus CI: Allow FreeBSD 13 task to fail.

Implement the workaround suggested in cirruslabs/cirrus-ci-docs#695.

[skip ci]

Cirrus CI: Align with Travis CI config more.

Define MAKEFLAGS, move comments to the end of respective lines, run
"make install" and "make releasetar".

[skip ci]

Remove an inexistant URL

BGP: Address a -Wformat-truncation= compiler warning

The warning was:
./print-bgp.c: In function 'bgp_vpn_rd_print':
./print-bgp.c:797:65: warning: '%u' directive output may be truncated
writing between 1 and 3 bytes into a region of size between 2 and 31
[-Wformat-truncation=]
  797 |         snprintf(pos, sizeof(rd) - (pos - rd), "%s:%u (%u.%u.%u.%u:%u)",
      |                                                                 ^~
./print-bgp.c:797:48: note: directive argument in the range [0, 255]
  797 |         snprintf(pos, sizeof(rd) - (pos - rd), "%s:%u (%u.%u.%u.%u:%u)",
      |                                                ^~~~~~~~~~~~~~~~~~~~~~~~
./print-bgp.c:797:48: note: directive argument in the range [0, 65535]
./print-bgp.c:797:9: note: 'snprintf' output between 15 and 50 bytes
into a destination of size 41
  797 |         snprintf(pos, sizeof(rd) - (pos - rd), "%s:%u (%u.%u.%u.%u:%u)",
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  798 |                     as_printf(ndo, astostr, sizeof(astostr), GET_BE_U_4(pptr + 2)),
      |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  799 |                     GET_BE_U_2(pptr + 6), GET_U_1(pptr + 2),
      |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  800 |                     GET_U_1(pptr + 3), GET_U_1(pptr + 4),
      |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
  801 |                     GET_U_1(pptr + 5), GET_BE_U_2(pptr + 6));
      |                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Rx: Make UDP ports 16-bit to compile (GH #868).

Gisle Vanem reported that GCC 7.1 for DJGPP sees u_int and uint32_t as
two different types, hence the forward declaration for rx_cache_find()
was different from the actual function. UDP port numbers are 16-bit and
udp_print() appropriately uses uint16_t for them. Use the same type in
the downstream Rx-specific code to make things simpler and consistent.

Fix a typo

IEEE 802.15.4: Use GET_U_1() to replace direct dereferences

This need adding 'netdissect_options *ndo' parameter to
ieee802_15_4_crc16() and ieee802_15_4_crc32().

DNS: Use GET_U_1() to replace a direct dereference

RESP: Use GET_U_1() to replace direct dereferences

Moreover:
Fix spaces

macsec: Update the outputs of two tests

This is a follow-up to 69a76fb67a36a19779142d07bb0cb31b7736a936.

macsec: Update ndo_protocol to lower case as in all other uses

Update the man page for "-d", "-i" and "-y".

Describe the changes in behaviour after commit 74e6075, which implements
the discussion made in GH #804.

[skip ci]

Fix spaces

Rename DLT_PPP_WITHDIRECTION to DLT_PPP_PPPD

In libpcap DLT_PPP_WITHDIRECTION was renamed to
DLT_LINUX_PPP_WITHDIRECTION and after to DLT_PPP_PPPD.

Rename other #defines to match.

SRv6: Add a test output without '-e'

This is a follow-up to commit 1103418a18d71c0fe448c0459a42ff1bc20bb0dc.

DHCP: Make option printing format more consistent.

rfc1048_print() used to print DHCP options in a few different formats
depending on the context (see GH issue #866):

* at the top of an RFC 1048 block:
  * known options: [string] name and [decimal] tag
  * unknown options: tag twice
* inside a Parameter Request (55) block:
  * known options: name
  * unknown options: tag

Change it to use the "Name (tag)" format (as in TCP, BGP, etc) in all
contexts. Update output for 8 tests.

DHCP: Remove a few options that are unassigned now.

Commit e89bfcd in December 2000 among other things had introduced
the following code points from the IANA "BOOTP Vendor Extensions and
DHCP Options" registry:

* TAG_AGENT_MASK (84)
* TAG_6OVER4 (96)
* TAG_IPX_COMPAT (110)
* TAG_FAILOVER (115)
* TAG_EXTENDED_REQUEST (126)
* TAG_EXTENDED_OPTION (127)

The current revision of the registry lists every code above as
"Removed/Unassigned" with a reference to RFC 3679 published in January
2004. Remove the code points and any associated code.

Add support for ethernet IP protocol

The SRv6 network programming defines a new IP protocol number [1].

The new protocol number is used in the Next Header field of an IPv6 header
or any extension header indicates that the payload is an Ethernet.

IANA has assigned 143 as the protocol number of ethernet [2].

The SRv6 packet has the following format:
IPv6 HDR + SRH + Ethernet HDR + IPv4/IPv6 packet

This patch allows TCPDUMP to dissect such SRv6 packets.

[1] https://tools.ietf.org/html/draft-ietf-spring-srv6-network-programming-16#section-9.1
[2] https://www.iana.org/assignments/protocol-numbers/protocol-numbers.xhtml

Signed-off-by: Ahmed Abdelsalam <[email protected]>

Broadcom LI: Pass correct caplen value to ether_print()

Apply the last step of the new way to update the link-layer header length

All the link-layer dissectors are now void functions.

All the functions were moved to the void_printers[] array.
Rename this array to printers[].
Remove the uint_printers[] array, now empty.
Remove the 'ndo_void_printer' flag field, now useless, from
netdissect_options.
Remove other transitional code.

Rework "Update more link-layer dissectors to void functions"

It's a follow-up to commit 81dbf4a0b05092760d0ff0cdd48c692e4769ba99.

There is no need to update ether_print(), ether_common_print() and
ether_switch_tag_print() to void functions: back to u_int functions.

There is also no need to add a flag parameter to ether_print(),
ether_common_print() and ether_switch_tag_print(): Remove it.

PFLOG: Update the link-layer dissector to a void function

Moreover:
Use GET_U_1() when needed.
Remove trailing "_if" from the protocol name.

AppleTalk: Update the link-layer dissector to a void function

Moreover:
Remove trailing "_if" from the protocol name.

CHDLC: Update the link-layer dissector to a void function

Moreover:
Remove trailing "_if" from the protocol name.

PPP: Update the link-layer dissectors to void functions

Moreover:
Remove trailing "_if" from some protocol names.
Update the outputs of two tests accordingly.

Remove the unused DLT_LANE8023 link-layer dissector

DLT_LANE8023 was never defined in libpcap.
It was, perhaps, defined in some SuSE libpcap update, but no evidence
of this.
It is not defined in OpenSuSE Leap 15.2 (information from Guy).

FR: Update the link-layer dissectors to void functions

Moreover:
Remove trailing "_if" from the protocol name.
Add two comments.

ATM: Update the link-layer dissector to a void function

Moreover:
Remove trailing "_if" from the protocol name.

IPFC: Update the link-layer dissector to a void function

Moreover:
Remove trailing "_if" from the protocol name.

Token Ring: Update the link-layer dissector to a void function

Moreover:
Remove trailing "_if" from the protocol name.

SLL: Update the link-layer dissectors to void functions

Moreover:
Fix two probably copy & paste errors (s/SLL_HDR_LEN/SLL2_HDR_LEN/).
Remove trailing "_if" from the protocol name.

FDDI: Update the link-layer dissector to a void function

Moreover:
Remove trailing "_if" from the protocol name.

CIP: Update the link-layer dissector to a void function

Moreover:
Remove trailing "_if" from the protocol name.

IEEE 802.15.4: Remove trailing "_if" from a protocol name

IEEE 802.11: Remove trailing "_if" from protocol names

IEEE 802.15.4: Update the link-layer dissectors to void functions

IEEE 802.11: Update the link-layer header length in two steps (PRISM)

This way, even if we longjmped out of the code because trucation,
the prism header length is subtracted for -x/-X print.

IEEE 802.11: Update the link-layer dissectors to void functions

Remove a trailing tab

PPPoE: Update the link-layer dissector to a void function

Moreover:
Add a length check.
Fix a return.
Use nd_print_trunc().
Remove trailing "_if" from the protocol name.
Style.

UDP: Harmonize some function calls

Some calls use '(const u_char *)(up+1)'.
Other calls use 'cp' (same value).
Update the calls to only use 'cp'.

(follow-up to ea0f25cc7765554e521bf7ea44d94f77500749f5)

Sync the Ethernet protocols database with the libpcap one

Commits in libpcap:
23a2d990d9bce2123e4f6c39bee6f61c9617a3b5
(Fix "unknown ether proto 'aarp'")
and
e902b3e97418c4b11caa07f71d0deae8a5e726de
(Remove undocumented and rather old "ether proto" protocols)

Use more BSD style references in the man page.

[skip ci]

Define FMAXINT only once and use it properly.

[skip ci]

Juniper: Minus the Ethernet hdr length for -x/-X print (JUNIPER_ETHER)

Juniper: Update the link-layer dissectors to void functions

Makefile.in: Remove files win32/prj/* from EXTRA_DIST

Files removed in commit dbe0eab45550fceaf268fbc740fc4b332ff66b39.

vsock: Update the link-layer dissector to a void function

Moreover:
Rename some variables/parameters from 'len' to 'caplen' because
they store the capture length.

ARCNET: Remove trailing "_if" from protocol names

IPoIB: Update the link-layer header length when needed

IPoIB: Update the link-layer dissector to a void function

Remove unused files win32/prj/*

The supported way to build tcpdump on Windows is with CMake.

Nobody in tcpdump-workers list says 'we need to keep them'.

Support for EAP Dump in RADIUS Messages

(pull request #864)

EAP: Add verbose output for the test case eapon1.pcap

autoconf: replace --with-system-libpcap with --disable-local-libpcap.

--with-system-libpcap didn't work - it didn't prevent searching for a
local libpcap - and had a misleading name.  Replace it with
--disable-local-libpcap.

This should address GitHub issue #823.

MPLS: Rework some tests

This should address a warning with some AppVeyor builds (VS 16 2019):
warning C4061: enumerator 'PT_UNKNOWN' in switch of enum
'mpls_packet_type' is not explicitly handled by a case label

Update more link-layer dissectors to void functions

brcm_tag_if_print()
brcm_tag_prepend_if_print()
dsa_if_print()
edsa_if_print()
ether_if_print()
netanalyzer_if_print()
netanalyzer_transparent_if_print()

Update ether_print(), ether_common_print() and ether_switch_tag_print()
to void functions.

Add a flag parameter to ether_print(), ether_common_print() and
ether_switch_tag_print() to increment the link-layer header length field
of the netdissect_options when needed.
The calls use TRUE when the return value of the funtions was used.
The calls with FALSE avoid increments when the calls are nested.

Moreover:
Remove trailing "_if" from some protocol names.

Rename a field of the netdissect_options structure

BRCMTAG: Remove a useless test

Because packet length (length) >= capture length (caplen), when
caplen >= BRCM_TAG_LEN, length >= caplen cannot be < BRCM_TAG_LEN.

(see the sanity checks in print.c, pretty_print_packet() function)

TCP: Remove an useless test

The length is tested in ldp_print().

Update the GitHub issue template (about attachments)

[skip ci]

RSVP: Make a constant unsigned

This change avoids:
print-rsvp.c:1205:40: runtime error: implicit conversion from type 'int'
of value -8 (32-bit, signed) to type 'unsigned int' changed the value
to 4294967288 (32-bit, unsigned)
    #0 0x848e20f in rsvp_obj_print ./print-rsvp.c:1205:40

IEEE 802.11: Fix signedness of the flags parameters of print_chaninfo()

uint32_t for flags, because there are calls with uint16_t and uint32_t.

This change avoids (for presentflags):
print-802_11.c:2795:41: runtime error: implicit conversion from type
'uint32_t' (aka 'unsigned int') of value 2147502191 (32-bit, unsigned)
to type 'int' changed the value to -2147465105 (32-bit, signed)
    #0 0x815e130 in print_radiotap_field ./print-802_11.c:2795:41

LLDP 8021 link aggregation TLV support

As per IEEE std 802.1-AB-2009, 802.3 link aggregation TLV has been
deprecated and link aggregation TLV has been added as an 802.1
TLV. Support for this is given.

man: Add an example for printing TCP with RST+ACK both set

Partially based on a Guy's comment in issue #860.

[skip ci]

IEEE 802.15.4: Fix printing the pending extended address list

Fix an index.
Fix a probably copy & paste error.

TESTrun: add definitions of true and false.

If we don't give them definitions, in some contents they are interpreted
as strings, and 'false', being a non-empty string, is interpreted as
being true.  See, for example:

https://stackoverflow.com/questions/6936194/is-there-a-built-in-true-false-boolean-value-in-perl

TESTrun: fix copy-and-pasteo.

(Matters only on Windows, and we're still not ready for the check target
to be used on Windows yet; for one thing, it requires that tcpdump be
built with the latest shiniest libpcap, to handle some capture files
that use post-1.9 LINKTYPE_ values, and we aren't yet set up to do
that.)

BGP: Fix a ND_TCHECK_LEN() use

The error was:
reading from file bgp-rt-prefix.pcap, link-type EN10MB (Ethernet), snapshot
length 262144
==22310==WARNING: MemorySanitizer: use-of-uninitialized-value
[...]

SUMMARY: MemorySanitizer: use-of-uninitialized-value print-bgp.c:1040:5
in decode_rt_routing_info

Moreover:
Simplify a pointer expression.

TESTrun: process the system return value differently on Windows and UN*X.

We can use the POSIX functions on UN*X, but not on Windows.  Use them on
UN*X, but extract the exit status manually on Windows.

Put in a pile of comments about the return status of commands, command
interpreters, and the Perl system function on UN*X and Windows.

If system simply fails to run at atll (returns -1), put a message into
the faked output file.

Get rid of some duplicate initializations while we're at it.

TESTrun: put white space between flags.

I guess that's not allowed, at least not by fc.

TESTrun: Windowsify one more path.

TESTrun: canonicalize the path of the standard error file.

DEC's decisions and the Multics developers' decisions affect us to this
day.  (Unix went with the Multics use of - to indicate options; DOS went
with the DEC use of / to indicate options, so they had to use \ as the
pathname component separator, and Windows followed suit, so here we
are.)

TESTrun: add and expand comments.

[skip ci]

TESTrun: put all redirection at the end of the command line.

That makes it a bit easier to read.

TESTrun: use the same code paths on Windows and UN*X.

There's no need to treat Windows specially in that code path.

TESTrun: always return a value from runtest().

Falling off the end isn't reported as an issue by the Perl interpreter
here, and doesn't appear to cause a reliable return value, so not all
failed tests are counted.

TESTrun: I missed one use of $printcmd; use showfile() for it.

Remove some trailing spaces/tabs

Moreover:
Remove some spaces before tabs.
Remove extra blank line.

Add two test files for the unsupported link-layer protocols printer

This is a follow-up to commit 25687e1a312a3bb63a6e8d7f0af0210a615e24a8.

These files come from the Wireshark menagerie:
0031419.pcap (link type 160) and 5632-Raw_D-BUS.cap (first two packets).

TESTrun: use fc, rather than diff, on Windows.

diff doesn't come standard with Windows; fc does.

TESTrun: clean up printing of files.

Many Windows commands only accept paths using backslashes, because
slashes are option separators.

Add a showfile() function that takes a pathname as an argument and:

on Windows, converts the pathname to canonical form - which
means any slashes will be converted to backslashes - and run
"type" on it;

on UN*X, run "cat" on it.

Convert

cat foo | diff bar -

to

diff bar foo

to avoid using cat at all.  (Note also that the closest built-in Windows
equivalent of diff, fc, does *not* support reading the standard input as
one of the files to compare, so it also will avoid that when we change
those to use fc on Windows.)

The one remaining use of cat is in a command with pathnames, so use a
type command, with backslash-separated paths, on Windows, and cat, with
slash-separated paths, on UN*X.

We might just want to do that directly in Perl; add a comment about
that.

Use \ rather than / as a pathname separator in Windows.

TESTrun: fix some problems when run on Windows.

Run tcpdump, not windump - we build it as tcpdump, not windump, and
there's not really a reason to have a separate windump program.

For now, at least, assume a Visual Studio debug build, so it's in the
Debug subdirectory.

Use that path for *all* calls to tcpdump.

Use "type" rather than "cat" to print files on Windows.

It still needs to be changed to, for example, not use sed.

DNS: Do the 'over TCP' processing in the printer

Add the parameter 'over_tcp'.
Move the shift by 2 bytes from the TCP printer to the DNS printer.
Move adding a prepended space from the TCP printer to the DNS printer.
Add a length check.
Add some comments about 'over_tcp' and 'is_mdns' call values.

DNS: Add two test files (over TCP and over UDP)

TCP: Update style

[skip ci]

UDP: Harmonize some function calls

Some calls use '(const u_char *)(up + 1)' or '(const void *)(up + 1)'.
Other calls use 'cp' (same value).
Update the calls to only use 'cp'.

Moreover:
Update some other casted 'up + 1' uses to 'cp'.
Update some pointers to packet bytes from 'void *' to 'u_char *'.

appveyor: add VS 2019 tests.

VS 2019 is different from earlier versions of VS in the way it handles
the target platform; handle that in the fashion that much bashing on
libpcap's .appveyor.yml got to work.

PTP: Declare 'static' some variables

Fix warnings given by '-Wmissing-variable-declarations':

./print-ptp.c:262:13: warning: no previous extern declaration for
non-static variable 'p_porigin_ts' [-Wmissing-variable-declarations]
const char *p_porigin_ts = "preciseOriginTimeStamp";
            ^
./print-ptp.c:262:7: note: declare 'static' if the variable is not
intended to be used outside of this translation unit
const char *p_porigin_ts = "preciseOriginTimeStamp";
      ^
./print-ptp.c:263:13: warning: no previous extern declaration for
non-static variable 'p_origin_ts' [-Wmissing-variable-declarations]
const char *p_origin_ts = "originTimeStamp";
            ^
./print-ptp.c:263:7: note: declare 'static' if the variable is not
intended to be used outside of this translation unit
const char *p_origin_ts = "originTimeStamp";
      ^
./print-ptp.c:264:13: warning: no previous extern declaration for
non-static variable 'p_recv_ts' [-Wmissing-variable-declarations]
const char *p_recv_ts = "receiveTimeStamp";
            ^
./print-ptp.c:264:7: note: declare 'static' if the variable is not
intended to be used outside of this translation unit
const char *p_recv_ts = "receiveTimeStamp";

Moreover:
Put a function definition name at the beginning of the line.

Compile with '-Wmissing-variable-declarations' if it's available

(In devel mode)

macsec: include <config.h>, not "config.h".

That fixes some cases where having an in-tree build and an out-of-tree
build (e.g., an in-tree autotools build and an out-of-tree CMake build)
gets the wrong config.h file.

macsec, ether: clean up printing of encrypted and changed packets.

If the packet is encrypted or changed, so that we just print the payload
as raw data, and we're not running with -e, print the MAC addresses (if
any) and an indication that it's an 802.11AE packet, followed by the
MACsec header.

ether: for MACsec packets, print the MACsec ethertype if E or C set.

If -e isn't specified, and a MACsec packet is encrypted or changed, so
that we don't print the payload, at least report that it's a MACsec
packet.

Merge pull request #603 from qsn/macsec

Add support for MACsec (IEEE 802.1AE-2006)

macsec: update the test files.

Add the frame number, as we now run the tests with -#.

Fix some output to match the current state of the code.