Adam Tkac cb63d3
diff -up bind-9.7.0-P1/lib/dns/resolver.c.rh507429 bind-9.7.0-P1/lib/dns/resolver.c
Adam Tkac cb63d3
--- bind-9.7.0-P1/lib/dns/resolver.c.rh507429	2010-03-26 14:53:12.776469537 +0100
Adam Tkac cb63d3
+++ bind-9.7.0-P1/lib/dns/resolver.c	2010-03-26 15:07:18.363969402 +0100
Adam Tkac cb63d3
@@ -6160,6 +6160,7 @@ resume_dslookup(isc_task_t *task, isc_ev
Adam Tkac cb63d3
 	if (fevent->result == ISC_R_CANCELED) {
Adam Tkac cb63d3
 		dns_resolver_destroyfetch(&fctx->nsfetch);
Adam Tkac cb63d3
 		fctx_done(fctx, ISC_R_CANCELED, __LINE__);
Adam Tkac cb63d3
+		goto cleanup;
Adam Tkac cb63d3
 	} else if (fevent->result == ISC_R_SUCCESS) {
Adam Tkac cb63d3
 
Adam Tkac cb63d3
 		FCTXTRACE("resuming DS lookup");
Adam Tkac cb63d3
@@ -6219,15 +6220,16 @@ resume_dslookup(isc_task_t *task, isc_ev
Adam Tkac cb63d3
 						  resume_dslookup, fctx,
Adam Tkac cb63d3
 						  &fctx->nsrrset, NULL,
Adam Tkac cb63d3
 						  &fctx->nsfetch);
Adam Tkac cb63d3
-		if (result != ISC_R_SUCCESS)
Adam Tkac cb63d3
+		if (result != ISC_R_SUCCESS) {
Adam Tkac cb63d3
 			fctx_done(fctx, result, __LINE__);
Adam Tkac cb63d3
-		else {
Adam Tkac cb63d3
-			LOCK(&res->buckets[bucketnum].lock);
Adam Tkac cb63d3
-			locked = ISC_TRUE;
Adam Tkac cb63d3
-			fctx->references++;
Adam Tkac cb63d3
+			goto cleanup;
Adam Tkac cb63d3
 		}
Adam Tkac cb63d3
 	}
Adam Tkac cb63d3
 
Adam Tkac cb63d3
+	LOCK(&res->buckets[bucketnum].lock);
Adam Tkac cb63d3
+	locked = ISC_TRUE;
Adam Tkac cb63d3
+	fctx->references++;
Adam Tkac cb63d3
+
Adam Tkac cb63d3
  cleanup:
Adam Tkac cb63d3
 	if (dns_rdataset_isassociated(&nameservers))
Adam Tkac cb63d3
 		dns_rdataset_disassociate(&nameservers);