Merge ~lvoytek/ubuntu/+source/bind-dyndb-ldap:MRE-lunar-9.18.18 into ubuntu/+source/bind-dyndb-ldap:ubuntu/lunar-devel

Proposed by Lena Voytek
Status: Merged
Approved by: git-ubuntu bot
Approved revision: not available
Merged at revision: 2f0515b3ff69b4141d62fa5322754260d4134979
Proposed branch: ~lvoytek/ubuntu/+source/bind-dyndb-ldap:MRE-lunar-9.18.18
Merge into: ubuntu/+source/bind-dyndb-ldap:ubuntu/lunar-devel
Diff against target: 82 lines (+60/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/remove-rpz_attach.patch (+52/-0)
debian/patches/series (+1/-0)
Reviewer Review Type Date Requested Status
Andreas Hasenack Approve
git-ubuntu bot Approve
Canonical Server Reporter Pending
Review via email: mp+451685@code.launchpad.net

Description of the change

Fix required for bind9 update to 9.18.18 in lunar

bind9 has to be built in proposed before this update is added to proposed itself - https://code.launchpad.net/~lvoytek/ubuntu/+source/bind9/+git/bind9/+merge/451681

PPA: https://launchpad.net/~lvoytek/+archive/ubuntu/bind9-mre

To post a comment you must log in.
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Security released an USN[1] for bind9 and will rebuild src:bind-dyndb-ldap in the security pocket. Once that happens and is published, this MP should be rebased on that, and released with the upcoming bind9 MRE.

1. https://ubuntu.com/security/notices/USN-6390-1

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Security published the no-change rebuild corresponding to their bind9 security update, so this now needs rebasing.

Revision history for this message
Lena Voytek (lvoytek) wrote :

Rebased onto security rebuild, and rebuilt in PPA. I'll run the dep-8 tests when it gets published

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

rechecked, looks good

review: Approve
Revision history for this message
git-ubuntu bot (git-ubuntu-bot) wrote :

Approvers: ahasenack, lvoytek
Uploaders: ahasenack
MP auto-approved

review: Approve
Revision history for this message
Lena Voytek (lvoytek) wrote :

Updated DEP-3 link for lunar and jammy

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Uploaded with rich history:

Uploading bind-dyndb-ldap_11.10-4ubuntu0.3.dsc
Uploading bind-dyndb-ldap_11.10-4ubuntu0.3.debian.tar.xz
Uploading bind-dyndb-ldap_11.10-4ubuntu0.3_source.buildinfo
Uploading bind-dyndb-ldap_11.10-4ubuntu0.3_source.changes

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/debian/changelog b/debian/changelog
2index 4f0fe4b..fd4897c 100644
3--- a/debian/changelog
4+++ b/debian/changelog
5@@ -1,3 +1,10 @@
6+bind-dyndb-ldap (11.10-4ubuntu0.3) lunar; urgency=medium
7+
8+ * d/p/remove-rpz_attach.patch: Remove rpz_attach to fix build failure against
9+ bind9 9.18.13+ (LP: #2028413)
10+
11+ -- Lena Voytek <lena.voytek@canonical.com> Thu, 21 Sep 2023 07:24:11 -0700
12+
13 bind-dyndb-ldap (11.10-4ubuntu0.2) lunar-security; urgency=medium
14
15 * No-change rebuild for bind9 security update.
16diff --git a/debian/patches/remove-rpz_attach.patch b/debian/patches/remove-rpz_attach.patch
17new file mode 100644
18index 0000000..f948d0d
19--- /dev/null
20+++ b/debian/patches/remove-rpz_attach.patch
21@@ -0,0 +1,52 @@
22+Description: Remove rpz_attach to fix build failure.
23+ rpz_attach is no longer provided by bind9, and bind-dyndb-ldap upstream
24+ removed the function since it would never be called anyway. As of 9.18.13
25+ building against bind9 with this function causes a build failure. Remove
26+ it by backporting the relevant upstream commit. Note that the changes are
27+ smaller here because
28+ https://pagure.io/bind-dyndb-ldap/c/5f37344d076f8cbd93a1842814b97763c3decc9a
29+ was never added, and its changes were reverted by this upstream commit.
30+Author: Petr Menšík <pemensik@redhat.com>
31+Origin: backport, https://pagure.io/bind-dyndb-ldap/c/1be57227a9727c6dce501a4bfa8e9eef35ba95d5
32+Bug-Ubuntu: https://bugs.launchpad.net/ubuntu/+source/bind-dyndb-ldap/+bug/2028413
33+Forwarded: not-needed
34+Last-Update: 2023-09-19
35+---
36+This patch header follows DEP-3: http://dep.debian.net/deps/dep3/
37+--- a/src/ldap_driver.c
38++++ b/src/ldap_driver.c
39+@@ -803,22 +803,6 @@
40+
41+ dns_db_rpz_attach(ldapdb->rbtdb, rpzs, rpz_num);
42+ }
43+-#else
44+-void
45+-rpz_attach(dns_db_t *db, void *void_rpzs, uint8_t rpz_num)
46+-{
47+- ldapdb_t *ldapdb = (ldapdb_t *) db;
48+- dns_rpz_zones_t *rpzs = (dns_rpz_zones_t *) void_rpzs;
49+- isc_result_t result;
50+-
51+- REQUIRE(VALID_LDAPDB(ldapdb));
52+-
53+- rpzs->zones[rpz_num]->db_registered = true;
54+- result = dns_db_updatenotify_register(ldapdb->rbtdb,
55+- dns_rpz_dbupdate_callback,
56+- rpzs->zones[rpz_num]);
57+- REQUIRE(result == ISC_R_SUCCESS);
58+-}
59+ #endif
60+
61+ /*
62+@@ -965,7 +949,11 @@
63+ resigned,
64+ isdnssec,
65+ getrrsetstats,
66++#if LIBDNS_VERSION_MAJOR < 1600
67+ rpz_attach,
68++#else
69++ NULL,
70++#endif
71+ NULL, /* rpz_ready */
72+ findnodeext,
73+ findext,
74diff --git a/debian/patches/series b/debian/patches/series
75index 19cff3d..b07a2a6 100644
76--- a/debian/patches/series
77+++ b/debian/patches/series
78@@ -6,3 +6,4 @@ fix-dns_db_allrdatasets.diff
79 fix-include.diff
80 fix-isc-error.diff
81 make-dscp-optional.diff
82+remove-rpz_attach.patch

Subscribers

People subscribed via source and target branches