On Monday, March 05, 2012 1:18:03 pm rank1seeker@gmai... wrote:
> ----- Original Message -----
> From: John Baldwin <jhb@free...>
> To: rank1seeker@gmai...
> Cc: hackers@free..., "Roman Divacky" <rdivacky@free...>, "Andriy Gapon" <avg@free...>
> Date: Mon, 5 Mar 2012 12:44:44 -0500
> Subject: Re: BUG: 9.0 stage 2 boot (/boot/boot)
>
> > On Saturday, March 03, 2012 3:59:19 am rank1seeker@gmai... wrote:
> > > Using:  MD5 (boot2_test.patch) = c2d6d2c9806df5915419987f200c05af
> > >
> > > Still fails to build.
> > >
> > > /usr/src/sys/boot/i386/boot2/boot2.c: In function 'main':
> > > /usr/src/sys/boot/i386/boot2/boot2.c:282: error: expected ':' before ')' token
> >
> > My test build didn't actually build all of boot2.  This should be fixed now.
> >
> > --
> > John Baldwin
>
>
> Using:  MD5 (boot2_test.patch) = b06e290859041cefc8ab38df37d06a36
>
> Built and installed ...
> Checking 2 times in a row ...
> BINGO! It works!
>
> Excellent job John!
> ;)
>
> So, what was an issue?

The problem is that in your case when you used a custom loader string,
kname was set to point to 'cmd', but 'cmd' is set to an empty string
(to avoid re-parsing it) after the initial parse is done.  This caused
kname to point to an empty string.  Even worse, if a user entered some
unparseable text, with kname pointing at 'cmd', kname could effectively
be overwritten with garbage.

--
John Baldwin
_______________________________________________
freebsd-hackers@free... mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-hackers
To unsubscribe, send any mail to "freebsd-hackers-unsubscribe@free..."