Mtr - Network monitoring tool

asn lookup seems to be broken

Reported by Yakovetsky VS on 2013-04-12
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mtr
Undecided
Unassigned

Bug Description

Could you please check this out?

fix:
--- asn.c.orig
+++ asn.c
@@ -85,6 +85,7 @@
     }

     pt += INT16SZ; /* class */
+ pt += INT32SZ; /* ttl */
     GETSHORT(size, pt);
     txtlen = *pt;

Antonio Querubin (aaq) wrote :

See patch filed under bug #1160321.

Hi,
I also filled patch under bug https://bugs.launchpad.net/mtr/+bug/701514
Some details:
- also originX.asn.cymru.com <http://originx.asn.cymru.com/> instead of
asn.routeviews.org
  (originX.asn.cymru.com <http://originx.asn.cymru.com/> is documented to
map both IPv4 and IPv6 addresses)
- libc hash instead of glib hash
  (the hcreate() and hsearch() functions conform to XPG4.2 and POSIX.1-2001)
- optional building (configure --without-ipinfo)
- extended info (Prefix, Country, etc.)
- all text modes (curses, curses-graph, report, report-wide)
- switching keys (y and z).

Kind Regards.

2013/4/14 Antonio Querubin <email address hidden>

> See patch filed under bug #1160321.
>
> --
> You received this bug notification because you are subscribed to the bug
> report.
> https://bugs.launchpad.net/bugs/1168367
>
> Title:
> asn lookup seems to be broken
>
> Status in Mtr - Network monitoring tool:
> New
>
> Bug description:
> Could you please check this out?
>
> fix:
> --- asn.c.orig
> +++ asn.c
> @@ -85,6 +85,7 @@
> }
>
> pt += INT16SZ; /* class */
> + pt += INT32SZ; /* ttl */
> GETSHORT(size, pt);
> txtlen = *pt;
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/mtr/+bug/1168367/+subscriptions
>

Yakovetsky VS (vsyakovetsky) wrote :

Oops, Sorry!
- - all text modes (curses, curses-graph, report, report-wide)
+ this patch works in next display modes: curses, curses-graph, report, report-wide.

+ I tested on the following platforms IPv4: netbsd-6.0/i386, freebsd-9.1/amd64, ubuntu-12.10/amd64,
+ and IPv6 (via IPv6 tunnel): ubuntu-12.10/amd64.

Please excuse my bad English.

Changed in mtr:
status: New → Fix Released
Yakovetsky VS (vsyakovetsky) wrote :

commit: 03f3da5
release: 0.85

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers