Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 13 Mar 2002 05:47:05 -0800
From:      Luigi Rizzo <luigi@FreeBSD.org>
To:        Robert Watson <rwatson@FreeBSD.org>
Cc:        cvs-committers@FreeBSD.org, cvs-all@FreeBSD.org
Subject:   Re: cvs commit: src/sys/nfsclient bootp_subr.c
Message-ID:  <20020313054705.B65383@iguana.icir.org>
In-Reply-To: <Pine.NEB.3.96L.1020313080233.36618I-100000@fledge.watson.org>
References:  <200203130923.g2D9NBL68938@freefall.freebsd.org> <Pine.NEB.3.96L.1020313080233.36618I-100000@fledge.watson.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, Mar 13, 2002 at 08:05:59AM -0500, Robert Watson wrote:
...
> Hmm.  This is in the private number range, I assume?  Hopefully the

of course yes.

> handling is sufficiently robust that whatever comes back, bad things won't
> happen (be it a binary blob with lots of nulls, *'s, shell
> meta-characters, etc, when handled in userland?).

I am not going, in the kernel, to escape characters which have
special meaning for arbitrary userland programs -- that's their
business.

The kernel copies chars up to the first NUL and within the buffer
size allocated for the purpose, and bootpc_tag() makes sure that
there is actually a NUL in the source buffer when it returns.

> >   The intended use is to pass information to userland for run-time
> >   configuration of a diskless client without having to run a bootp/dhcp
> >   client for the third time (after the one in pxeboot/etherboot, and
> >   the one in the kernel bootp), also because these clients generally
> >   screwup the interface configuration, which is not exactly what you
> >   want when you have your disks nfs-mounted.
> 
> It seems to me that a slightly more useful incantation would actually
> expose all of the bootp options to userland via a more dimmensional sysctl
> mib.  This would allow userland to inspect any other options that turned

Sure, i don't mind doing that using dynamic sysctls if people have
no objections.  I can create kern.bootp.tX (where X is the tag
number) sysctl variables according to what came back from the server.
Using names instead of numeric values is not going to work I am
afraid, as half of the namespace is not standardized anyways.

	cheers
	luigi

To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe cvs-all" in the body of the message




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20020313054705.B65383>