Merge branch 'traviscross:master' into master

Merge pull request #430 from bewing/max_display_path

add --max-display-paths option

add --max-display-paths option

It is not unknown to have datacenters with over 8 ECMP paths between two
hops.  Expand the maximum number of ECMP hosts stored in addrs to 128,
and add a new option to control the number to display, defaulting to the
old MAX_PATHS of 8

Merge pull request #429 from bewing/ecmp_order

use addrs for static host ordering in curses

use addrs for static host ordering in curses

Change d2552ca has the side effect of moving hosts around in the
multipath view, as the last host to reply to a packet was set as
addr.  Using addrs[0] istead of addr to draw the first host keeps
the ordering deterministic, in order of first reply.

The release script bumped the version number to 95

increased max length suggested by YVS2014

Merge pull request #416 from gmax0/master

Report secondary servers when CSV + wide report is enabled

Report secondary servers when CSV + wide report is enabled

ipv6 udp checksums like ipv4 but with ipv6 pseudoheader

this fixes a problem where both source and destination port are supplied on the commandline,
this lead to packets not showing up any more due to checksum being different.
this was mentioned first on https://github.com/traviscross/mtr/issues/351

Merge pull request #411 from trofi/master

ui/curses: always use "%s"-style format for printf()-style functions

ui/curses: always use "%s"-style format for printf()-style functions

`ncuses-6.3` added printf-style function attributes and now makes
it easier to catch cases when user input is used in palce of format
string when built with CFLAGS=-Werror=format-security:

  ui/curses.c:765:42:
    error: format not a string literal and no format arguments [-Werror=format-security]
    765 |         mvprintw(rowstat - 1, startstat, msg);
        |                                          ^~~

Let's wrap all the missing places with "%s" format.

Merge pull request #407 from a1346054/fixes

Minor cleanup

trim trailing whitespace

fix spelling

unify codestyle

fix shellcheck warnings

fix wrong bash completion flag

Merge pull request #399 from krisl/bits_and_pieces

dont show json option if not available

dont show json option if not available

Merge pull request #397 from rpaaron/consistent_resolution_strategy

Consistent resolution strategy

rename function

make Hostname as const

export get_hostent_from_name

freeaddrinfo

reset addr family before searching again

export DEFAULT_AF

switch gui to addrinfo

remove conversion function

use addrinfo

move conversion call to caller

extract convert_addrinfo_to_hostent function

remove temporaries

prepare host with h_addr_list

init structures correctly wired up

remove wrapper only function

resolve ipv6 only if we have ipv6

tell dns process if we want 4 or 6

Merge pull request #395 from krisl/reset_address_family

Reset address family

accept only value used in structure

accept only value used in structure

accept only value used in structure

reset ctl address family at net_reopen

Merge pull request #393 from rpaaron/fix_changing_source

Fix changing source

Slight cleanup, but no fix for code that came up in a bugreport.

refactor - group local, remote inits

additional call from net_reopen

re-init source too

addr -> hostent for consistency

move net_send_batch call to its caller

Merge pull request #391 from vincentbernat/fix/mpls-report

report: fix display of MPLS labels when using --report

Merge pull request #390 from vincentbernat/fix/mpls-display

net: fix MPLS display for curses and report

report: fix display of MPLS labels when using --report

Previously, using --report --mpls would not display MPLS labels unless
combined with --aslookup or -y. Fix this case.

```
 23:03 ❱ ./mtr  -z --report-wide -e 203.0.113.10
Start: 2021-03-09T23:03:43+0100
HOST: H1                    Loss%   Snt   Last   Avg  Best  Wrst StDev
  1. AS???    192.0.2.1      0.0%    10    1.0   1.1   0.8   1.8   0.3
  2. AS???    169.254.0.2    0.0%    10    2.5   2.5   2.4   2.8   0.1
       [MPLS: Lbl 299776 TC 0 S u TTL 1]
       [MPLS: Lbl 16 TC 0 S u TTL 1]
  3. AS???    203.0.113.1    0.0%    10    2.2   2.7   2.1   4.8   1.1
  4. AS???    203.0.113.10   0.0%    10    2.3   2.3   2.2   2.6   0.1
 23:04 ❱ ./mtr   --report-wide -e 203.0.113.10
Start: 2021-03-09T23:04:07+0100
HOST: H1           Loss%   Snt   Last   Avg  Best  Wrst StDev
  1.|-- 192.0.2.1     0.0%    10    0.8   2.9   0.7  20.9   6.3
  2.|-- 169.254.0.2   0.0%    10    2.8   2.7   2.4   2.8   0.1
       [MPLS: Lbl 299776 TC 0 S u TTL 1]
       [MPLS: Lbl 16 TC 0 S u TTL 1]
  3.|-- 203.0.113.1   0.0%    10   16.1   3.7   2.0  16.1   4.4
  4.|-- 203.0.113.10  0.0%    10    3.5   2.5   2.2   3.5   0.4
```

This is not really elegant and I didn't try to display with vertical
bars like it should be done when no additional IP info is requested,
but the code already has many branches with very similar code and I
didn't want to add even more complexity to it.

Fix #341

net: fix MPLS display for curses and report

After #364, MPLS labels were not displayed anymore due to a logic
invertion. Previously, the addr/mpls fields were populated first and
addrs[i]/mplss[i] copied from these values. After the patch, the
reverse is happening but populating addrs[i] and mpls[i] was still
done by copying the value from addr/mpls. This was not a problem for
displaying addresses as addr was used however for mpls labels, this is
mplss which is used exclusively (net_mpls() is not using mpls field,
but mplss field).

After #364, hosts position is changing depending on the most recent
paths. I didn't fix that as it may be seen as a feature but this can
be a bit disturbing.

Also, unrelated, but the unnecessary/unsafe casts to void pointer are
removed for net_addr(), net_addrs(), net_mpls() and net_mplss(). Also,
net_mpls() is using mpls field instead of mplss as this seems more
correct.

This has been tested with curses and report. Also, raw is still
working.

Fix #370.

Merge pull request #389 from vincentbernat/fix/avoid-void-casting

ui: don't cast to void* when calling display_rawhost()

ui: don't cast to void* when calling display_rawhost()

The provided types are compatible with the function signature.
Moreover, this seems to hide a bug where `display_rawhost()` is called
with an address instead of an MPLS struct.

Merge pull request #376 from epanoff/fix-readme

Updated Readme for windows: added libjansson

Updated Readme

added libjansson-devel for building on win

added clarification to readme suggested by Zenithal

Code formatting for Zenithal pull

Merge pull request #373 from ZenithalHourlyRate/master

Add display of destination with resolved addr under curses mode

Add display of destination with resolved addr under curses mode

The release script bumped the version number to 94

The release script bumped the version number to 94

Changed MAXPATH to MAX_PATH for AIX compatibility. -- aixtools

Added extra help text to configure --yvs

Merge pull request #364 from atibdialpad/custom-mtr

Code to allow maxTTL to change dynamically when path changes in the middle of mtr execution

TODO list changes

Change TTL dynamically to adjust for path changes

Merge pull request #362 from atib92/mtr-109

Added code to print multiple addresses regitered on the same hop count

Added code to print multiple addresses regitered on the same hop count

Merge pull request #355 from ltWolfik/jansson

report: json: reworked with libjansson

report: json: reworked with libjansson

Merge pull request #354 from bluPhy/master

Updating some coments (SIC) :-)

Updating some comments

Merge pull request #352 from bluPhy/master

few updates to manual page and README.md

few updates to manual page and README.md

Merge pull request #345 from markeganfuller/display_target

Add display of destination.

Add display of destination.

Merge pull request #340 from Sea-n/master

Fix parameter in ui/net.c

Fix parameter in ui/net.c

The 'net_send_query' function was use global varible 'packetsize'
and ignoring 'packet_size' parameter.

Merge pull request #336 from ym/master

show mpls information in raw output

show mpls information in raw output

More compilation warning fixes from obouizi

Merge branch 'master' of github.com:traviscross/mtr

Added include errno --obouizi

Merge pull request #331 from commonism/ip6udp

ip6 udp - fix probes with local or remote port

ip6 udp - fix probes with local or remote port

only set the remote port in raw packet, not as sendto parameter

does not fix local and remote port, seq number is checksum then,
payload has to be adjusted as with ipv4

fix closing brace

proposed patch for bsd compile error

net find local address fix by meingtsla

Merge pull request #325 from kbucheli/tun_interface_segfault

fix segmentation fault if there is no IP address on an interface (fixes #320)

fix segmentation fault if there is no IP address on an interface (fixes #320)

Merge pull request #322 from commonism/cleanups

sockaddr related cleanups

simplification - improve readability

simplification - address addrcmp

Merge pull request #319 from krisl/fix/stack_use_after_scope

Fix/stack use after scope

avoid stack use after scope

introduce libasan

Merge pull request #310 from krisl/fix/raw_dns

Fix/raw dns