BIND 10 trac2108_3, updated. d1c78d4a09351ccb7f8aab34f74fe5245ba590d6 [2108] Fix addWildCards() further

BIND 10 source code commits bind10-changes at lists.isc.org
Mon Sep 3 07:10:30 UTC 2012


The branch, trac2108_3 has been updated
       via  d1c78d4a09351ccb7f8aab34f74fe5245ba590d6 (commit)
      from  bba5b86a7cbeb0400d5edab44b63a54cc9848a0a (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 d1c78d4a09351ccb7f8aab34f74fe5245ba590d6
Author: Mukund Sivaraman <muks at isc.org>
Date:   Mon Sep 3 12:40:17 2012 +0530

    [2108] Fix addWildCards() further

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

Summary of changes:
 src/lib/datasrc/memory/memory_client.cc |   14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/lib/datasrc/memory/memory_client.cc b/src/lib/datasrc/memory/memory_client.cc
index dca7450..d14826f 100644
--- a/src/lib/datasrc/memory/memory_client.cc
+++ b/src/lib/datasrc/memory/memory_client.cc
@@ -135,7 +135,8 @@ public:
     //
     // We also perform the same trick for empty wild card names possibly
     // contained in 'name' (e.g., '*.foo.example' in 'bar.*.foo.example').
-    void addWildcards(const Name& zone_name, const Name& name)
+    void addWildcards(const Name& zone_name, ZoneData& zone_data,
+                      const Name& name)
     {
         Name wname(name);
         const unsigned int labels(wname.getLabelCount());
@@ -147,20 +148,17 @@ public:
                 LOG_DEBUG(logger, DBG_TRACE_DATA, DATASRC_MEM_ADD_WILDCARD).
                     arg(name);
 
-                ZoneNode *node;
-
                 // Ensure a separate level exists for the "wildcarding" name,
                 // and mark the node as "wild".
-                ZoneTree::Result result(zone_data_->insertName(local_mem_sgmt,
-                                                               wname.split(1),
-                                                               &node));
+                ZoneNode *node;
+                zone_data.insertName(local_mem_sgmt, wname.split(1), &node);
                 node->setFlag(ZoneData::WILDCARD_NODE);
 
                 // Ensure a separate level exists for the wildcard name.
                 // Note: for 'name' itself we do this later anyway, but the
                 // overhead should be marginal because wildcard names should
                 // be rare.
-                result = zone_data_->insertName(local_mem_sgmt, wname, &node);
+                zone_data.insertName(local_mem_sgmt, wname, &node);
             }
         }
     }
@@ -458,7 +456,7 @@ public:
         // tree.
         // Note: this can throw an exception, breaking strong exception
         // guarantee.  (see also the note for contextCheck() below).
-        addWildcards(zone_name, rrset->getName());
+        addWildcards(zone_name, zone_data, rrset->getName());
 
         // Get the node
         DomainNode* node;



More information about the bind10-changes mailing list