From: Thomas Markwalder <tmark@isc.org>
Date: Tue, 9 Sep 2014 11:50:56 +0000 (-0400)
Subject: [v4_1_esv] Server now supports a failover split value of 256.
X-Git-Tag: v4_1_esv_r11b1~19
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=7b709fa97181407ca6cff84e8bca1a6655d21665;p=thirdparty%2Fdhcp.git

[v4_1_esv] Server now supports a failover split value of 256.

    Merges in rt36664
---

diff --git a/RELNOTES b/RELNOTES
index 858ba8a70..e344e0ea5 100644
--- a/RELNOTES
+++ b/RELNOTES
@@ -76,6 +76,9 @@ by Eric Young (eay@cryptsoft.com).
   [ISC-Bugs #36712] Corrects Coverity reported "high" impact issues
   [ISC-Bugs #36933] Corrects Coverity reported "medium" impact issues
 
+- Server now supports a failover split value of 256.
+  [ISC-Bugs] #36664]
+
 			Changes since 4.1-ESV-R10rc1
 
 - None
diff --git a/server/confpars.c b/server/confpars.c
index e97cdcee8..ec912d803 100644
--- a/server/confpars.c
+++ b/server/confpars.c
@@ -1085,8 +1085,9 @@ void parse_failover_peer (cfile, group, type)
 				return;
 			}
 			split = atoi (val);
-			if (split > 255) {
-				parse_warn (cfile, "split must be < 256");
+			if (split > 256) {
+				parse_warn (cfile, "split must be between "
+                                                   "0 and 256, inclusive");
 			} else {
 				memset (hba, 0, sizeof hba);
 				for (i = 0; i < split; i++) {
diff --git a/server/dhcpd.conf.5 b/server/dhcpd.conf.5
index 1904a074b..82dad4fcf 100644
--- a/server/dhcpd.conf.5
+++ b/server/dhcpd.conf.5
@@ -669,7 +669,7 @@ The
 statement
 .RS 0.25i
 .PP
-.B split \fIindex\fR\fB;\fR
+.B split \fIbits\fR\fB;\fR
 .PP
 The split statement specifies the split between the primary and
 secondary for the purposes of load balancing.  Whenever a client
@@ -681,7 +681,9 @@ the secondary is responsible.  The \fBsplit\fR value determines
 how many of the leading bits are set to one.  So, in practice, higher
 split values will cause the primary to serve more clients than the
 secondary.  Lower split values, the converse.  Legal values are between
-0 and 255, of which the most reasonable is 128.
+0 and 256 inclusive, of which the most reasonable is 128.  Note that
+a value of 0 makes the secondary responsible for all clients and a value
+of 256 makes the primary responsible for all clients.
 .RE
 .PP
 The