- Failover port configuration can now be left to defaults (port 647) as

described in the -12 revision of the Failover draft (and assigned by
  IANA).  [ISC-Bugs #17738]

RELNOTES

@@ -38,7 +38,9 @@ work on other platforms. Please report any problems and suggested fixes to
 
 			Changes since 4.1.0 (new features)
 
-
+- Failover port configuration can now be left to defaults (port 647) as
+  described in the -12 revision of the Failover draft (and assigned by
+  IANA).
 
 			Changes since 4.1.0 (bug fixes)
 

includes/failover.h

@@ -74,6 +74,16 @@ typedef struct {
 # define DEFAULT_MAX_RESPONSE_DELAY	20
 #endif
 
+/*
+ * IANA has assigned ports 647 ("dhcp-failover") and 847 ("dhcp-failover2").
+ * Of these, only port 647 is mentioned in the -12 draft revision.  We're not
+ * sure if they are supposed to indicate primary and secondary?  No matter,
+ * we'll stick to the -12 draft revision level.
+ */
+#ifndef  DEFAULT_FAILOVER_PORT
+# define DEFAULT_FAILOVER_PORT		647
+#endif
+
 #define FM_OFFSET(x) (long)(&(((failover_message_t *)0) -> x))
 
 /* All of the below definitions are mandated by draft-ietf-dhc-failover-12.

server/confpars.c

@@ -1131,11 +1131,10 @@ void parse_failover_peer (cfile, group, type)
 	if (!peer -> partner.address)
 		parse_warn (cfile, "peer address may not be omitted");
 
-	/* XXX - when/if we get a port number assigned, just set as default */
+	if (!peer->me.port)
-	if (!peer -> me.port)
+		peer->me.port = DEFAULT_FAILOVER_PORT;
-		parse_warn (cfile, "local port may not be omitted");
+	if (!peer->partner.port)
-	if (!peer -> partner.port)
+		peer->partner.port = DEFAULT_FAILOVER_PORT;
-		parse_warn (cfile, "peer port may not be omitted");
 
 	if (peer -> i_am == primary) {
 	    if (!peer -> hba) {

server/dhcpd.conf.5

@@ -28,7 +28,7 @@
 .\" see ``http://www.vix.com''.   To learn more about Nominum, Inc., see
 .\" ``http://www.nominum.com''.
 .\"
-.\" $Id: dhcpd.conf.5,v 1.100 2009/01/22 21:22:42 dhankins Exp $
+.\" $Id: dhcpd.conf.5,v 1.101 2009/02/20 18:20:00 dhankins Exp $
 .\"
 .TH dhcpd.conf 5
 .SH NAME
@@ -594,9 +594,9 @@ statement
 .B port \fIport-number\fR\fB;\fR
 .PP
 The \fBport\fR statement declares the TCP port on which the server
-should listen for connections from its failover peer.   This statement
+should listen for connections from its failover peer.  This statement
-may not currently be omitted, because the failover protocol does not
+may be omitted, in which case the IANA assigned port number 647 will be
-yet have a reserved TCP port number.
+used by default.
 .RE
 .PP
 The 
@@ -608,10 +608,8 @@ statement
 .PP
 The \fBpeer port\fR statement declares the TCP port to which the
 server should connect to reach its failover peer for failover
-messages.   This statement may not be omitted because the failover
+messages.  This statement may be omitted, in which case the IANA
-protocol does not yet have a reserved TCP port number.   The port
+assigned port number 647 will be used by default.
-number declared in the \fBpeer port\fR statement may be the same as
-the port number declared in the \fBport\fR statement.
 .RE
 .PP
 The