Date: Sat, 23 Nov 2002 13:32:53 -0800 (PST) From: Nate Lawson <nate@root.org> To: Brooks Davis <brooks@one-eyed-alien.net> Cc: hackers@FreeBSD.ORG Subject: Re: [brooks@one-eyed-alien.net: [PATCH] switching to if_xname] Message-ID: <Pine.BSF.4.21.0211231332421.77310-100000@root.org> In-Reply-To: <20021123125056.A10727@Odin.AC.HMC.Edu>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 23 Nov 2002, Brooks Davis wrote: > On Tue, Nov 19, 2002 at 02:15:34PM -0800, Nate Lawson wrote: > > On Tue, 19 Nov 2002, Brooks Davis wrote: > > > On Fri, Nov 01, 2002 at 10:54:23AM -0800, Brooks Davis wrote: > > > > I'm trying to get some review for the following patch. I realize it's > > > > quite large, but most it is is trivial. The ipfw code is the only thing > > > > that worries me significantly. I have promised Kris that I will fix > > > > ports that break with this change so you don't need to worry about that > > > > issue. > > > > > > I've uploaded a new version of the patch addressing the concerns I've > > > heard to: > > > > > > http://people.freebsd.org/~brooks/patches/if_xname.diff > > > > Thanks for fixing things. Are all the uses of sprintf() below safe and > > intentional? > > No, they weren't intentional. I managed to forget to scan for those. > Most of them were safe, but I've removed all of them. (Grep still hits > a few due to ipf using sprintf). I've uploaded a new version. I'm now > testing a world build on panther. Looks good to me now. -Nate To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-hackers" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?Pine.BSF.4.21.0211231332421.77310-100000>