From: Wouter Wijngaards <wouter@nlnetlabs.nl>
Date: Fri, 15 Jun 2012 19:44:09 +0000 (+0000)
Subject: - code review.
X-Git-Tag: release-1.4.18rc1~36
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=452b7fce3adfa1615063329fc6b17912e63f85f0;p=thirdparty%2Funbound.git

- code review.


git-svn-id: file:///svn/unbound/trunk@2687 be551aaa-1e26-0410-a405-d3ace91eadb9
---

diff --git a/doc/Changelog b/doc/Changelog
index 5509be54d..77511edca 100644
--- a/doc/Changelog
+++ b/doc/Changelog
@@ -2,6 +2,7 @@
 	- implement log-time-ascii on windows.
 	- The key-cache bad key ttl is now 60 seconds.
 	- updated iana ports list.
+	- code review.
 
 11 June 2012: Wouter
 	- bug #452: fix crash on assert in mesh_state_attachment.
diff --git a/iterator/iter_fwd.c b/iterator/iter_fwd.c
index 04976db18..f12531f47 100644
--- a/iterator/iter_fwd.c
+++ b/iterator/iter_fwd.c
@@ -260,10 +260,10 @@ read_forwards(struct iter_forwards* fwd, struct config_file* cfg)
 		 * last resort will ask for parent-side NS record and thus
 		 * fallback to the internet name servers on a failure */
 		dp->has_parent_side_NS = (uint8_t)!s->isfirst;
-		if(!forwards_insert(fwd, LDNS_RR_CLASS_IN, dp))
-			return 0;
 		verbose(VERB_QUERY, "Forward zone server list:");
 		delegpt_log(VERB_QUERY, dp);
+		if(!forwards_insert(fwd, LDNS_RR_CLASS_IN, dp))
+			return 0;
 	}
 	return 1;
 }