Subject: Re: rt_tables broken
To: None <dyoung@pobox.com>
From: KIYOHARA Takashi <kiyohara@kk.iij4u.or.jp>
List: current-users
Date: 08/07/2007 11:18:14
Hi! dyoung,


From: David Young <dyoung@pobox.com>
Date: Mon, 6 Aug 2007 14:26:52 -0500

> On Tue, Aug 07, 2007 at 03:41:29AM +0900, KIYOHARA Takashi wrote:

> > From: David Young <dyoung@pobox.com>
> > Date: Sun, 5 Aug 2007 15:07:27 -0500
> > 
> > > On Sat, Aug 04, 2007 at 03:25:13AM -0500, David Young wrote:
> > > > On Sat, Aug 04, 2007 at 03:47:12PM +0900, KIYOHARA Takashi wrote:

> > > > > The rt_tables was broken since 07/19/2007 20:49:00 perhaps.  X-<
> > > > > 
> > > > >   http://mail-index.netbsd.org/source-changes/2007/07/19/0032.html
> > > > 
> > > > The bug is not new since 19 July 2007.  The kernel has been sloppy
> > > > about copying link-layer addresses for a long time.  fwip(4) suffers
> > > > the buffer overflow because its link-layer addresses are the longest.
> > > > I have attached a patch that shows an approach to a fix for the bugs.
> > > > I leave it to you to make the patch compile and run. :-)
> > > 
> > > Here is a new patch, also untested.  Lengthening sockaddr_dl is still
> > > left as an exercise for the reader. :-)
> > 
> > I tried your patch.  However repeated the message.  X-<
> > 
> >   sockaddr_dl_setaddr: no room for 16-byte media address
> > 
> > 
> > I think that the return value of sockaddr_dup() that is called in
> > rt_setgate() is a problem.  This is a value of pool(9).
> > Can you resize it?
> > 
> > Do you keep it to a more longer size beforehand?
> 
> Try again with a longer sockaddr_dl.  For the size of sdl_data, I
> recommend IFNAMSIZ+16 bytes

I have already expanded it to 22bytes.  It is working now.

Thanks,
--
kiyohara