BIND 10 trac3295_2, updated. e1938e5c1fe7e19553e52c611aecf921acc87822 [3295] Do not generate NCRs if DDNS is disabled.

BIND 10 source code commits bind10-changes at lists.isc.org
Wed Jan 29 14:33:54 UTC 2014


The branch, trac3295_2 has been updated
       via  e1938e5c1fe7e19553e52c611aecf921acc87822 (commit)
      from  41f5d870b1701e4efd738156ee18f61d9a5f6b8b (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit e1938e5c1fe7e19553e52c611aecf921acc87822
Author: Marcin Siodelski <marcin at isc.org>
Date:   Wed Jan 29 15:33:43 2014 +0100

    [3295] Do not generate NCRs if DDNS is disabled.
    
    This is a result of the second review.

-----------------------------------------------------------------------

Summary of changes:
 src/bin/dhcp6/dhcp6_srv.cc |   10 ++++++++++
 1 file changed, 10 insertions(+)

-----------------------------------------------------------------------
diff --git a/src/bin/dhcp6/dhcp6_srv.cc b/src/bin/dhcp6/dhcp6_srv.cc
index 25aa91a..06436d4 100644
--- a/src/bin/dhcp6/dhcp6_srv.cc
+++ b/src/bin/dhcp6/dhcp6_srv.cc
@@ -1044,6 +1044,11 @@ Dhcpv6Srv::processClientFqdn(const Pkt6Ptr& question, const Pkt6Ptr& answer) {
 
 void
 Dhcpv6Srv::createNameChangeRequests(const Pkt6Ptr& answer) {
+    // Don't create NameChangeRequests if DNS updates are disabled.
+    if (!FQDN_ENABLE_UPDATE) {
+        return;
+    }
+
     // The response message instance is always required. For instance it
     // holds the Client Identifier. It is a programming error if supplied
     // message is NULL.
@@ -1126,6 +1131,11 @@ Dhcpv6Srv::createNameChangeRequests(const Pkt6Ptr& answer) {
 
 void
 Dhcpv6Srv::createRemovalNameChangeRequest(const Lease6Ptr& lease) {
+    // Don't create NameChangeRequests if DNS updates are disabled.
+    if (!FQDN_ENABLE_UPDATE) {
+        return;
+    }
+
     // If we haven't performed a DNS Update when lease was acquired,
     // there is nothing to do here.
     if (!lease->fqdn_fwd_ && !lease->fqdn_rev_) {



More information about the bind10-changes mailing list