allow combination of -a with --print-tos

This is similar to how -e, --elapsed works.
Also adjust -h output regarding --print-tos.

Fix warning unused parameter 'reply_timestamp' under macOS

Print returned TTL value

Co-authored-by: Luís Fonseca <luis@lhf.pt>

docs: adjust description for --print-tos

Every IPv4 packet has a TOS byte.  Every IPv6 packet has a TC byte.
But fping can only read it when using raw sockets, and only for IPv4.

Increase maximum -b value to 65507

Correct definitions: maximum theoretical IPv4 packet size and minimum
IPv4 header size (previously probably IPv6 size was used).

Because fping does not know ahead whether address is IPv4 or IPv6 assume
IPv4. Previously fping allowed only 65488, but real maximum for IPv4 on
Linux is 65507 (IPv6 would allow 65527 follow the current approach and
allow smaller value than reachable for IPv6).

Update test affected by this change. While at this, test 65508 (one
above the limit) instead of 65509.

Signed-off-by: Petr Vorel <pvorel@suse.cz>

CI: use chmod and File::Copy's copy, not system

Before, ci/test-11-unpriv.pl used Perl's "system" function
to execute a Unix command line to copy a file and change the
permissions of the copy.

Now, the Perl functions "copy" (from File::Copy) and "chmod"
are used instead.

CI: use File::Temp to create temporary directory

Before, ci/test-11-unpriv.pl used a hard-coded file name of
/tmp/fping.copy to create an unprivileged copy of fping.  This
file was not deleted after it was no longer used.

Now, the unprivileged fping copy is created in a temporary
directory that is deleted after all tests from the file.

This addresses GH issue #339.

fix -k, --fwmark with setuid fping executable

* Setting the SO_MARK socket option requires root privileges (or
  CAP_NET_ADMIN), thus temporarily elevate privileges for this
  operation, if possible, by using the p_setsockopt() function.
  This allows to use -k, --fwmark with setuid fping.
* Adjust README.md since setuid now works for -k, --fwmark.
* Remove "sudo" from testing -k since it is no longer needed.
* Test failure of setting firewall mark without privileges or
  capabilities for both IPv4 and IPv6.

test error resolving host name

The top level domain '.invalid' is reserved for such tests.

basic test for -d, --rdns

Usually, -d gives the same result as -n.  Test this.

When the target is a name that resolves to an IP, but the IP
does not resolve to the same name or any name at all, the
output differs.  This is harder to test, because this is
usually seen as a misconfiguration, so leave this out for now.

more tests with reply source != target

The tests for --check-source verify that Echo reply packets
from a source IP that was not used as the target of the Echo
request are not accepted.  It does so for default fping, for
use with -c, and for use with -C.  Test if the source address
is really printed without --check-source, but with -c or -C.

test --print-tos with an IPv6 target

Currently, --print-tos only works with IPv4.  In contrast
to IPv4, IPv6 raw sockets do not provide access to the
raw IP packet.  Thus the method used to access the TOS
byte value of an IPv4 packet does not work for the TC byte
value of an IPv6 packet.  This results in always generating
'(TOS unknown)' for IPv6 targets.

test --print-tos without -O

Printing the TOS byte of the received Echo response does not
require setting a non-default TOS byte value in the Echo request.

test --print-tos without raw socket

The current code can only read the TOS byte when using a raw
socket.  It prints '(TOS unknown)' when a datagram socket is
used.

test-11-unpriv.pl: always use /tmp/fping.copy

Document that -a and -u are overridden by -c and -C

The other way around was already documented in commit
ae0bc2380462878466f19aa45c7d38774f4b6e3f, but GitHub
pull request #337 by Moritz Lenz implies that this is
insufficient.

Print returned TOS value

new option --check-source

Using the new option --check-source discards Echo reply packets
sourced from an address that is different from the specified target
address.  This can be useful on busy monitoring hosts, because it
is possible that two fping processes running at he same time use
the same ICMP Echo Reply Identifier for ICMP Echo messages sent to
different hosts.

This aims to address GH issue #206.

Sets SEQMAP_TIMEOUT_IN_NSSEQMAP_TIMEOUT_IN_NS as INT64_C

adjust generator limit to allow an IPv4 /15

fix off-by-one error in generator limit

This commit does not add tests that verify the exact limit,
because pinging 100000 localhost addresses takes over 15
minutes on my PC.  I have tested this fix manually.

also apply generator limit to use with CIDR

As described in GH issue #299, the MAX_GENERATE+1 limit is only
applied when using an address range, not when using CIDR.  This
commit changes this to always honor the generator limit.

* refactor target address generation to use the same new function
  for both range and CIDR notation
* check the limit for addresses to generate in the new function
* document the generator limit in "fping --help" output
* document the generator limit in the fping man page
* test that the address generation limit applies when using CIDR
  notation

align functions add_cidr() and add_range()

* call freeaddrinfo() before checking the prefix length in
  add_cidr()
* add a comment to the generator limit check in add_range()

ci: skip an unreliable test on macOS

Sometimes, this test fails on macOS, at least on the GitHub CI.
It looks as if the fping process is killed before it produces
the required output.  This test works reliably on GNU/Linux, so
keep it, but skip it when running on macOS.

ci: test timestamp plausibility

Some changes of fping resulted in some systems reporting
implausible timestamps, see, e.g., GH issue #203.  This
commit adds a simplistic plausability check for timestamps
in Unix time format: they need to be greater or equal to
1000000000 (2001-09-09 03:46:40+02:00).

ci: test empty string argument instead of number

Giving an empty string instead of a number as option argument
was not reliably caught, see GH issue #324 and GH PR #323.

Fix incorrect return-value check for a scanf like function

configure.ac check required function strftime

Extended dokumentation for parameter timestamp-format

Test 05 for --timestamp-format extended

New parameter timestamp-format for changing the timestamp output

Handling optparse long option

Change to codeql-action-v2

Debug with Visual Studio Code

test error when binding to wrong local address

Test "fping -S" with both IPv4 and IPv6 addresses not configured
on local interfaces using addresses reserved for documentation.
Check the fping error message, but accept any system generated
additional information.  This additional information differs
between operation systems.

test help output during normal option parsing

The existing test for "fping -h" triggers the special code path
for only one option, either -h or --help, that is handled before
normal option parsing.

Print cumulative stats with -Q SECS,cumulative

With -Q SECS, fping emits information for the just finished
interval.  This is good for long-running fping processes where
one would like to get an update on recent results, ignoring
missing responses from older intervals.  This seems appropriate
for use with other software like netdata or smokeping.

But for short(er)-running fping processes, e.g., during a change
window, it may be more interesting to get a status update since
the beginning of the change window, i.e., the start of fping.
Adding a comma followed by the keyword "cumulative" to -Q SECS
(i.e., -Q SECS,cumulative) changes the interim reports to output
the current per system overall statistics, unless -N is used, too.

The new syntax is documented in fping.pod and added to fping -h
output.

Four tests are added:

  1. Characters after SECS are ignored as before.
  2. Unknown keywords are ignored as before.
  2. Adding ",cumulative" changes -Q output.
  3. Adding ",cumulative" also affects the additional -o output.

This addresses issue #243.

Prepare v5.2 release.

Removes the unnecessary file 18

Extended configure.ac with --enable-debug and create debug output cpu time use

Update Azure Pipline add macOS pipline job

Update Azure Pipline linux test based on github pipline with matching exclude

Fix number of tests

Remove flaky test for now.

Remove the 'fping -X 2 --generate 8.8.0.0/29' test, because it
is currently failing on the MacOS runner. Also, I am not sure that
it is a good idea to ping the IPs 8.8.0.1 and 8.8.0.2 and rely on
them being up.

Check if the variable next_value->ping_ts != 0 in seqmap_add()

always output error message in crash_and_burn()

This aligns crash_and_burn() with errno_crash_and_burn()
and addresses issue #300.

improve help output for options -c and -C

use function stats_reset_interval()

Git commit bb8d71ef27f673f25c7e1648ea982c27bb61ac0d from 2020 among many
other things introduced the function stats_reset_interval() to reset the
*_i interval (or split) host statistics, but did not use it.  This commit
replaces the open coded reset of interval stats inside print_host_splits()
and print_netdata() with calling stats_reset_interval().

typo fixes pertaining to ICMP for IPv6 sockets

As mentioned in issue #300, error messages pertaining to
socket creation may not be printed.  But they should be
corrected anyway, just as the comments.  Having correct
error messages also makes a fix for issue #300 more
obvious, because the messages itself would not change.

Document that -c N overrides -a or -u

Decouple -a/-u effects from -c

As described in issue #295, the option combinations "-c N -u"
and "-c N -a" inadvertently have the same effect as "-c N -q".
Prevent this by first acting on -c, which includes disabling
either -a or -u, if -c is given, before acting on -a/-u.

Move description of -i MSEC to probing options

The -i MSEC, --interval=MSEC option does not directly affect
the output format, but rather the probe generation, i.e., the
minimum time between probes.  Thus it is more fitting to list
it under "probing options" than under "output options".

Test ping to "all IPv6 nodes" multicast address

Any Echo Response packet uses a different source address than
used for the target, because it is not allowed to use a multicast
address as source.  This results in extra output in a specific
format on standard error.

This should work, because IPv6 requires multicast support, and
every IPv6 node is supposed to join the "all nodes" multicast
group, including the node the test runs on.  It at least works
on my Ubuntu 20.04 LTS system.

Test edge cases of -f and -g

* -f with non-existing file
* -f with input file containing comment and empty line
* -g with non-numeric address in "CIDR" format
* -g with one non-numeric address in start resp. end position
* -g with one IPv6 address in start resp. end position

Test -l together with -Q or -t

Test that -x, -X, -k require numeric argument

Test that negative numbers are not accepted

Fix a comment describing a test

Defensive coding: guard against a division by zero

In the function print_per_system_stats(), if packets have been
lost, the number of sent packets is checked to be positive before
dividing by it.  If no packets have been lost, this is not checked.
Either the existing check is not needed, or both code paths need
the check.

The function print_per_system_splits() is quite similar to
print_per_system_stats(), and has the equivalent guards against a
division by zero in both code paths, not just one of them.

In the spirit of defensive coding, I think it is better to be safe
and add the missing guard against a division by zero.

Update exit status description in man page

With the options -x and -X, fping does not require all hosts to
be reachable for an exit status of 0.

This addresses a comment in issue #99 suggesting such a man page
update.

Test examples from man page

The man page gives two examples combining -C and -q, but
these combinations are not yet tested.  Add tests to help
ensure the documented examples continue to work.

Add test for option combination -Q SECS -o

The -o option adds outage duration to the periodic reports.
Each report shows the outage since the last report, not since
the beginning.

Add longer test for -Q SECS

The statistics printed every SECS seconds show the results
since the the last report, not since the beginning.  Also,
every report starts with a timestamp.

Test that -g does not accept IPv6

Test edge case behavior of "fping -g"

* wrong number or kind of arguments fails and prints usage;
* an empty range silently pings nothing and fping returns 1;
* a too large range fails with an error message;
* a zero CIDR prefix length fails with an error message.

Add --fwmark/-k option for Linux

Change obsolete macro from AC_CANONICAL_SYSTEM to AC_CANONICAL_TARGET

Move the OS detection to configure.ac script to set the use of clock_realtime

Fix deprected github action ubuntu-18.04 to ubuntu-22.04

Checking obsolete macro AC_PROG_CC_STDC as of autoconf-2.70

Checking obsolete macro AC_GNU_SOURCE as of autoconf-2.62

Remove dockerfile from root

Move to contrib folder

Dockerfile for fping

Repair the faulty test 15

remove host from netdata chart titles

remove www

update netdata link

Updating documentation regarding root requirements since for --file parameter

fping: Initialize msghdr struct in a portable way

Initializing the structure assuming glibc layout results in
compile errors on musl, therefore do partial intialization and then
assigning the members individually.

Signed-off-by: Khem Raj <raj.khem@gmail.com>

Test for fping: add option to exit immediately once N hosts have been found

fping: add option to exit immeditely once N hosts have been found

Fixing build warning long int on seqmap.c line 84

OpenBSD is not affected by the "broken alpha headers" bug, too.

Fallback to SO_TIMESTAMP if SO_TIMESTAMPNS is not available

correctly call getnameinfo()

reformat

NetBSD/alpha is not affected by the "broken alpha headers" bug.

fix socket4 vs socket6 typo (#249, thanks @sfan5)

fix unprivileged ping broken in v5.1 (fixes #248)

update changelog and 5.1 release date

Update autotools and fix code coverage issues.

add missing files to autoclean.sh

At least on my system (Ubuntu GNU/Linux 18.04.6 LTS), autoclean.sh
misses three files: compile, doc/Makefile, and src/Makefile.  Add
them.

Remove dublicate include time.h entry

GH action test

Create codeql-analysis.yml (#241)

Create codeql-analysis.yml

Merge pull request #240 from auerswal/doc-fixes

Documentation improvements

Improve -Q documentation

The `-Q, --squiet=SECS` option does not replace the summary
statistics of `-q, --quiets`, but adds interval summary
statistics every SECS seconds to the output.

Improve -d and -n documentation

* Adjust `fping.pod` entries for `-d, --rdns` and `-n, --name`:

  * The code performs reverse DNS lookups on the given targets,
    not on source addresses from ICMP Echo Responses or ICMP
    error messages.  Adjust the `-d` description to reflect this.

  * The sentence describing `-n` seems incomplete, so provide a
    completion in similar spirit to `-d`.

* Add `-d, --rdns` to `-h, --help` output since `-d` and `-n` are
  no longer equivalent.

Merge pull request #236 from schweikert/travis-fix

Prepare 5.1 release