[Freeipa-devel] [PATCH 0061] Add missing DNS view attach/detach to LDAP instance management code

[Petr Spacek]

On 09/20/2012 01:17 PM, Petr Spacek wrote:
> On 09/13/2012 01:37 PM, Petr Spacek wrote:
>> Hello,
>>
>>      Add missing DNS view attach/detach to LDAP instance management code.
>>      This fixes race condition in BIND shutdown after SIGINT:
>>      - failing assert caused by use-after-free in dns_zt_find():
>>      (((zt) != ((void *)0)) && (((const isc__magic_t *)(zt))->magic
>>      == ((('Z') << 24 | ('T') << 16 | ('b') << 8 | ('l')))))
>>
>> Petr^2 Spacek
>
> This patch causes deadlock in shut-down sequence in certain situations, I'm
> still searching for the root cause.
>
> Old comment from 2009-02-10
> /* commented out for now, cause named to hang */
> was correct, apparently ...
>
Apparently my statement above was wrong. Deadlock I observed was caused by 
incorrect zone attach/detach sequence. This problem is fixed in
"[PATCH 0067] Fix error handling in ldap_get_zone_serial()".

You can proceed with reviewing this patch.

Petr^2 Spacek