On Tue, Apr 05, 2011 at 04:57:30PM +0100, Stuart Henderson wrote:
> On 2011/04/05 16:51, Stuart Henderson wrote:
> > if -t is used, display a notice when the TOS changes en-route.
> > ok?
> 
> oh, it's better with a (contrived) example:
> 
> $ traceroute -nt 7 naiad
> traceroute to naiad.spacehopper.org (195.95.187.35), 64 hops max, 40 byte 
> packets
>  1  85.158.44.145  0.441 ms (TOS=33!)  0.328 ms  0.396 ms
>  2  193.178.223.245  14.637 ms  15.213 ms  15.466 ms
>  3  194.39.143.145  16.600 ms  15.468 ms  16.206 ms
>  4  194.39.143.205  16.483 ms  16.210 ms  15.843 ms
>  5  193.203.5.182  17.475 ms  16.708 ms  16.986 ms
>  6  195.95.187.248  17.211 ms (TOS=4!)  16.460 ms  16.468 ms
>  7  195.95.187.35  27.99 ms  16.207 ms  17.467 ms
> 

Diff looks good. What I don't like to specify and display the TOS as
decimal since I have no idea what decimal numbers are which TOS/DiffServ
Code points. No that is porbably a different diff, so OK claudio@ for this
one.

> 
> 
> > Index: traceroute.8
> > ===================================================================
> > RCS file: /cvs/src/usr.sbin/traceroute/traceroute.8,v
> > retrieving revision 1.44
> > diff -u -p -r1.44 traceroute.8
> > --- traceroute.8    8 Jul 2010 20:23:03 -0000       1.44
> > +++ traceroute.8    5 Apr 2011 15:50:38 -0000
> > @@ -177,6 +177,8 @@ in probe packets to the following value 
> >  The value must be a decimal integer in the range 0 to 255.
> >  This option can be used to
> >  see if different types-of-service result in different paths.
> > +If this option is used, changes to the type-of-service in the
> > +returned packets are displayed.
> >  (If you are not running a
> >  .Bx 4.3 tahoe
> >  or later system, this may be academic since the normal network
> > @@ -384,6 +386,8 @@ ever occur and the associated gateway is
> >  (destination network or host unreachable for TOS),
> >  .Sy !<code>
> >  (other ICMP unreachable code).
> > +.Sy TOS=xxx
> > +(TOS bit in returned packet differs from last hop).
> >  If almost all the probes result in some kind of unreachable,
> >  .Nm
> >  will give up and exit.
> > Index: traceroute.c
> > ===================================================================
> > RCS file: /cvs/src/usr.sbin/traceroute/traceroute.c,v
> > retrieving revision 1.74
> > diff -u -p -r1.74 traceroute.c
> > --- traceroute.c    22 Mar 2011 10:16:23 -0000      1.74
> > +++ traceroute.c    5 Apr 2011 15:50:38 -0000
> > @@ -293,11 +293,13 @@ main(int argc, char *argv[])
> >     int mib[4] = { CTL_NET, PF_INET, IPPROTO_IP, IPCTL_DEFTTL };
> >     int ttl_flag = 0, incflag = 1, protoset = 0, sump = 0;
> >     int ch, i, lsrr = 0, on = 1, probe, seq = 0, tos = 0;
> > +   int last_tos, tos_returned;
> >     size_t size = sizeof(max_ttl);
> >     struct sockaddr_in from, to;
> >     struct hostent *hp;
> >     u_int32_t tmprnd;
> > -   struct ip *ip;
> > +   struct ip *ip, *inner_ip;
> > +   struct icmp *icp;
> >     u_int8_t ttl;
> >     char *ep;
> >     const char *errstr;
> > @@ -427,7 +429,7 @@ main(int argc, char *argv[])
> >                     l = strtol(optarg, &ep, 10);
> >                     if (errno || !*optarg || *ep || l < 0 || l > 255)
> >                             errx(1, "tos must be 0 to 255.");
> > -                   tos = (int)l;
> > +                   last_tos = tos = (int)l;
> >                     break;
> >             case 'v':
> >                     verbose++;
> > @@ -636,9 +638,21 @@ main(int argc, char *argv[])
> >                                     ++got_there;
> >                                     break;
> >                             }
> > +
> > +                           icp = (struct icmp *) (((u_char 
> > *)ip)+(ip->ip_hl<<2));
> > +                           inner_ip = (struct ip *) (((u_char *)icp)+8);
> > +
> > +                           tos_returned = inner_ip->ip_tos;
> > +
> > +                           if (tos_returned != last_tos)
> > +                                   printf (" (TOS=%d!)", tos_returned);
> > +
> > +                           last_tos = tos_returned;
> > +
> >                             /* time exceeded in transit */
> >                             if (i == -1)
> >                                     break;
> > +
> >                             code = i - 1;
> >                             switch (code) {
> >                             case ICMP_UNREACH_PORT:
> 

-- 
:wq Claudio

Reply via email to