[PATCH] dhcp: add RFC5970 options to dhcp

[Neil Horman]

On Thu, Mar 01, 2012 at 10:58:19AM +0100, Jiri Popelka wrote:
> On 02/29/2012 07:07 PM, Neil Horman wrote:
> >diff -up ./common/tables.c.orig ./common/tables.c
> >--- ./common/tables.c.orig	2011-05-20 10:33:27.000000000 -0400
> >+++ ./common/tables.c	2012-02-29 12:03:22.610000294 -0500
> >@@ -459,6 +459,17 @@ static struct option dhcpv6_options[] =
> >  	{ "lq-relay-data", "6X",		&dhcpv6_universe, 47, 1 },
> >  	{ "lq-client-link", "6A",		&dhcpv6_universe, 48, 1 },
> >
> >+			/* RFC5970 OPTIONS */
> >+	{ "bootfile-url", "t",			&dhcpv6_universe, 59, 1},
> >+#if 0
> >+	/* Can't implement this until arrays of strings with length "StA"
> >+	 * are implemented
> >+	 */
> >+	{ "bootfile-param", "StA",		&dhcpv6_universe, 60, 1},
> >+#endif
> >+	{ "arch-type", "Sa",			&dhcpv6_universe, 61, 1},
> >+	{ "net-id", "BA",			&dhcpv6_universe, 62, 1},
> 
> Shouldn't the net-id be defined as
> 	{ "net-id", "BBB",		&dhcpv6_universe, 62, 1},
> ?
> 
I thought so too, at least according to the RFC, but the use of BBB seems to
introduce parser problems (either that or I have the wrong syntax for providing
3 bytes in dhcpd.conf).  I had assumed a format of BBB could be specified as:

option dhcp6.net-id 1,2,3;

Is that incorrect?
Neil

>