build-sys: tell function locality explicitly

Most of the function prototypes in header files are indeed external, with
few exceptions that can be set to local scope with static key word.
Additionally get rid of net_duplicate() prototype that does not exist
anywhere.

build-sys: use system getopt_long() when it is available

And move the getopt sources to portability support directory.

cleanup: move max port number to be a define in net.h

usage: use error(3) error-reporting function

There are two benefits of using error(3).  First of all this function
unifies error printing nice way, and when done this way a programmer cannot
accidentally mix stderr and stdout error print outs.  Secondly the error(3)
appends program name to errors, making it clear what printed an error.  To
demonstrate this see below for old and new output, in which the old error
could have in theory been reported by sudo(8).

$ sudo mtr -u -T # old
-u , -T and -S are mutually exclusive.

$ sudo ./mtr -u -T # new
./mtr: -u , -T and -S are mutually exclusive

usage: be careful when parsing numeric user input

Functions atoi(3) and atof(3) do not perform any error checking.  Use
strtol(3) function family, with alterations to fail if input is not
expected.

Merge pull request #142 from kerolasa/pkg-config

configuration cleanup, Pkg config.

cleanup: remove preprocessor missing functions go-arounds

Calling empty functions when there is no support is messy, and harder to
read than not calling at all functionality that is not present.

usage: reflect ./configure choices in available command line options

Features that are turned off should not be available to users as an options.
This commit also ensures matching long and short options, and makes the
long-only --displaymode option to be totally unreachable with short options.

cleanup: remove obsolete herror() function

Applications should use getaddrinfo(3), getnameinfo(3), and gai_strerror(3)
instead.  Systems without getaddrinfo() are from now on considered too
broken to be supported.  These include for example platforms: HP-UX 11.11,
IRIX 6.5, OSF/1 5.1, Solaris 7, Cygwin 1.5.x, mingw, MSVC 9, Interix 3.5,
and BeOS.

build-sys: remove unused autoconf check values

Return values from checks are not used anywhere in this project.

build-sys: simplify finding resolver library

cleanup: remove NO_SPLIT preprocessor check

Apparently ./configure --without-split was planned to be added in commit
433bf6c22ce90e25f3ffb0b4852631e5724ac608 but that has not happen in 17
years.  Time has come to get rid of a half implemented idea.

build-sys: get rid of double negative ipinfo autotools settings

build-sys: use pkg-config to find ncurses

The curses function attron() has been part of posix since 1997.  There is no
need to go-around for it.

Reference: http://pubs.opengroup.org/onlinepubs/7908799/xcurses/attron.html

build-sys: use pkg-config to find gtk+-2.0

build-sys: check pkg-config availability

fixed #141 compile without SCTP if not available

fixed #27 and #35 where the fix was tested a long time ago.

Merge pull request #140 from aquerubin/stdev

Fix standard deviation calculation.

Fix standard deviation calculation.

Partial reverse of 6bb5b6b3b.

The bug in automake does not create a vulnerable distirbution. It
allows local attackers that are logged in (or running in the
background) when a distributer types "make distcheck" to possibly gain
root access on the build system. This is NOT creating a vulnerable
distribution. The fact that automake has a vulnerability does not
warrant build failures on older systems.

more cleanup

Merge pull request #138 from kerolasa/2016-08-17

2016 08 17 regression fixes and warnings removal

warnings: fix --disable-ipv6 --without-ipinfo compilation warnings

Add include <strings.h> to avoid implicit declaration of built-in functions,
replace bzero() with memset(), mark function argument unused, and use %zu to
print size_t data type.

Signed-off-by: Sami Kerola <kerolasa@iki.fi>

build-sys: fix ./configure --disable-ipv6

build-sys: fix --without-ipinfo regressions

The commit e2d898cc16398609821f7a786ae86dcbac4884af was far from perfect it
broke all --without features.  Oh dear me.

Looks like I am not the only one breaking configure.  The --disable-ipv6 has
been broken since commit d90dd1fd8f9289fa9e880c006ca6dff0ef7d4918 and
further damaged by 2cf22b240958b72fb8d84f3ee663f84cbad2c65b.  These issues
will be addressed later.

warnigns: add void to functions that do not take any arguments

The use of function declarators with empty parentheses (not prototype-format
parameter type declarators) is an obsolescent feature.

fixed no-gtk build bug introduced with e2d898cc

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

Merge pull request #136 from kerolasa/2016-08-16

2016 08 16 make build system better

cleanup: remove unnecessary function

readability: always use EXIT_* definitions from stdlib.h

warnings: fix unused variable when ./configure --without-gtk is used

Use of preprocessor to enable/disable code blocks makes messy code.  Perhaps
something can be done in future to make this nicer.

build-sys: require automake 1.11.6 or newer

Do not let anyone to create vulnerable dist package.

Reference: https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2012-3386

build-sys: improve configure.am

Unify indentation, use strict quoting, and use appropriate macros.  Add few
comments that can be found from ./configure script so that one has some idea
what macros are generating the lines one is looking.

build-sys: use build version script from gnulib

This makes distribution tar-balls from snapshot to indicate they are
snapshot versions.  Additionally the version markup ends up in config.h
correct way, so earlier code that was specific to this project can be
removed.

Reference: http://git.savannah.gnu.org/cgit/gnulib.git/tree/build-aux/git-version-gen

build-sys: remove old dist Makefile kludge

Autotools dist works, there is no longer need for this.

build-sys: fix make distcheck

New distribution package can, and should, now be built using 'make
distcheck' facility from autotools.  Personally I would run the following.

git clean -xdf && git reset --hard HEAD &&
./bootstrap.sh && ./configure && make distcheck

warnings: remove dead code

These functions are not used, remove them.

cast: do not downgrade to float when double should be used

Regression from commit 78f019aa9d49be5ada89296a462939738c20c181

Requested-by: rewolff <R.E.Wolff@BitWizard.nl>
Reference: https://github.com/traviscross/mtr/pull/135/files#r74889063

comment: add value range note to initialization

Requested-by: rewolff <R.E.Wolff@BitWizard.nl>
Reference: https://github.com/traviscross/mtr/pull/135#discussion_r74889440

warnings: fix some missed unsigned vs signed variable comparisions

Commit 507092a9099a286f86639d9cd38dcdc1f28117ff fixed almost all of them,
this should be the last one.

Merge pull request #135 from kerolasa/warnings

Warnings branch

warnings: multiply timeval seconds only when the value is small

It should be safe to multiply timeval seconds by million after substracting
sequence timestamp from now.  The reason to postpone multiply operation is
to ensure there is smallest possible change for interger overflow to happen.

warnings: fix couple unsigned vs signed variable comparisions

warnings: mark unused function input variables

warnings: do not take abs() when data type is unsigned

Taking the absolute value of unsigned type 'unsigned int' has no effect.

build-sys: default to ,/configure --enable-silent-rules

This makes noticing compiler warnings easier.

man: use url macro to urls and fix reference manual notations

build-sys; do not use subdirectory object

None-recursive builds are quicker, and less problematic from make(1) object
tracking point of view.  See reference for full explanation.

References: http://miller.emu.id.au/pmiller/books/rmch/

cleanup: remove unnecessary null check

The fmt_ipinfo() returns variable from stack.  In order a stack variable to
point to NULL there has to go so much things wrong that the program cannot
run - effectively this can happen when system memory is so low that even
global variables cannot be allocated.

warnings: fix printf data types

warnings: remove code that cannot be reached

Variable block_split gets value from static constants.  It is impossible to
reach the true if-case, so remote the test can the code block.

warnings: stop reassigning a value before the old one has been used

Merge pull request #134 from kerolasa/kerolasa

new usage() and few cleanups

posix: replace bzero() and index() with modern equivelants

Replace bzero() with memset() and index() with strchr().  The earlier
functions are marked as legacy in posix.

Reference: http://pubs.opengroup.org/onlinepubs/009695399/functions/index.html
Reference: http://pubs.opengroup.org/onlinepubs/009695399/functions/bzero.html

dns: remove unnecessary dns_events() function

No longer needed after 02a9e6193e5dec01198f7824b7ed92f320b4e7a9

warnings: stop variable shadowing

Rename local variables not to conflict at following locations:

asn.c:160:26: warning: declaration of 'txtrec' shadows a global declaration [-Wshadow]
dns.c:351:50: warning: declaration of 'af' shadows a global declaration [-Wshadow]
dns.c:89:40: warning: declaration of 'af' shadows a global declaration [-Wshadow]
gtk.c:213:31: warning: declaration of 'Entry' shadows a global declaration [-Wshadow]
net.c:1408:31: warning: declaration of 'host' shadows a global declaration [-Wshadow]
net.c:1726:38: warning: declaration of 'af' shadows a global declaration [-Wshadow]
net.c:1744:39: warning: declaration of 'af' shadows a global declaration [-Wshadow]
net.c:876:7: warning: declaration of 'sequence' shadows a global declaration [-Wshadow]

usage: add short and long options and descriptions to usage()

Earlier usage did not help users.

warnings: remove unnecessary file

The hello.c was added in commit de3d6bc16efea7e63910cce568498eed0e2cf45a

Fixed pull #133 another way....

increased default unknownhosts #92 #132 #130
(I give in).

format sent and rcvd fields correctly for big numbers #66

Updated NEWS as in v0.87.1

fixed double printout of start time, issue 131

Merge branch 'newdns'

The release script bumped the version number

issue 128: compile should be in .gitignore

Merge pull request #127 from nikolaik/json

Add support for JSON as report output format

Add support for JSON as report output format

This is based on the equivalent code for XML.

Merge pull request #126 from russor/master

add option to set graceperiod

add option to set graceperiod

useful when running a single probe on a network where many hops never
respond

Merge pull request #124 from aquerubin/master

Fix GTK display order.  Update an email address in the GTK credits display.

Merge pull request #125 from aquerubin/newdns

Merge the GTK display order fix and credits update from master into newdns.

Merge branch 'newdns' of https://github.com/aquerubin/mtr into newdns

Merge branch 'master' into newdns

Update Tony's email address in the GTK credits.

Fix Avg and Best column order to match column headers in GTK display.

Merge pull request #123 from aquerubin/newdns

Sync up recent master updates into newdns

Merge updates from branch 'master' into newdns

Merge pull request #122 from h-ume/gtk-aslookup

Add aslookup support to gtk interface

Add aslookup support to gtk interface

Merge pull request #120 from yupeng820921/master

add max-unknown option

add max-unknown option

remove the MAX_UNKNOWN_HOSTS macro, use maxUnknown variable
instead of it, and add --max-unknown option, let it can be
configured at run time.

Merge pull request #118 from zorun/improve_ipinfo

Improve ipinfo

Merge pull request #119 from zorun/cosmetic

Cosmetic cleanup of the option-parsing code

Fix wrap-around bug when displaying IP info (-y option)

Some IP prefixes have an allocation date, but some don't.
For instance:

$ dig +short 49.60.231.80.origin.asn.cymru.com TXT
"6453 | 80.231.0.0/16 | EU | ripencc | 2002-03-20"

$ dig +short 186.83.219.195.origin.asn.cymru.com TXT
"6453 | 195.219.0.0/16 | EU | ripencc |"

In the latter case, the parsing code would miss the last (empty) field
and wrap around to the AS number, producing an inconsistent output
(see below).

While we're at it, also make the parsing logic easier to understand...

Before this patch:

$ mtr --report -y 4 nic.co.jp
Start: Thu Jun  9 19:13:50 2016
HOST: gilead                      Loss%   Snt   Last   Avg  Best  Wrst StDev
  1. 2004-04-05 gatwick.net.cl.ca  0.0%    10    2.0   4.7   2.0   7.1   1.9
  2. 786        cl-wgb.route-west  0.0%    10    0.6  11.3   0.6 104.7  32.8
  3. 786        d-we.c-ce.net.cam  0.0%    10    0.7   0.6   0.5   0.8   0.0
  4. 786        c-ce.b-ec.net.cam  0.0%    10    0.7   6.9   0.3  52.6  16.2
  5. 786        ae1.cambab-rbr1.e  0.0%    10    0.3   0.5   0.3   0.7   0.0
  6. 786        146.97.65.118      0.0%    10    0.7   0.6   0.5   0.8   0.0
  7. 786        146.97.38.17       0.0%    10    3.5   9.3   3.5  17.9   6.2
  8. 786        ae29.londtn-sbr1.  0.0%    10    3.7   4.0   3.5   6.6   0.9
  9. 2002-03-20 ix-ae-11-0.tcore1  0.0%    10    3.8   5.0   3.5  16.8   4.1
 10. 6453       195.219.83.186     0.0%    10    4.3   4.4   3.9   5.6   0.3
 11. 2914       ae-4.r22.londen03  0.0%    10    7.7   8.8   3.9  25.5   6.9
 12. 2914       ae-6.r21.sngpsi05  0.0%    10  185.4 185.8 174.4 226.5  14.9
 13. 2914       ae-20.r31.tokyjp0  0.0%    10  249.9 250.0 249.8 250.4   0.0
 14. 2914       ae-21.r00.tokyjp0  0.0%    10  265.9 265.9 265.6 266.1   0.0
 15. 2000-09-29 ae-0.gmo.tokyjp05  0.0%    10  253.7 252.5 244.7 257.5   4.4
 16. ???        ???               100.0    10    0.0   0.0   0.0   0.0   0.0
 17. 7506       g-o-p-4ee-a01-1-e  0.0%    10  250.3 250.3 250.2 250.5   0.0
 18. 7506       unused-157-7-041-  0.0%    10  257.0 256.9 256.7 257.2   0.0
 19. 7506       157-7-107-115.vir  0.0%    10  258.6 258.4 258.2 258.6   0.0

After this patch:

$ mtr --report -y 4 nic.co.jp
Start: Thu Jun  9 19:12:30 2016
HOST: gilead                      Loss%   Snt   Last   Avg  Best  Wrst StDev
  1. 2004-04-05 gatwick.net.cl.ca  0.0%    10    3.6   7.3   1.2  18.1   5.9
  2.            cl-wgb.route-west  0.0%    10    0.6   0.8   0.5   1.3   0.0
  3.            d-we.c-ce.net.cam  0.0%    10    0.5   0.7   0.5   1.0   0.0
  4.            c-ce.b-ec.net.cam  0.0%    10    0.4   1.6   0.3   9.1   2.6
  5.            ae1.cambab-rbr1.e  0.0%    10    0.4   0.5   0.3   0.7   0.0
  6.            146.97.65.118      0.0%    10    0.5   1.1   0.5   5.0   1.3
  7.            146.97.38.17       0.0%    10    3.7   8.4   3.4  51.9  15.3
  8.            ae29.londtn-sbr1.  0.0%    10    3.7   9.0   3.5  54.8  16.1
  9. 2002-03-20 ix-ae-11-0.tcore1  0.0%    10    5.5   4.1   3.5   6.0   0.8
 10.            195.219.83.186     0.0%    10    4.0   4.2   3.9   4.6   0.0
 11.            ae-4.r22.londen03  0.0%    10    3.8   9.5   3.8  33.6  11.6
 12.            ae-6.r21.sngpsi05 20.0%    10  185.8 181.0 179.5 185.8   2.1
 13.            ae-20.r31.tokyjp0  0.0%    10  249.9 249.7 249.5 249.9   0.0
 14.            ae-21.r00.tokyjp0  0.0%    10  266.1 266.0 265.7 266.4   0.0
 15. 2000-09-29 ae-0.gmo.tokyjp05  0.0%    10  243.8 243.8 243.4 244.4   0.0
 16. ???        ???               100.0    10    0.0   0.0   0.0   0.0   0.0
 17.            g-o-p-4ee-a01-1-e  0.0%    10  246.1 246.2 245.7 247.4   0.0
 18.            unused-157-7-041-  0.0%    10  257.3 258.0 256.9 260.9   1.2
 19.            157-7-107-115.vir  0.0%    10  258.3 258.4 258.2 258.8   0.0

Cosmetic cleanup of the option-parsing code

Align everything nicely, update helpful comments, and use NULL instead
of 0 for the third element of long_options: it's a pointer, see getopt(3).

Document the -y option in the manpage

Allow enabling IP info and ASN lookup from the curses interface

Toggling IP info and ASN lookup from the curses interface can be done
using 'z' or 'y'.  But it was only allowed if it was first enabled via
a command-line switch (either -z or -y).

Merge pull request #115 from aquerubin/newdns

Use setcap instead of setuid when installing the binary.

Merge pull request #114 from aquerubin/master

Use setcap instead of setuid when installing the binary.

Merge remote-tracking branch 'origin/master' into newdns

Use setcap instead of setuid when installing the binary.

Merge pull request #113 from aquerubin/newdns

Need to error check getnameinfo().

Merge pull request #112 from aquerubin/master

Correct psize for IPv6.

Need to error check getnameinfo().

Correct psize for IPv6.

Merge pull request #111 from aquerubin/newdns

Merge master into newdns

Merge remote-tracking branch 'origin/master' into newdns

Conflicts:
dns.c