Makefile.in: don't remove configure and config.h.in in make distclean.

In the GNU makefile standards, "make distclean" removes everything
that's not part of the source distribution; this does *not* include the
configure script or the config.h.in file, as those are in the release
tarball, so that building from a source tarball doesn't require having
autoconf.  This allows somebody building from the source tarball to do a
"make distclean" and then reconfigure.

Do, however, remove any release tarball in make distclean, by having
distclean depend on clean.

tcpdump: plug a memory leak.

If the -V flag is used, and not all files in the -V file have the same
link-layer type, when the filter is recompiled for a new link-layer
type, the old filter program is leaked.  Free the old filter before
compiling the new filter.

Have a common routine for converting dates and times to strings.

Have a routine that takes a buffer, a strftime format, and a struct tm *
as arguments, and:

* checks whether the struct tm * is null and, if so, returns a string
indicating that the date and time couldn't be converted;

* otherwise, passes it to strftime(), along with the buffer and the
format argument and, if strftime() returns 0, meaning the string didn't
fit into the buffer and thus that the buffer's contents are undefined,
returns a string indicating that the date and time didn't fit into the
buffer;

* otherwise, returns a pointer to the buffer.

Call that routine instead of directly calling strftime() in printers;
that prevents printing a buffer with undefined data if the buffer isn't
big enough for the string.

Also, when generating file names using an strftime format, check the
return value of strftime() to make sure the buffer didn't overflow.

Cirrus CI: Switch FreeBSD from 12.3 to 12.4. [skip appveyor]

(same as in tcpslice and libpcap)

autoconf: Lose AC_LBL_CHECK_64BIT_FORMAT. [skip ci]

This macro has been unused since commit 14c81f0 in 2017.

autoconf: Add autogen.sh, remove configure and config.h.in

Put autoconf-generated files in the release tarball.

The minimum required version of autoconf is currently 2.69.

If version 2.69 or later is already installed and there is no
autoconf default, it may be necessary to set the AUTORECONF
environment variable to enable the one to use, like:
AUTORECONF=autoreconf-2.69 ./autogen.sh
or
AUTORECONF=autoreconf-2.71 ./autogen.sh

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.

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

configure, CMakeLists.txt: don't check for snprintf().

It's specified by the C90 standard (and, as I remember, by the C89
standard, although I no longer have my paper copy); no need to worry
about ancient environments that lack it, and we have some cases where we
call it in code not protected by #ifdef HAVE_STRFTIME/#endif and haven't
seen any reports of problems.

CHANGES: Add a change backported to 4.99

[skip ci]

smbutil.c: Fix the strftime buffer size

tm_year is an int, thus 'yyyy' for '%Y' is not enough.

This avoids a stack buffer overflow in some cases.

configure: Modernize AC_CONFIG_HEADER. [skip appveyor]

This works with Autoconf 2.69 and squelches another warning from
Autoconf 2.71.  Lose an empty optional arument to AC_CONFIG_COMMANDS
while at it.

configure: Apply autoupdate 2.69. [skip appveyor]

Among other things this squelches many warnings from Autoconf 2.71.

Lose LBL_LIBS and various pfopen() entourage.

tcpdump does not call pfopen().

build_common: fix missing ldd on Haiku

CHANGES: add a commit that will be backported to 4.99. [skip ci]

configure: boost the minimum autoconf version to 2.69.

We get an error with 2.64; jump to 2.69.

Reduce aclocal.m4 a little bit more. [skip ci]

Remove a copyright boilerplate that stood for IPv6 macros added via
commit c9d84d1 in 1999 and removed via commit 11f73ad in 2015.  Remove
AC_LBL_SSLEAY, which is unused and obsolete.  This change does not
result in any changes in any other files.

Remove some unused declarations from aclocal.m4. [skip ci]

AC_LBL_FIXINCLUDES, AC_LBL_UNION_WAIT and AC_LBL_HAVE_RUN_PATH have no
purpose in this source tree.  AC_LBL_SHLIBS_INIT does not exist.  This
change does not introduce meaningful changes in any other files.

Spell INSTALL.md in Autoconf messages. [skip ci]

CHANGES: Add a change backported to 4.99

[skip ci]

LSP ping: Fix "Unused value" warnings from Coverity

This change should fix the following Coverity Scan issues:

CID 1397795 (#1 of 1): Unused value (UNUSED_VALUE)
assigned_pointer: Assigning value from tlv_tptr + 4UL to tlv_tptr here,
but that stored value is overwritten before it can be used.

CID 1397796 (#1 of 1): Unused value (UNUSED_VALUE)
assigned_value: Assigning value from tlv_tlen - 4UL to tlv_tlen here,
but that stored value is overwritten before it can be used.

Fixup a comment in CMakeLists.txt. [skip ci]

Remove init_crc10_table() and the entourage.

As Guy Harris points out in bug report GH #1022, the function has been a
busy no-op since commit e6c39e6 in 2010.  While at it, fixup the Python
code to work on Python 3:

    for i in range(len(crc_table)/8):
TypeError: 'float' object cannot be interpreted as an integer

CMake: search for gethostbyaddr() in libnetwork

autoconf: search for gethostbyaddr() in libnetwork

This is needed for building on Haiku.

configure: check for gethostbyaddr(), not gethostbyname().

We use gethostbyaddr() to translate IP addresses in packets to names; we
don't use gethostbyname() to translate names to IP addresses (libpcap
does the latter when compiling filters, so we leave that up to it, and
nothing's left for us to do).

This probably has the same effect as checking for gethostbyname(), but
it makes it a bit clearer why we'd check for it.

CHANGES: Do section headers for 4.99.3, 4.99.4 and main

[skip ci]

Makefile.in: Get rid of a remain of gnuc.h

gnuc.h was removed in commit fcf0144ed224f770c906866a42a9ffa1d707c429.

[skip ci]

CHANGES: add another change backported to the 4.99 branch. [skip ci]

CMake: set man page section numbers.

We use both MAN_FILE_FORMATS and MAN_MISC_INFO in tcpdump.1; set them

Also set _SUN on AIX, as we do with the configure script.

Remove version.c from .gitignore. [skip ci]

The file does not exist since commit 4943adf.

CHANGES: Add an entry for the previous commit. [skip ci]

arista: update test pcap with hwinfo values

Fix some typos in comments

[skip ci]

CHANGES: Add a change backported to 4.99

[skip ci]

CHANGES: Add some changes backported to 4.99

CHANGES: Add a change backported to 4.99

[skip ci]

PTP: Print un-allocated values for the message field as "Reserved"

instead of "none".

PTP: use the proper values for the control field

CHANGES: Move "Source code" section before "Building and testing" one

[skip ci]

smbutil.c: Replace obsolete function call (asctime)

CMake: Fix a typo in a comment. [skip ci]

CHANGES: Move some changes backported to 4.99

CHANGES: Update after the 4.99.2 release

Add the 4.99.3 section.

CHANGES: Refine the 4.99.2 section. [skip ci]

List user-visible changes that have been made but did not appear in the
change log.  Remove entries that are duplicate (including EIGRP from
4.99.1) or have been superseded by later changes (make releasetar).

Cirrus CI: Run the "make whitespacecheck" command in the Linux task

There is no need to add this command in all tasks.

Makefile.in: Add the whitespacecheck target

Display errors when there are trailing space(s)/tab(s).

[skip ci]

TESTLIST: Update some comments (.sh -> .tests)

Lose an unused variable in print-snmp.c.

./print-snmp.c:1200:17: warning: variable 'ind' set but not used
  [-Wunused-but-set-variable]

clang version 15.0.1
Target: aarch64-portbld-freebsd13.1

PTP: Replace some ' ,' with ', ' (style)

[skip ci]

PTP: Remove an unused macro

Moreover:
Remove extra blank lines.

[skip ci]

Fix a typo

[skip ci]

Makefile.in: Add "make -s install" in the releasecheck target

Cirrus CI: Switch macOS to AArch64. [skip appveyor]

Same as in tcpslice and libpcap.

CHANGES: Add a main-only change

[skip ci]

bgp: Add support for BGP Role capabilty and OTC attribute

Signed-off-by: Donatas Abraitis <[email protected]>

CHANGES: Add a main-only change

CI: Introduce and use TCPDUMP_CMAKE_TAINTED

The new environment variable provides means to declare all or some libpcap
cmake run of a specific environment (CI or a working copy) as tainted.
Add explicit conditions to exempt the cmake run that emit warnings
now and request the compiler to treat warnings as errors *iff* the cmake
run is not tainted. This way if a cmake run that is warning-free now
degrades later, CI will fail it.

The treat warnings as errors cmake option used is:
-Werror=dev
       Make developer warnings errors.

       Make  warnings  that  are  meant  for  the  author of the CMake-
       Lists.txt files errors. By default this will also turn on depre-
       cated warnings as errors.

Cirrus CI: Add the "make releasecheck" command in the Linux task

There is no need to add this command in all tasks.

Makefile.in: Add the releasecheck target

This change do, in a fresh directory from a release archive:
1) a "configure --enable-smb" and build,
2) a "cmake -DENABLE_SMB=yes" and build.

It may allow to find some necessary files missing in the release archive.

Define CMAKE_MESSAGE_LOG_LEVEL=NOTICE to minimize the cmake output.
Define CMAKE_RULE_MESSAGES=OFF to minimize the make output in cmake case.

[skip ci]

cmake: Update the minimum required version to 2.8.12 (except Windows)

Windows minimum required is already 3.12.

This change avoids this warning:
CMake Deprecation Warning at CMakeLists.txt:7 (cmake_minimum_required):
  Compatibility with CMake < 2.8.12 will be removed from a future version of
  CMake.

  Update the VERSION argument <min> value or use a ...<max> suffix to tell
  CMake that the project does not need compatibility with older versions.

(Same as in libpcap.)

CREDITS: Add authors of changes backported to 4.99.2

[skip ci]

CHANGES: Add more changes backported to 4.99.2

[skip ci]

CHANGES: Move a change backported to 4.99

[skip ci]

CHANGES: Add a change backported to 4.99

Remove the release candidate rcX targets

These tricky targets used "autoreconf -f" (via "make releasetar") and
they changed the VERSION and configure files locally.

If building a release candidate tar archive is needed, it's better to:
Update VERSION with rcX suffix and configure via "autoreconf -f",
commit them, add a tag and build the archive via "make releasetar".
It should display: Archive build from tag tcpdump-...rcX.

Note: the "autoreconf -f" command changes the configure file in an
uncontrolled way (runstatedir, #define LARGE_OFF_T, etc.) depending on
the autoconf version.
It is necessary to choose the parts that will be added in the commit
with "git add -i".

This is a follow-up to c378c2a04cdb3c8ec35341ca503cf1a2889a77d5.

[skip ci]

Refine the change log. [skip ci]

In the 4.99.2 section fix a typo and move changes that are not specific
to a particular protocol to the "source code" sub-section.

Add a change log entry for CMake RPATH bug fix. [skip ci]

doc: Update README.NetBSD.md after commit dcfed11. [skip ci]

Use "git archive" for the "make releasetar" process

Use the release tag if it exists or use HEAD.

Remove the "autoreconf -f" command, because it changes the configure file
locally in an uncontrolled way (runstatedir, #define LARGE_OFF_T, etc.)
depending on the autoconf version. This command is run in the release
process before a commit and we can choose the parts that will be added
in the commit.

Note:
The following target (rcX) must be updated ou removed in a next step
because it was using "autoreconf -f" (via "make releasetar") and it
changes the VERSION and configure files locally.

This change
1) Ensures that we only release files from tag/HEAD, not locally
   modified ones.
2) Avoids disclosing personal data such as the username/group of the
   local user.
3) Puts by default a umask of 0002, which turns off the world write bit
   on files in the archive.
4) Avoids problems on some OSes (no more tar, Git builtin tar.gz
   handling).

man: Lose an excess newline in tcpdump(1).

This improves the derived HTML version and has no effect on the plain
text version, as far as I can tell.

[skip ci]

CONTRIBUTING: Add some tags

CONTRIBUTING.md: Add information about the instrumentation of functions

Rename the suffix of a pcapng test file to .pcapng

CHANGES: Add a change backported to 4.99

PTP: fix printing of the correction fields

The nanosecond part of the correction field is 48 bits and the
sub-nanosecond part is 16 bits.

CHANGES: Add a change backported to 4.99

[skip ci]

NFS: Fix the format for printing an unsigned int

CHANGES: Add a main-only change

bgp: Add Origin Validation State extended community

Signed-off-by: Donatas Abraitis <[email protected]>

BGP: Update an error message

From:
    [invalid total segments len 7]
To:
    [total segments length 7 != N x 6] (invalid)

CHANGES: Add a main-only change

BGP: add dissector for BGPsec capability and path

BGPsec is specified in RFC8205. It adds a path attribute to BGP update
messages and a capability to BGP open messages.

tests: add pcap for BGPsec test

configure: use pcap-config --static-pcap-only if available.

If we're linking with a libpcap in ../libpcap*, it's static, but we only
need to link with the libraries on wich it immediately depends, we don't
need to link with the libraries on which those libraries depend, etc..

So, if ../libpcap*/pcap-config supports --static-pcap-only, use that.

Remove a trailing space

[skip ci]

Update config.{guess,sub}, timestamps 2022-08-01

From repository git://git.savannah.gnu.org/config.git

cmake: expand a comment. [skip ci]

The requirement for setting the run-time path in an executable is not
unique to NetBSD; it also appears in the other BSDs, Linux, and Solaris.

cmake: prevent stripping of the rpath on installation.

This should fix issue #1008.

Update some references to README files. [skip ci]

doc: Add a README file for NetBSD.

[skip ci]

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

configure.ac: fix configure tests broken with Clang 15 (implicit function declarations)

Clang 15 makes implicit function declarations fatal by default which
leads to some of tcpdump's configure tests silently failing/returning
the wrong result.

This adds the needed #includes to various tests for the functions used,
resolving the following errors:
```
net-analyzer/tcpdump-4.99.1/clang15.log:47:error: type specifier missing, defaults to 'int'; ISO C99 and later do not support implicit int [-Wimplicit-int]
net-analyzer/tcpdump-4.99.1/clang15.log:51:error: call to undeclared library function 'strcmp' with type 'int (const char *, const char *)'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
net-analyzer/tcpdump-4.99.1/clang15.log:55:error: call to undeclared library function 'sscanf' with type 'int (const char *restrict, const char *restrict, ...)'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
net-analyzer/tcpdump-4.99.1/clang15.log:68:error: call to undeclared library function 'memset' with type 'void *(void *, int, unsigned long)'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
net-analyzer/tcpdump-4.99.1/clang15.log:112:error: call to undeclared function 'ether_ntohost'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
net-analyzer/tcpdump-4.99.1/clang15.log:115:error: call to undeclared library function 'exit' with type 'void (int) __attribute__((noreturn))'; ISO C99 and later do not support implicit function declarations [-Wimplicit-function-declaration]
```

Signed-off-by: Sam James <[email protected]>

CHANGES: Add a change backported to 4.99

[skip ci]

ICMPv6: Fix the Node Information flags

Don't use htons() when defining the constants because the flags
field is already eventually swapped by fetching with GET_BE_U_2().
Remove unused flags NI_SUPTYPE_FLAG_COMPRESS and NI_FQDN_FLAG_VALIDTTL.
Add a test file with various NI_NODEADDR_FLAG* flags.

CHANGES: Add some main-only changes

[skip ci]

CHANGES: Add some main-only changes

[skip ci]

CHANGES: Add some changes backported to 4.99

[skip ci]

CHANGES: Add a change backported to 4.99

[skip ci]

ICMPv6: Fix output for Router Renumbering messages

In icmp6_rrenum_print() function.

From:
    router renumberingrouter renum: command
To:
    router renumbering, command

Add a test file from the Wireshark menagerie (5691-icmpv6-RFC2894-RR.pcap).