[Freeipa-devel] [PATCH 0123-0126] Separate master and forward zones (add idnsForwardZone object class)

[Petr Spacek]

On 22.3.2013 13:03, Petr Spacek wrote:
> Hello,
>
> this patch set separates master zones (idnsZone objectClass) from forward
> zones (idnsForwardZone objectClass). Support for forward zones in idnsZone
> objectClass is still present to ease upgrades.
>
> See each commit message for all the gory details.

I encountered a crash caused by bug in patch 126. Fixed version is attached.

Diff between patch 126 version 1 and 2:

--- a/src/ldap_helper.c
+++ b/src/ldap_helper.c
@@ -3391,7 +3391,7 @@ update_zone(isc_task_t *task, isc_event_t *event)

         CHECK(dn_to_dnsname(inst->mctx, pevent->dn, &currname, NULL));

-       if (result == ISC_R_SUCCESS &&
+       if (ldap_qresult_zone != NULL &&
             HEAD(ldap_qresult_zone->ldap_entries) != NULL) {
                 entry_zone = HEAD(ldap_qresult_zone->ldap_entries);
                 CHECK(ldap_entry_getclass(entry_zone, &objclass));

-- 
Petr^2 Spacek
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0126-2-Add-support-for-pure-forward-zones-idnsForwardZone-o.patch
Type: text/x-patch
Size: 30313 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20130402/b9677967/attachment.bin>