- 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 cc1d47eabd4c874279ec2dba98da6b2a898faacc..7a7f214f3e170a8b245b1329cd3420a9430ffd64 100644 (file)
--- 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 567a0c3f1fc92c930cca05db14f0335fc6f977e2..5eeef2c92ce8531b622756f7f782a5d23296f2a0 100644 (file)
--- 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)