From: Shawn Routhier <sar@isc.org>
Date: Mon, 22 Aug 2011 20:39:19 +0000 (+0000)
Subject: Fix the code that checks for an existing DDNS transaction to cancel
X-Git-Tag: v4_3_0a1~157
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=d208bb043169878b6475c13d42e6067518aba739;p=thirdparty%2Fdhcp.git

Fix the code that checks for an existing DDNS transaction to cancel
when removing DDNS information, so that we will continue with the
processing if we have a lease even if it doesn't have an outstanding
transaction.  [ISC-Bugs #24682]
---

diff --git a/RELNOTES b/RELNOTES
index ba8e44d8d..a58fa381b 100644
--- a/RELNOTES
+++ b/RELNOTES
@@ -39,6 +39,13 @@ The system has only been tested on Linux, FreeBSD, and Solaris, and may not
 work on other platforms. Please report any problems and suggested fixes to
 <dhcp-users@isc.org>.
 
+			Changes since 4.2.2
+
+- Fix the code that checks for an existing DDNS transaction to cancel
+  when removing DDNS information, so that we will continue with the
+  processing if we have a lease even if it doesn't have an outstanding
+  transaction.  [ISC-Bugs #24682]
+
 			Changes since 4.2.1
 
 ! In dhclient check the data for some string options for
diff --git a/server/ddns.c b/server/ddns.c
index 99c087148..4149494e0 100644
--- a/server/ddns.c
+++ b/server/ddns.c
@@ -1592,8 +1592,7 @@ ddns_removals(struct lease    *lease,
 	} else if ((lease6 != NULL) && (lease6->ddns_cb != NULL)) {
 		ddns_cancel(lease6->ddns_cb);
 		lease6->ddns_cb = NULL;
-	} else
-		goto cleanup;
+	}
 
 	/* allocate our control block */
 	ddns_cb = ddns_cb_alloc(MDL);