--- lib/PXElator/dhcpd.pm	2009/08/03 20:53:46	129
+++ lib/PXElator/dhcpd.pm	2009/08/06 20:14:43	167
@@ -40,7 +40,7 @@
 sub client_ip {
 	my ( $mac ) = @_;
 
-	my $conf = "$server::base_dir/conf/$server::ip";
+	my $conf = $server::conf;
 	mkdir $conf unless -e $conf;
 
 	if ( -e "$conf/mac/$mac" ) {
@@ -49,24 +49,27 @@
 		return $ip;
 	}
 
-	mkdir $_ foreach grep { ! -e $_ } map { "$conf/$_" } ( 'ip', 'mac' );
-
 	my $p = Net::Ping->new;
 
 	my $prefix = $server::ip;
 	$prefix =~ s{\.\d+$}{.};
 	my $ip = $prefix . $addr;
-	while ( -e "conf/ip/$ip" || $p->ping( $ip ) ) {
+
+	while ( -e "$conf/ip/$ip" || $p->ping( $ip ) ) {
 		$ip = $prefix . $addr++;
 		die "all addresses allocated!" if $addr == $server::ip_to;
 	}
 
 	write_file "$conf/mac/$mac", $ip;
-	if ( -l "$conf/ip/$ip" && readlink "$conf/ip/$ip" ne "$conf/mac/$mac") {
-		unlink     "$conf/ip/$ip";
-		symlink    "$conf/mac/$mac", "$conf/ip/$ip";
-		warn "$mac IP changed to $ip";
-	}
+
+	my $ip_path = "$conf/ip/$ip";
+	mkdir $ip_path unless -e $ip_path;
+
+	if ( -l "$ip_path/mac" && readlink "$ip_path/mac" ne "$conf/mac/$mac" ) {
+		warn "$mac IP changed from ", readlink "$ip_path/mac", " to $ip";
+		unlink "$ip_path/mac";
+	};
+	symlink "$conf/mac/$mac", "$ip_path/mac";
 
 	print "$mac NEW $ip\n";
 
@@ -76,6 +79,7 @@
 use log;
 use config;
 use pxelinux;
+use client;
 
 our $file;
 our $transaction = 0; # FIXME predictible transaction numbers
@@ -126,15 +130,41 @@
 		Giaddr	=> $dhcp->giaddr(),
 		Chaddr	=> $dhcp->chaddr(),
 		File	=> $file,
+		DHO_DHCP_SERVER_IDENTIFIER()	=> $server::ip,	# busybox/udhcpc needs it but doesn't request
+	};
+
+	my $options = {
+		DHO_SUBNET_MASK()	=> $server::netmask,
+		DHO_ROUTERS()		=> $server::ip,
+		DHO_DOMAIN_NAME()	=> $server::domain_name,
+		DHO_NAME_SERVERS()	=> $server::ip,
+		DHO_DOMAIN_NAME_SERVERS() => $server::ip,
+		DHO_HOST_NAME()		=> client::conf( $ip, 'hostname' ),
+		DHO_BROADCAST_ADDRESS() => $server::bcast,
+#		DHO_NTP_SERVERS() => '',
 	};
 
+	my @requested = split(/\s/, $dhcp->getOptionValue(DHO_DHCP_PARAMETER_REQUEST_LIST));
+	warn "options ",dump( $options ), ' requested: ',dump( @requested ) if $debug;
+
+	my @missing;
+	foreach ( @requested ) {
+		if ( defined $options->{$_} ) {
+			$packet->{$_} = $options->{$_};
+		} else {
+			push @missing, $_;
+		}
+	}
+
+	warn "W: options requested but missing: ",dump( @missing );
+
 	foreach my $opt ( 'magic', 'config_file', 'path_prefix', 'reboot_time' ) {
 		my $DH0 = eval 'DHO_PXELINUX_' . uc $opt;
 		warn "DH0: $@" if $@;
 		my $v = eval "\$pxelinux::$opt";
 		warn "v: $@" if $@;
 		next unless defined $v;
-		warn "pxelinux dhcp option $opt = $DH0 = $v";
+		warn "pxelinux dhcp option $opt = $DH0 = $v\n" if $debug;
 		$packet->{ $DH0 } = $v;
 	}
 
@@ -150,11 +180,6 @@
 		if ( $ip eq $requested_ip ) {
 			$packet->{DHO_DHCP_MESSAGE_TYPE()}      = DHCPACK();
 			$packet->{DHO_DHCP_LEASE_TIME()}        = 5 * 60; # 5 min
-#			$packet->{DHO_DHCP_SERVER_IDENTIFIER()} = $server::ip;		# FIXME
-			$packet->{DHO_SUBNET_MASK()}            = '255.255.255.0';
-			$packet->{DHO_ROUTERS()}                = $server::ip;
-#			$packet->{DHO_DOMAIN_NAME()}            = 'pxelator.lan';
-#			$packet->{DHO_NAME_SERVERS()}           = $server::ip;
 #			$packet->{DHO_ROOT_PATH()}              = '/exports/foobar';
 		} else {
 			$packet->{DHO_DHCP_MESSAGE_TYPE()} = DHCPNAK();
@@ -168,8 +193,6 @@
 
 	warn ">> $mac == $ip server: $server::ip", $file ? " file: $file\n" : "\n" if $debug;
 
-warn "## created packet ", dump( $packet );
-
 	$packet = new Net::DHCP::Packet( %$packet );
 	warn "send ",$packet->toString() if $debug;