re: bin/44889

[matthew sporleder <msporleder%gmail.com@localhost>]

The following reply was made to PR bin/44889; it has been noted by GNATS.

From: matthew sporleder <msporleder%gmail.com@localhost>
To: gnats-bugs%netbsd.org@localhost, gnats-admin%netbsd.org@localhost
Cc: 
Subject: re: bin/44889
Date: Sun, 12 Feb 2012 09:52:18 -0500

 From elad in irc:
 
 Index: if.c
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
 =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
 RCS file: /cvsroot/src/usr.bin/netstat/if.c,v
 retrieving revision 1.72
 diff -u -p -r1.72 if.c
 --- if.c =A0 =A0 =A0 =A016 Sep 2011 15:39:27 -0000 =A0 =A0 =A01.72
 +++ if.c =A0 =A0 =A0 =A012 Feb 2012 03:54:15 -0000
 @@ -83,7 +83,8 @@ struct =A0 =A0 =A0 =A0iftot {
 =A0 =A0 =A0 =A0int ift_dr; =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 /* drops=
  */
 =A0};
 
 -static void print_addr(struct sockaddr *, struct sockaddr **, struct
 if_data *);
 +static void print_addr(struct sockaddr *, struct sockaddr **, struct if_da=
 ta *,
 + =A0 =A0struct ifnet *);
 =A0static void sidewaysintpr(u_int, u_long);
 
 =A0static void iftot_banner(struct iftot *);
 @@ -220,7 +221,7 @@ intpr_sysctl(void)
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0}
 
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0printf("%-5s %-5" PRIu64, na=
 me, ifd->ifi_mtu);
 - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 print_addr(rti_info[RTAX_IFP]=
 , rti_info, ifd);
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 print_addr(rti_info[RTAX_IFP]=
 , rti_info, ifd, NULL);
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0break;
 
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0case RTM_NEWADDR:
 @@ -238,24 +239,25 @@ intpr_sysctl(void)
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0get_rtaddrs(ifam->ifam_addrs=
 , sa, rti_info);
 
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0printf("%-5s %-5" PRIu64, na=
 me, ifd->ifi_mtu);
 - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 print_addr(rti_info[RTAX_IFA]=
 , rti_info, ifd);
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 print_addr(rti_info[RTAX_IFA]=
 , rti_info, ifd, NULL);
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0break;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0}
 =A0 =A0 =A0 =A0}
 =A0}
 -
 -static void
 -intpr_kvm(u_long ifnetaddr, void (*pfunc)(const char *))
 -{
 - =A0 =A0 =A0 struct ifnet ifnet;
 - =A0 =A0 =A0 union {
 +union ifaddr_u {
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0struct ifaddr ifa;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0struct in_ifaddr in;
 =A0#ifdef INET6
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0struct in6_ifaddr in6;
 =A0#endif /* INET6 */
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0struct iso_ifaddr iso;
 - =A0 =A0 =A0 } ifaddr;
 +};
 +
 +static void
 +intpr_kvm(u_long ifnetaddr, void (*pfunc)(const char *))
 +{
 + =A0 =A0 =A0 struct ifnet ifnet;
 + =A0 =A0 =A0 union ifaddr_u ifaddr;
 =A0 =A0 =A0 =A0u_long ifaddraddr;
 =A0 =A0 =A0 =A0struct ifnet_head ifhead; =A0 =A0 =A0 /* TAILQ_HEAD */
 =A0 =A0 =A0 =A0char name[IFNAMSIZ + 1]; =A0 =A0 =A0 =A0/* + 1 for `*' */
 @@ -321,7 +323,7 @@ intpr_kvm(u_long ifnetaddr, void (*pfunc
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0cp =3D (CP(ifaddr.ifa.ifa_ad=
 dr) - CP(ifaddraddr)) +
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0CP(&ifaddr);
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0sa =3D (struct sockaddr *)cp=
 ;
 - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 print_addr(sa, (void *)&ifadd=
 r, &ifnet.if_data);
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 print_addr(sa, (void *)&ifadd=
 r, &ifnet.if_data, &ifnet);
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0}
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0ifaddraddr =3D (u_long)ifaddr.ifa.ifa_list.t=
 qe_next;
 =A0 =A0 =A0 =A0}
 @@ -329,7 +331,8 @@ intpr_kvm(u_long ifnetaddr, void (*pfunc
 =A0}
 
 =A0static void
 -print_addr(struct sockaddr *sa, struct sockaddr **rtinfo, struct if_data *=
 ifd)
 +print_addr(struct sockaddr *sa, struct sockaddr **rtinfo, struct if_data *=
 ifd,
 + =A0 =A0struct ifnet *ifnet)
 =A0{
 =A0 =A0 =A0 =A0char hexsep =3D '.'; =A0 =A0 =A0 =A0 =A0 =A0 =A0/* for hexpr=
 int */
 =A0 =A0 =A0 =A0static const char hexfmt[] =3D "%02x%c"; =A0/* for hexprint =
 */
 @@ -380,13 +383,14 @@ print_addr(struct sockaddr *sa, struct s
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0n =3D 17;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0printf("%-*.*s ", n, n, cp);
 
 -#if 0 /* XXX-elad */
 - =A0 =A0 =A0 =A0 =A0 =A0 =A0 if (aflag) {
 +#if 1 /* XXX-elad */
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 if (aflag && ifnet) {
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0u_long multiaddr;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0struct in_multi inm;
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 union ifaddr_u *ifaddr =3D (u=
 nion ifaddr_u *)rtinfo;
 
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0multiaddr =3D (u_long)
 - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 ifaddr.in.ia_multiadd=
 rs.lh_first;
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 ifaddr->in.ia_multiad=
 drs.lh_first;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0while (multiaddr !=3D 0) {
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0kread(multia=
 ddr, (char *)&inm,
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 sizeof =
 inm);
 @@ -397,7 +401,7 @@ print_addr(struct sockaddr *sa, struct s
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 (u_long=
 )inm.inm_list.le_next;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0}
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0}
 -#endif /* 0 */
 +#endif /* 1 */
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0break;
 =A0#ifdef INET6
 =A0 =A0 =A0 =A0case AF_INET6:
 @@ -441,14 +445,15 @@ print_addr(struct sockaddr *sa, struct s
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0n =3D 17;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0printf("%-*.*s ", n, n, cp);
 
 -#if 0 /* XXX-elad */
 - =A0 =A0 =A0 =A0 =A0 =A0 =A0 if (aflag) {
 +#if 1 /* XXX-elad */
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 if (aflag && ifnet) {
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0u_long multiaddr;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0struct in6_multi inm;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0struct sockaddr_in6 as6;
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 union ifaddr_u *ifaddr =3D (u=
 nion ifaddr_u *)rtinfo;
 
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0multiaddr =3D (u_long)
 - =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 ifaddr.in6.ia6_multia=
 ddrs.lh_first;
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 ifaddr->in6.ia6_multi=
 addrs.lh_first;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0while (multiaddr !=3D 0) {
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0kread(multia=
 ddr, (char *)&inm,
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 sizeof =
 inm);
 @@ -484,7 +489,7 @@ print_addr(struct sockaddr *sa, struct s
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 (u_long=
 )inm.in6m_entry.le_next;
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0}
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0}
 -#endif /* 0 */
 +#endif /* 1 */
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0break;
 =A0#endif /*INET6*/
 =A0#ifndef SMALL
 @@ -548,9 +553,9 @@ print_addr(struct sockaddr *sa, struct s
 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0 =A0(unsigned long long)ifd->ifi=
 _collisions);
 =A0 =A0 =A0 =A0}
 =A0 =A0 =A0 =A0if (tflag)
 - =A0 =A0 =A0 =A0 =A0 =A0 =A0 printf(" %4d", 0 /* XXX-elad ifnet.if_timer *=
 /);
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 printf(" %4d", ifnet ? ifnet->if_timer : 0); =
 /* XXX-elad */
 =A0 =A0 =A0 =A0if (dflag)
 - =A0 =A0 =A0 =A0 =A0 =A0 =A0 printf(" %5d", 0 /* XXX-elad ifnet.if_snd.ifq=
 _drops */);
 + =A0 =A0 =A0 =A0 =A0 =A0 =A0 printf(" %5d", ifnet ? ifnet->if_snd.ifq_drop=
 s : 0);
 /* XXX-elad */
 =A0 =A0 =A0 =A0putchar('\n');
 =A0}