summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDavid Hankins <dhankins@isc.org>2006-11-07 23:43:08 +0000
committerDavid Hankins <dhankins@isc.org>2006-11-07 23:43:08 +0000
commite2a02c6dcacc8467c744a47566c654d27d7c4af1 (patch)
treeaa45f0fa100a349bc07cac48c03c6a239d3da08f
parent9f6031e1fbf6d6c47d356a9ad69f939f55a4e1ba (diff)
downloadisc-dhcp-e2a02c6dcacc8467c744a47566c654d27d7c4af1.tar.gz
- A logic error in omapi interface code was repaired that might result in
incorrectly indicating 'up' state when any flags were set, rather than specifically the INTERFACE_REQUESTED flag. [ISC-Bugs #16431]
-rw-r--r--RELNOTES6
-rw-r--r--common/discover.c4
2 files changed, 8 insertions, 2 deletions
diff --git a/RELNOTES b/RELNOTES
index c4e3c4f2..4450c359 100644
--- a/RELNOTES
+++ b/RELNOTES
@@ -79,6 +79,12 @@ thanks to all of these good people here, both for working on the code
and for prodding me into improving it.
+ Changes since 3.0.5
+
+- A logic error in omapi interface code was repaired that might result in
+ incorrectly indicating 'up' state when any flags were set, rather than
+ specifically the INTERFACE_REQUESTED flag.
+
Changes since 3.0.5rc2
- Failover servers try harder to retransmit binding updates upon entering
diff --git a/common/discover.c b/common/discover.c
index 4ad213cb..6a9f41d6 100644
--- a/common/discover.c
+++ b/common/discover.c
@@ -34,7 +34,7 @@
#ifndef lint
static char copyright[] =
-"$Id: discover.c,v 1.42.2.21 2006/02/22 22:43:27 dhankins Exp $ Copyright (c) 2004-2006 Internet Systems Consortium. All rights reserved.\n";
+"$Id: discover.c,v 1.42.2.22 2006/11/07 23:43:08 dhankins Exp $ Copyright (c) 2004-2006 Internet Systems Consortium. All rights reserved.\n";
#endif /* not lint */
#include "dhcpd.h"
@@ -938,7 +938,7 @@ isc_result_t dhcp_interface_stuff_values (omapi_object_t *c,
status = omapi_connection_put_name (c, "state");
if (status != ISC_R_SUCCESS)
return status;
- if (interface -> flags && INTERFACE_REQUESTED)
+ if ((interface->flags & INTERFACE_REQUESTED) != 0)
status = omapi_connection_put_string (c, "up");
else
status = omapi_connection_put_string (c, "down");