From: David Hankins <dhankins@isc.org>
Date: Thu, 13 Dec 2007 16:56:24 +0000 (+0000)
Subject: - The 'min-secs' configuration parameter's log message has been updated to
X-Git-Tag: v4_1_0a1~53
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=00638355f16cb656add6785abe34521f24aab9c2;p=thirdparty%2Fdhcp.git

- The 'min-secs' configuration parameter's log message has been updated to
  be more helpful.  [ISC-Bugs #17299]
---

diff --git a/RELNOTES b/RELNOTES
index cc1d47eab..7a7f214f3 100644
--- a/RELNOTES
+++ b/RELNOTES
@@ -59,6 +59,9 @@ suggested fixes to <dhcp-users@isc.org>.
   they were declared has been updated to be more helpful and identify the
   typo in configuration that created the spanning addresses.
 
+- The 'min-secs' configuration parameter's log message has been updated to
+  be more helpful.
+
 			Changes since 4.0.0b3
 
 - The reverse dns name for PTR updates on IPv6 addresses has been fixed to
diff --git a/server/dhcp.c b/server/dhcp.c
index 567a0c3f1..5eeef2c92 100644
--- a/server/dhcp.c
+++ b/server/dhcp.c
@@ -1684,9 +1684,10 @@ void ack_lease (packet, lease, offer, when, msg, ms_nulltp, hp)
 					   &lease -> scope, oc, MDL)) {
 			if (d1.len &&
 			    ntohs (packet -> raw -> secs) < d1.data [0]) {
-				log_info ("%s: %d secs < %d", msg,
-					  ntohs (packet -> raw -> secs),
-					  d1.data [0]);
+				log_info("%s: configured min-secs value (%d) "
+					 "is greater than secs field (%d).  "
+					 "message dropped.", msg, d1.data[0],
+					 ntohs(packet->raw->secs));
 				data_string_forget (&d1, MDL);
 				free_lease_state (state, MDL);
 				if (host)