From owner-svn-src-all@FreeBSD.ORG Thu Jul 18 02:36:50 2013 Return-Path: Delivered-To: svn-src-all@FreeBSD.org Received: from mx1.freebsd.org (mx1.freebsd.org [8.8.178.115]) by hub.freebsd.org (Postfix) with ESMTP id 01A472AE; Thu, 18 Jul 2013 02:36:50 +0000 (UTC) (envelope-from hrs@FreeBSD.org) Received: from mail.allbsd.org (gatekeeper.allbsd.org [IPv6:2001:2f0:104:e001::32]) by mx1.freebsd.org (Postfix) with ESMTP id ADA64CC7; Thu, 18 Jul 2013 02:36:48 +0000 (UTC) Received: from alph.d.allbsd.org (p2049-ipbf1102funabasi.chiba.ocn.ne.jp [122.26.101.49]) (authenticated bits=128) by mail.allbsd.org (8.14.5/8.14.5) with ESMTP id r6I2aUFg049281 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-SHA bits=256 verify=NO); Thu, 18 Jul 2013 11:36:40 +0900 (JST) (envelope-from hrs@FreeBSD.org) Received: from localhost (localhost [IPv6:::1]) (authenticated bits=0) by alph.d.allbsd.org (8.14.5/8.14.5) with ESMTP id r6I2aSDc011097; Thu, 18 Jul 2013 11:36:30 +0900 (JST) (envelope-from hrs@FreeBSD.org) Date: Thu, 18 Jul 2013 11:36:17 +0900 (JST) Message-Id: <20130718.113617.241320635949438644.hrs@allbsd.org> To: gjb@FreeBSD.org, scott4long@yahoo.com Subject: Re: svn commit: r253427 - head/sbin/route From: Hiroki Sato In-Reply-To: <2AE597F6-02A5-405F-B1D8-34054A0DA1C4@yahoo.com> References: <201307171405.r6HE5KpR005269@svn.freebsd.org> <20130718010038.GD2264@glenbarber.us> <2AE597F6-02A5-405F-B1D8-34054A0DA1C4@yahoo.com> X-PGPkey-fingerprint: BDB3 443F A5DD B3D0 A530 FFD7 4F2C D3D8 2793 CF2D X-Mailer: Mew version 6.5 on Emacs 24.3 / Mule 6.0 (HANACHIRUSATO) Mime-Version: 1.0 Content-Type: Multipart/Signed; protocol="application/pgp-signature"; micalg=pgp-sha1; boundary="--Security_Multipart(Thu_Jul_18_11_36_17_2013_082)--" Content-Transfer-Encoding: 7bit X-Virus-Scanned: clamav-milter 0.97.4 at gatekeeper.allbsd.org X-Virus-Status: Clean X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.2.7 (mail.allbsd.org [133.31.130.32]); Thu, 18 Jul 2013 11:36:40 +0900 (JST) X-Spam-Status: No, score=-89.3 required=13.0 tests=CONTENT_TYPE_PRESENT, DIRECTOCNDYN,DYN_PBL,RCVD_IN_PBL,RCVD_IN_RP_RNBL,SPF_SOFTFAIL, USER_IN_WHITELIST autolearn=no version=3.3.2 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on gatekeeper.allbsd.org Cc: svn-src-head@FreeBSD.org, svn-src-all@FreeBSD.org, src-committers@FreeBSD.org X-BeenThere: svn-src-all@freebsd.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "SVN commit messages for the entire src tree \(except for " user" and " projects" \)" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 18 Jul 2013 02:36:50 -0000 ----Security_Multipart(Thu_Jul_18_11_36_17_2013_082)-- Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Scott Long wrote in <2AE597F6-02A5-405F-B1D8-34054A0DA1C4@yahoo.com>: sc> sc> On Jul 17, 2013, at 7:00 PM, Glen Barber wrote: sc> sc> > On Wed, Jul 17, 2013 at 02:05:20PM +0000, Hiroki Sato wrote: sc> >> Author: hrs sc> >> Date: Wed Jul 17 14:05:20 2013 sc> >> New Revision: 253427 sc> >> URL: http://svnweb.freebsd.org/changeset/base/253427 sc> >> sc> >> Log: sc> >> - Add support of MK_INET_SUPPORT=no. sc> >> - Fix a bug in sodump() which prevented struct sockaddr_in6 from sc> >> - displaying. sc> >> - Fix a bug in in fiboptlist_csv() which could cause free() of sc> >> - uninitialized sc> >> pointer. sc> >> - Style cleanups: sc> >> . Add missing "static" keywords. sc> >> . Use an array of struct sockaddr_storage instead of sockunion for sc> >> rtmsg. sc> >> . Use err() and errx() instead of pair of fprintf(stderr, "...") + sc> >> exit(1). sc> >> . Use nitems() macro. sc> >> . Various style(9) fixes. sc> >> sc> >> Modified: sc> >> head/sbin/route/Makefile sc> >> head/sbin/route/route.c sc> >> sc> > sc> > [...] sc> > sc> >> Modified: head/sbin/route/route.c sc> >> ============================================================================== sc> >> --- head/sbin/route/route.c Wed Jul 17 14:04:18 2013 (r253426) sc> >> +++ head/sbin/route/route.c Wed Jul 17 14:05:20 2013 (r253427) sc> > [...] sc> >> @@ -1470,14 +1498,12 @@ rtmsg(int cmd, int flags, int fib) sc> >> rtm.rtm_rmx = rt_metrics; sc> >> rtm.rtm_inits = rtm_inits; sc> >> sc> >> - if (rtm_addrs & RTA_NETMASK) sc> >> - mask_addr(); sc> >> - NEXTADDR(RTA_DST, so_dst); sc> >> - NEXTADDR(RTA_GATEWAY, so_gate); sc> >> - NEXTADDR(RTA_NETMASK, so_mask); sc> >> - NEXTADDR(RTA_GENMASK, so_genmask); sc> >> - NEXTADDR(RTA_IFP, so_ifp); sc> >> - NEXTADDR(RTA_IFA, so_ifa); sc> >> + NEXTADDR(RTA_DST, so[RTAX_DST]); sc> >> + NEXTADDR(RTA_GATEWAY, so[RTAX_GATEWAY]); sc> >> + NEXTADDR(RTA_NETMASK, so[RTAX_NETMASK]); sc> >> + NEXTADDR(RTA_GENMASK, so[RTAX_GENMASK]); sc> >> + NEXTADDR(RTA_IFP, so[RTAX_IFP]); sc> >> + NEXTADDR(RTA_IFA, so[RTAX_IFA]); sc> >> rtm.rtm_msglen = l = cp - (char *)&m_rtmsg; sc> >> if (verbose) sc> >> print_rtmsg(&rtm, l); sc> > sc> > I think this breaks head/, at least for ia64. sc> > sc> sc> Yes, as well as for amd64. This breakage occurred at the exact moment sc> that we tried to cut over to HEAD at Netflix. It's indeed a Brave New sc> World. Should we back it out? Sorry, should be fixed just now. It seems I need more sleep. -- Hiroki ----Security_Multipart(Thu_Jul_18_11_36_17_2013_082)-- Content-Type: application/pgp-signature Content-Transfer-Encoding: 7bit -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.13 (FreeBSD) iEYEABECAAYFAlHnVKEACgkQTyzT2CeTzy3M0wCgqPWOZHk3ztiRNyUZlqHjdTE/ kIkAnjnVfKCn/nPU0aQZ5/nUJZiDSm1I =E12z -----END PGP SIGNATURE----- ----Security_Multipart(Thu_Jul_18_11_36_17_2013_082)----