- Slightly smaller critical region in one case in infra cache.

git-svn-id: http://unbound.nlnetlabs.nl/svn/trunk@2611 be551aaa-1e26-0410-a405-d3ace91eadb9
jedisct1 · Feb 10, 2012 · af3a845 · af3a845
1 parent 70c199f
commit af3a845
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/doc/Changelog b/doc/Changelog
@@ -1,3 +1,6 @@
+10 February 2012: Wouter
+	- Slightly smaller critical region in one case in infra cache.
+
 9 February 2012: Wouter
 	- Fix AHX_BROKEN_MEMCMP for autoheader mess up of #undef in config.h.
 

diff --git a/services/cache/infra.c b/services/cache/infra.c
@@ -465,11 +465,11 @@ infra_get_lame_rtt(struct infra_cache* infra,
 		/* minus 1000 because that is outside of the RTTBAND, so
 		 * blacklisted servers stay blacklisted if this is chosen */
 		if(host->rtt.rto >= USEFUL_SERVER_TOP_TIMEOUT) {
+			lock_rw_unlock(&e->lock);
 			*rtt = USEFUL_SERVER_TOP_TIMEOUT-1000;
 			*lame = 0;
 			*dnsseclame = 0;
 			*reclame = 0;
-			lock_rw_unlock(&e->lock);
 			return 1;
 		}
 		lock_rw_unlock(&e->lock);