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

net.c fix from AQ.

Merge pull request #110 from russor/master

allow udp traceroute with fixed source and destination ports

correct checksum calculation when adding the overflow overflows

you can see this with a bit pattern of FF and a large packet length

copy odd byte into a 16-bit temp value; used bit-sized types for calrity

endian neutral placement of alternate checksum

fix placement of zeros when running alternate udp checksum

fix improper aliasing

automatically set udp address if needed

set the local address for display if it was bound

fix checksum for odd sized packets

allow setting local and remote port for UDP probing

This allows for testing over a single network path in a load balanced
network where flows are hashed on the ips and ports (relatively common), the
default behavior is to use the destination port as the sequence number when
doing UDP probes, so you see (more or less) an average of all the paths.

If the remote port is set, we need to use the checksum field to store
sequence and adjust the paylod so the checksum is valid; therefore we need
to have the source address, as it's part of the checksum calculation.  We
also need to increase the minimum packet size so we have at least 2 bytes of
payload.

Merge pull request #103 from jwilk/master

Fix typos

Fix typos.

Merge pull request #102 from tbaschak/32bitasns

Update asn.c - 32bit ASN widths

Update asn.c - 32bit asn widths

Now that 32-bit ASNs are becoming more common the output of -z should take their width into consideration.

Merge pull request #101 from trittweiler/master

Fix setting length field of UDP header to broken value on BSD systems.

Fix setting length field of UDP header to broken value on BSD systems.

As per http://stackoverflow.com/a/15881825 which references the manpage
ip(4) on *BSD and MacOSX:

  "The ip_len and ip_off fields must be provided in host byte order.
   All other fields must be provided in network byte order."

Before this commit, mtr would incorrectly set the length field of the UDP
header in host byte order after the first packet, as can be witnessed in
the following excerpt:

1st packet (TTL=1):

IP (tos 0x0, ttl 1, id 51530, offset 0, flags [none], proto UDP (17), length 64)
    192.168.178.61.1143 > 8.8.8.8.33000: UDP, *** length 36 ***
IP (tos 0xc0, ttl 64, id 34040, offset 0, flags [none], proto ICMP (1), length 92)
    192.168.178.1 > 192.168.178.61: ICMP time exceeded in-transit, length 72
      IP (tos 0x0, ttl 1, id 51530, offset 0, flags [none], proto UDP (17), length 64)
         192.168.178.61.1143 > 8.8.8.8.33000: UDP, length 36

2nd packet (TTL=2):

IP (tos 0x0, ttl 2, id 17256, offset 0, flags [none], proto UDP (17), length 64)
    192.168.178.61.1143 > 8.8.8.8.33001: UDP, *** bad length 11256 > 36 ***
IP (tos 0xc0, ttl 254, id 6765, offset 0, flags [none], proto ICMP (1), length 56)
    62.52.201.192 > 192.168.178.61: ICMP time exceeded in-transit, length 36
      IP (tos 0x0, ttl 1, id 17256, offset 0, flags [none], proto UDP (17), length 64)
         192.168.178.61.1143 > 8.8.8.8.33001: UDP, bad length 11256 > 36

3rd packet (TTL=3):

IP (tos 0x0, ttl 3, id 54343, offset 0, flags [none], proto UDP (17), length 64)
    192.168.178.61.1143 > 8.8.8.8.33002: UDP, *** bad length 11256 > 36 ***
IP (tos 0x0, ttl 253, id 0, offset 0, flags [none], proto ICMP (1), length 56)
    62.53.22.220 > 192.168.178.61: ICMP time exceeded in-transit, length 36
      IP (tos 0x0, ttl 1, id 54343, offset 0, flags [none], proto UDP (17), length 64)
         192.168.178.61.1143 > 8.8.8.8.33002: UDP, bad length 11256 > 36

Merge pull request #100 from trittweiler/master

Fix and improvements for --csv

asn.h: Guard against being included twice.

In order to surpress a warning on clang.

--csv: Print a header line as the first line which names all columns.

--csv: Don't print spaces in columns.

Fix typo in csv_close() that prevented any of the data columns from being printed.

Merge pull request #98 from garethrandall/doc-updates

State that Github is the preferred way to report bugs.

State that Github is the preferred way to report bugs.

Merge pull request #91 from garethrandall/master

Some improvements to the documentation.

Merge pull request #93 from Narthorn/master

curses: Fix background transparency in terminal

curses: Fix background transparency in terminal

Patch comes from, and closes traviscross/mtr#72.

Add a section about granting limited security capabilities.

Fix typos and update mailing list references.

Merge pull request #90 from garethrandall/master

Remove a warning message at compile time.

Remove a warning message at compile time.
Warning was produced when running:
$ ./configure --without-gtk
$ make
display.c: In function ‘display_keyaction’:
display.c:169:5: warning: ‘return’ with no value, in function returning non-void [-Wreturn-type]
     return mtr_curses_keyaction();
     ^

Merge pull request #85 from culugyx/x-tag-r1

Raw output: add x for a ping-packet-sent event, add sequence numbers to the replies.

Raw output: add x for a ping-packet-sent event.

This is needed for loss rate calculation.
Supports ICMP/UDP/TCP.
Also adds sequence number to pingline.

Merge pull request #84 from garethrandall/master

Filled in some of the missing man page sections.

Filled in some of the missing man page sections.

Merge pull request #81 from swordfeng/sctp-support

Add SCTP support by swordfeng

fix sctp header structure

remove comment

Add SCTP support (same way with tcp)

changed the name of the ping timout timer from 'tag' to 'ping timeout timer'

Merge pull request #79 from kurkav/pauseFix

Fixed behaviour of Pause button

Fixed behaviour of Pause button

Merge pull request #77 from dhduvall/master

Fix issue #76: rationalize the discovery of a terminal handling library

Fix issue #76: rationalize the discovery of a terminal handling library

modified name of timeout variable to prevent warning on solaris.

fix for printing space field in XML.

Merge pull request #71 from aquerubin/newdns

Correct psize bug for IPv6. -- Thanks!

Correct psize for IPv6.

Merge pull request #70 from kiwano/master

Added --displaymode option

Added --displaymode option

added use-default-colors...

removed warning about IPV6 socket when IPV6 is not available at runtime

removed AC check for features newdns doesn't use

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

explanation of the version numbers in NEWS.

Merge pull request #63 from garethrandall/readme-fixes

Corrected the "without gtk" reference to "./configure --without-gtk", be...

Corrected the "without gtk" reference to "./configure --without-gtk", because the previous version "--WITHOUT_GTK" caused an error.
The error message was:
    configure: error: unrecognized option: `--WITHOUT_GTK'
Also fixed a couple of README typos.

The release script bumped the version number

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

Merge pull request #61 from dmbaturin/master

Make XML report output well formed.

Make XML report output well formed.
Remove % from Loss tag name, add quotes around attribute names,
add XML header.

fix ping graph bug

One more patch to fix a getnameinfo corruption problem. -- AQ

removed last debug output from dns.c

AQ: Added include for redhat, and fixed salen for BSD

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

merged antonios's bufsize fixes

Merge pull request #58 from elindsey/master

document which version added TCP support

document which version added TCP support

removed the include mess...

moved towards IPV6 compatibilty...

New DNS works for IPV4....

Merge pull request #57 from hloeung/master

Revert previous set of changes that turned out to be "debian-only".