BIND 10 master, updated. f8ef5bcb1e8ebc747b32192348faae9fd32fdba9 Merge branch 'master' into trac1084
BIND 10 source code commits
bind10-changes at lists.isc.org
Thu Jun 30 14:23:10 UTC 2011
The branch, master has been updated
via f8ef5bcb1e8ebc747b32192348faae9fd32fdba9 (commit)
via 4191945aad5aaf0873b15727716d0a988b1c978d (commit)
from 666d6e49e1cd46fd293b3fdce239e34588666ed6 (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 f8ef5bcb1e8ebc747b32192348faae9fd32fdba9
Merge: 4191945aad5aaf0873b15727716d0a988b1c978d 666d6e49e1cd46fd293b3fdce239e34588666ed6
Author: Stephen Morris <stephen at isc.org>
Date: Thu Jun 30 15:22:20 2011 +0100
Merge branch 'master' into trac1084
commit 4191945aad5aaf0873b15727716d0a988b1c978d
Author: Stephen Morris <stephen at isc.org>
Date: Thu Jun 30 13:37:19 2011 +0100
[trac1084] Reorder statements in MemoryZone::add
Only log details of the zone being added after input validation,
when it is known the the zone pointer is non-null. (Validation
detects null pointers and throws an exception.)
-----------------------------------------------------------------------
Summary of changes:
src/lib/datasrc/memory_datasrc.cc | 7 +++++--
1 files changed, 5 insertions(+), 2 deletions(-)
-----------------------------------------------------------------------
diff --git a/src/lib/datasrc/memory_datasrc.cc b/src/lib/datasrc/memory_datasrc.cc
index 3c57d1b..b8019a2 100644
--- a/src/lib/datasrc/memory_datasrc.cc
+++ b/src/lib/datasrc/memory_datasrc.cc
@@ -225,10 +225,13 @@ struct MemoryZone::MemoryZoneImpl {
*/
// Implementation of MemoryZone::add
result::Result add(const ConstRRsetPtr& rrset, DomainTree* domains) {
+ // Sanitize input. This will cause an exception to be thrown
+ // if the input RRset is empty.
+ addValidation(rrset);
+
+ // OK, can add the RRset.
LOG_DEBUG(logger, DBG_TRACE_DATA, DATASRC_MEM_ADD_RRSET).
arg(rrset->getName()).arg(rrset->getType()).arg(origin_);
- // Sanitize input
- addValidation(rrset);
// Add wildcards possibly contained in the owner name to the domain
// tree.
More information about the bind10-changes
mailing list