Consistent variable names for command-line options

ci: coverity scan check COVERITY_SCAN_TOKEN

work around NetBSD's lack of IP_RECVTOS

NetBSD does not know the IP_RECVTOS socket option.  Thus
conditionally compile setting this option only when it is
defined.

This is the same change as used in the NetBSD package[1],
with thanks to "adam".

[1] https://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/net/fping/patches/patch-src_fping.c?rev=1.5;content-type=text%2Fplain

configure.ac: improve portability

Testing string equality using "test" is done with the "="
operator.  Bash also accepts "==", but this is not portable.
The issue was noticed on NetBSD, which uses a patch to the
generated "configure" script[1].

[1] https://cvsweb.netbsd.org/bsdweb.cgi/pkgsrc/net/fping/patches/patch-configure?rev=1.3;content-type=text%2Fplain

Performance: use MSG_DONTWAIT instead of select.

This also improves a bit the conversion from nanoseconds to
seconds + microseconds.

ci: Removing travis-ci

Implement coverity scan with GitHub Action

Include JSON schema in release tarball

Release v5.5

Fix usage pod2man under FreeBSD and OpenBSD

Release 5.5-rc2

Reject negative numbers for options requiring positive arguments

Replace sscanf with strtoul/strtod for stricter option parsing

Replace atoi/atof with stroul/strtod

This improves validation of the flag values (so that for example
-c 3foobar is not accepted anymore).

ci/prepare-linux.sh: do not run setcap unless necessary

Small code coverage improvements

Add .local to .gitignore.

Make it easier add files that are ignored when working with git or jj.

JSON: rename globalSum to "stats" and avoid spaces in key names.

Implement GPG verification of autotools tarballs

Improve robustness of downloading of autotools tarballs.

GitHub actions: don't run tests twice for github PRs

Update autotools.

Use SIGINT to kill fping so that coverage data is collected.

Tests performance and robustness fixes.

ci: improve autotools build reliability with ftpmirror and retries

Fix measurement of time for timed reports (-Q) to start after DNS name resolution.

Prepare v5.5-rc1 release

Fix GitHub markdown alert format

JSON documentation: reformat

ci: Github action Build and Push Docker Image rollback sigstore osign-installer version

ci: Update github action version

ci: Fix test unkown host

JSON format documentation

json: simplify recv and timeout output (and code)

Allow --json also for loop-mode

ci: Fix spelling mistake in GitHub action Build and Push Docker Image

ci: Start the GitHub action Build and Push Docker Image only when pushing to develop branch

Remove double quote from URL Coverity Scan badge

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

Change status image from Travis to Github Action

ci: Github action Build and Push Docker Image

improve limits documentation in man page

* Mention --enable-safe-limits in -i and -p descriptions
* Mention new unconditional limit for -p in option description
* Mention new unconditional limit for -p in limits section
* More consistent formatting in limits section by using B<>
  for the generator options

adjust current CHANGELOG.md entries

* add thanks to @hoodadt for the merged pull request
* adjust line lengths using `fmt -c`

initialize all resp_times array entries

Before, the resp_times array was initialized starting with the
second entry, leaving the first entry uninitialized.  This is
at least confusing.  Initializing one additional int64_t value
per ping target should not add too much overhead, so do this just
to be on the safe side.

Assuming that the first array entry need not be initialized when
creating the array now, this guards against potential problems
from future code changes that do not take this unexpected special
case into account.

Print receive ping moved to new functions

prevent allocation size computation overflow

On systems where size_t is unsigned int, computing the size
of the resp_times array can overflow.  Add a check to prevent
this.

On a 64-bit x86_64 Ubuntu 22.04.5 LTS system, the overflow check
results in a compiler warning:

```
fping.c: In function ‘add_addr’:
fping.c:3444:20: warning: comparison is always false due to limited range of data type [-Wtype-limits]
 3444 |         if (trials > (SIZE_MAX / sizeof(int64_t)))
      |                    ^
```

Thus limit the check to systems with a size_t equal to (or less
than) unsigned int.

Fix missing escape character in JSON output test

Fix the missing new line character on JSON output for an invalid host warning

Fix changelog entry for the new -p lower limit

Update changelog

Safe memory allocation for event storage.

- Check calloc return value.
- Make sure that the per-host interval can't be unreasonably low (making the event storage allocation very large).

The issue was reported by David.A.

Fix off-by-one boundary check in seqmap code

Remove macos-specific test to verify that -I is not supported, since that seems to work now.

Failed test:

```
#   Failed test 'exit_is_num: fping -I lo0 127.0.0.1, 3'
#   at ci/test-07-options-i-m.pl line 63.
#          got: 0
#     expected: 3

#   Failed test 'stdout_is_eq: fping -I lo0 127.0.0.1, fping: cant bind to a particular net interface since SO_BINDTODEVICE is not supported on your os.
# '
#   at ci/test-07-options-i-m.pl line 64.
#          got: '127.0.0.1 is alive
# '
#     expected: 'fping: cant bind to a particular net interface since SO_BINDTODEVICE is not supported on your os.
# '
# Looks like you failed 2 tests of 28.
```

Update .gitignore

Update Dockerfile

More Labels

Fixed Dockerfile Typo

Update CHANGELOG.md

Update Dockerfile

Better Dockerfile

Better Dockerfile

suppress duplicate reports with -c/-C and -q/-Q

If fping receives a duplicate Echo Reply packet, it reports
this.  This report was suppressed for loop mode, and for
other operating modes where each receive event is reported.
Now, this duplicate report is also suppressed when in count
mode with options -q, --quiet or -Q, --squiet.

This addresses GitHub issue #389.

fix ICMPv6 error message handling and reporting

Handling of received ICMPv6 messages different from Echo Reply
was based on IPv4 code.  It would thus use a too small minimum
size, and interpret the wrong data as ICMPv6 fields.  Thus
received ICMPv6 error messages were either not reported at all,
or were reported incorrectly.

This commit introduces the following changes to fix this and thus
address GitHub issue #390:

- Filter out irrelevant ICMPv6 messages: fping needs to receive
  ICMPv6 Echo Reply and ICMPv6 error messages, nothing else
  (only relevant for raw sockets).
- Fix offset calculation to look at the included packet that
  caused this ICMPv6 error message.
- Ignore invoking packets that do not have ICMPv6 as the first
  IPv6 Next Header in the packet (there could be, e.g., IPv6
  Extension Headers).
- Fix ICMPv6 message type determination, and print respective
  warning messages.

Entry for JSON output in the changelog

Change of the JSON return text if -c or -C was not specified

optparse_long for json changed to OPTPARSE_NONE

Add JSON support for --timestamp-format

Fix JSON output print_per_system_stats_json dash to null

Add JSON timeout event type

Format change for handle JSON output for invalid hosts

New option -J / --json for the JSON output

ci: Fix forgotten github action codeql init v3

ci: Change codeql deprected action v2 to v3

new option --seqmap-timeout=MSEC

This allows to control the time after which sequence numbers
can be reused for new probe packets.  This is intended to
help using count or loop modes with tens of thousands of
target hosts.

See GH issue #385 for more information.

print TOS & TTL also for IPv6 and w/o privileges

When using IPv4 raw sockets, fping has access to the received
IP packet data including the IP header.  This allows to simply
read the TOS byte and/or TTL value from the IP header.

But, using IPv4 raw sockets requires privileges.  Additionally,
IPv6 raw sockets do not provide access to the IPv6 header.

For IPv6, or when using IPv4 datagram sockets, i.e., when fping
runs without privileges, a different method needs to be used to
access TOS / TC and TTL / Hop Count of received packets.  The
so called "control messages", also called "ancillary data", can
be used for this.  This requires several actions:

* set the respective socket options to request additional data,
* ensure the buffer used to receive additional data is sized
  large enough,
* read and process the requested additional data.

Fix: OpenBSD sprintf warning

fix build without IPv6 support

In commit 57d91b13f20529a23df8c29eaf612d8483e318d8, I missed
one set of "#ifdev IPV6 ... #endif" guards for a piece of code
only relevant for IPv6 support in the generator.  This broke
the build with --disable-ipv6, or without IPv6 support on the
build system.

restore inadvertantly removed IPv4 test for -k

This was removed in an commit by me that should only have added
a new test (commit 4160d03a2653684b37a776744095aac43f7efc88).

ci: Change github action unsupported ubuntu-20.04

fix debug build use of dbg_printf in fping.c

config.h must be include before fping.h, because
configure --enable-debug writes the DEBUG #define
to config.h, and the dbg_printf macro definition
requires a defined DEBUG to actually produce any
output.

Unbreaking dbg_printf() exposed an instance of
wrong use, so fix that.  Also, add a newline to
the debug output reporting used CPU time.

test that --icmp-timestamp requires privileges

fix typo in S_BINDTODEVICE error message

Also, add tests with -I.  In the currently use GitHub CI
system, only GNU/Linux supports this option.  But, network
device names other than "lo" change too often on GNU/Linux
to know in advance what an interface might be called.  This
limits what can be reliably tested.

handle long input lines when reading from file

This addresses GitHub isse #377.

Before, input lines longer than the static buffer used to read
data would be handled as if each buffer part were one line.
This could result in splitting input target names into multiple
different target names.

Since the static buffer was significantly smaller then the maximum
length of DNS names (just short of 255 one octet characters),
valid long DNS names could not be read as targets via file.

This commit fixes the splitting of long lines, and also increases
the maximum length of target names to 255 one octet characters.

The additional line parsing code is kept similar to the existing
code and attempts to keep all intended and/or useful features to
minimize the risk of regressions.

tests for useful input file features

* whitespace in sufficiently short lines is skipped
* additional words after the target are ignored in sufficiently
  short lines
* whitespace-only (i.e., blank, but not empty) lines are skipped
* DOS-format text files, i.e., lines terminated with CRLF, work
* input need not be a correct POSIX text file, i.e., non-empty
  input does not need to end with LF

IPv6 support for -g, --generate

The limit for generated addresses is increased by two to
allow /111 IPv6 prefixes similar to /15 IPv4 prefixes.

The implementation uses two unsigned 64 bit integers to
represent a single IPv6 address.  The required arithmetic
is open-coded and limited to what is needed to implement
IPv6 support for -g, --generate.

This addresses GitHub issue #59.

reorganize add_cidr()/add_range()

This creates a code structure suitable to add IPv6 versions
of the functions used to generate addresses.

check address family consistency in generator

Before, "fping -6 -g IPv4_ADDR_SPEC" would generate all requested
IPv4 addresses and try to add them to the host table, but that
would fail because of address family mismatch.

Now, "fping -6 -g IPv4_ADDR_SPEC" detects the problem before
generating any addresses, and fails early.

Mixing IPv4 and IPv6 addresses when using -g START END is detected
before generating any addresses and fails early, too.

complete SO_TIMESTAMPNS to SO_TIMESTAMP fallback

Commit e8660637ccc3bf2aae163fa7c26216843ae98180 added a
fallback from setting the socket option SO_TIMESTAMPNS to
setting the socket option SO_TIMESTAMP if the nanosecond
timestamp option could not be set.  But it did not add
code to also look for the control message related to
SO_TIMESTAMP.  Thus microsecond timestamps were requested,
but not read.

This commit adds the missing code to read microsecond
timestamp control messages.

The problem was reported in GitHub issue #374 by @payload.

Release v5.3

Prepare 5.3-rc3

adjust ICMP Timestamp output

* print TOS and/or TTL after the timestamps
* use the same output format with and without loop or count modes
* omit the comma in front of the timestamps with -a, --alive
* slightly extend man page description of --icmp-timestamp
* adjust existing ICMP Timestamp tests to output format changes
* add tests for some combinations of --icmp-timestamp with -a, -e,
  --print-tos, and --print-ttl

Prepare 5.3-rc2

support --print-{tos,ttl} with -cN, -CN, and -l

This addresses GitHub issue #367.

CHANGELOG.md tweaks

Prepare 5.3-rc1

ci: Upgrade actions/upload-artifact to v4

Prepare v5.3 release: CHANGELOG

add local receive timestamp to --icmp-timestamp

This addresses GH issue #361.

ensure IPv4 is used with --icmp-timestamp

The ICMP Timestamp message is defined for IPv4, but not for IPv6.

Before, combining --icmp-timestamp with an IPv6 address or a DNS
name for a dual-stacked target on a dual-stacked system resulted
in sending ICMPv6 Echo over IPv6.

Now, DNS names for dual-stacked hosts just work, i.e., fping uses
IPv4 with --icmp-timestamp.  Combining --icmp-timestamp with an
IPv6 address or an IP6-only DNS name now fails.

use correct data size when sending ICMP Timestamp

ICMP Timestamp messages have a fixed size.  The data portion
following the ICMP header comprises 12 octets to hold three
timestamps of 4 octets each.

Giving the --icmp-timestamp option now automatically sets the
data size to 12 octets.

Error out when -b N is used together with --icmp-timestamp.

Adjust man page to mention incompatibility of -b N, --size=N
with --icmp-timestamp.  Also mention in man page that
cap_net_raw can be used instead of root privileges for ICMP
Timestamp.

stricter response type checking

When sending an ICMP Echo message, accept only an Echo Reply
message as a normal response.  When sending an ICMP Timestamp
message, accept only an ICMP Timestamp message as a normal
response.

This could matter when multiple fping processes are running
in parallel, some using Echo, others Timestamp messages.