rpms/dhcp/F-13 dhclient-script,1.19,1.20 dhcp.spec,1.300,1.301

[Jiří Popelka]

Author: jpopelka

Update of /cvs/pkgs/rpms/dhcp/F-13
In directory cvs01.phx2.fedoraproject.org:/tmp/cvs-serv9004

Modified Files:
	dhclient-script dhcp.spec 
Log Message:
* Wed Apr 21 2010 Jiri Popelka <jpopelka at redhat.com> - 12:4.1.1-17
- If the Reply was received in response to Renew or Rebind message,
  client adds any new addresses in the IA option to the IA (#578097)



Index: dhclient-script
===================================================================
RCS file: /cvs/pkgs/rpms/dhcp/F-13/dhclient-script,v
retrieving revision 1.19
retrieving revision 1.20
diff -u -p -r1.19 -r1.20
--- dhclient-script	24 Mar 2010 11:06:33 -0000	1.19
+++ dhclient-script	21 Apr 2010 14:56:05 -0000	1.20
@@ -89,6 +89,9 @@ make_resolv_conf() {
             if [ -n "${new_domain_search}" ]; then
                 echo "search ${new_domain_search//\\032/ }" >> ${rscf}
             elif [ -n "${new_domain_name}" ]; then
+                # Note that the DHCP 'Domain Name Option' is really just a domain
+                # name, and that this practice of using the domain name option as
+                # a search path is both nonstandard and deprecated.
                 echo "search ${new_domain_name//\\032/ }" >> ${rscf}
             fi
         fi
@@ -407,14 +410,11 @@ dhconfig() {
     fi
 }
 
-dh6config() {
-    case "${reason}" in
-        BOUND6)
-            if [ -z "${new_ip6_address}" ] &&
-               [ -z "${new_ip6_prefixlen}" ]; then
-                exit_with_hooks 2
-            fi
-
+# Section 18.1.8. (Receipt of Reply Messages) of RFC 3315 says:
+# The client SHOULD perform duplicate address detection on each of
+# the addresses in any IAs it receives in the Reply message before
+# using that address for traffic.
+add_ipv6_addr_with_DAD() {
             ip -6 addr add ${new_ip6_address}/${new_ip6_prefixlen} \
                 dev ${interface} scope global
 
@@ -439,11 +439,28 @@ dh6config() {
                 ip -6 addr del ${new_ip6_address}/${new_ip6_prefixlen} dev ${interface}
                 exit_with_hooks 3
             fi
+}
+
+dh6config() {
+    case "${reason}" in
+        BOUND6)
+            if [ -z "${new_ip6_address}" ] &&
+               [ -z "${new_ip6_prefixlen}" ]; then
+                exit_with_hooks 2
+            fi
+
+            add_ipv6_addr_with_DAD
 
             make_resolv_conf
             ;;
 
         RENEW6|REBIND6)
+            if [ -n "${new_ip6_prefixlen}" ] &&
+               [ -n "${new_ip6_address}" ] &&
+               [  ! "${new_ip6_address}" = "${old_ip6_address}" ]; then
+                add_ipv6_addr_with_DAD
+            fi
+
             if [ ! "${new_dhcp6_name_servers}" = "${old_dhcp6_name_servers}" ] ||
                [ ! "${new_dhcp6_domain_search}" = "${old_dhcp6_domain_search}" ]; then
                 make_resolv_conf


Index: dhcp.spec
===================================================================
RCS file: /cvs/pkgs/rpms/dhcp/F-13/dhcp.spec,v
retrieving revision 1.300
retrieving revision 1.301
diff -u -p -r1.300 -r1.301
--- dhcp.spec	19 Apr 2010 10:33:46 -0000	1.300
+++ dhcp.spec	21 Apr 2010 14:56:05 -0000	1.301
@@ -13,7 +13,7 @@
 Summary:  Dynamic host configuration protocol software
 Name:     dhcp
 Version:  %{basever}
-Release:  16%{?dist}
+Release:  17%{?dist}
 # NEVER CHANGE THE EPOCH on this package.  The previous maintainer (prior to
 # dcantrell maintaining the package) made incorrect use of the epoch and
 # that's why it is at 12 now.  It should have never been used, but it was.
@@ -226,6 +226,7 @@ libdhcpctl and libomapi static libraries
 %patch24 -p1 -b .retransmission
 
 # Fill in Elapsed Time Option in Release message (#582939)
+# (Submitted to dhcp-bugs at isc.org - [ISC-Bugs #21171])
 %patch25 -p1 -b .release6-elapsed
 
 # Copy in documentation and example scripts for LDAP patch to dhcpd
@@ -510,6 +511,10 @@ fi
 %attr(0644,root,root) %{_mandir}/man3/omapi.3.gz
 
 %changelog
+* Wed Apr 21 2010 Jiri Popelka <jpopelka at redhat.com> - 12:4.1.1-17
+- If the Reply was received in response to Renew or Rebind message,
+  client adds any new addresses in the IA option to the IA (#578097)
+
 * Mon Apr 19 2010 Jiri Popelka <jpopelka at redhat.com> - 12:4.1.1-16
 - Fill in Elapsed Time Option in Release/Decline messages (#582939)