[Freeipa-devel] [PATCH 0110] Fix crash during per-zone cache flush

[Adam Tkac]

On Mon, Feb 04, 2013 at 05:37:05PM +0100, Petr Spacek wrote:
> Hello,
> 
>  Fix crash during per-zone cache flush.
> 
>  https://fedorahosted.org/bind-dyndb-ldap/ticket/107

Ack, thanks for the patch.

Regards, Adam

> From 42edc724eedd401bd1b1e06f1b1c6ef2ad878ed2 Mon Sep 17 00:00:00 2001
> From: Petr Spacek <pspacek at redhat.com>
> Date: Mon, 4 Feb 2013 17:35:24 +0100
> Subject: [PATCH] Fix crash during per-zone cache flush.
> 
> https://fedorahosted.org/bind-dyndb-ldap/ticket/107
> 
> Signed-off-by: Petr Spacek <pspacek at redhat.com>
> ---
>  src/zone_register.c | 6 ++++--
>  1 file changed, 4 insertions(+), 2 deletions(-)
> 
> diff --git a/src/zone_register.c b/src/zone_register.c
> index 03eb1d0765de371420e4da6beb1a7b2e2e52db94..e52397357f2ef482d8dfee33f7564cd9157afa56 100644
> --- a/src/zone_register.c
> +++ b/src/zone_register.c
> @@ -267,8 +267,10 @@ zr_flush_all_caches(zone_register_t *zr) {
>  		ldap_cache_t *cache;
>  
>  		CHECK(dns_rbtnodechain_current(&chain, NULL, NULL, &node));
> -		cache = ((zone_info_t *)(node->data))->cache;
> -		CHECK(flush_ldap_cache(cache));
> +		if (node->data != NULL) { /* skip auxiliary RBT nodes */
> +			cache = ((zone_info_t *)(node->data))->cache;
> +			CHECK(flush_ldap_cache(cache));
> +		}
>  		result = dns_rbtnodechain_next(&chain, NULL, NULL);
>  	}
>  
> -- 
> 1.7.11.7
> 


-- 
Adam Tkac, Red Hat, Inc.