Fixes the indentation of the blocks

Set ipi6_addr in IPV6_PKTINFO when source address is bound

Remove unnecessary IPV6_RECVPKTINFO setsockopt

Test asymmetric routing (--oiface)

Fix: set ipi_spec_dst from -S source address in sendmsg()

Add IPv6 test for --oiface

The --oiface option is not allowed with -I --iface

New option --oiface for outgoing interface

Expansion of README to include docker usage

Add IBM i PASE detection

ci: Update autoconf to version 2.73

Move option parsing to a separate file fping.c and fping.h

Move printing functions and stats functions (global stats) to a separate file

Move the most function declarations from fping.c to fping.h

avoid crash with option -0

Instead of using the possible option character '0' (ASCII value 0x30)
to indicate a long option without a short alias, use the value 0 as
indicator.  This value is treated specially by the used option parsing
code, and might even be the intended value to use for long-only options.

Before, using option `-0` sometimes, but not always, resulted in a
crash with `Segmentation fault (core dumped)`.

This fixes GitHub issue #456.

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.