[Freeipa-devel] [PATCH 0120] Fix automatic reloading of invalid zone after each change in zone data
Adam Tkac
atkac at redhat.com
Mon Mar 25 14:49:57 UTC 2013
On Thu, Mar 21, 2013 at 01:38:42PM +0100, Petr Spacek wrote:
> Hello,
>
> Fix automatic reloading of invalid zone after each change in zone data.
>
> Reload wasn't done when serial_autoincrement feature was disabled.
>
> https://fedorahosted.org/bind-dyndb-ldap/ticket/102
>
Ack.
But before the push, please add explicit comment to ldap_get_zone_serial()
call that the only reason of this call is to return ISC_R_SUCCESS in case the
zone is loaded or DNS_R_NOTLOADED in case it isn't. I studied the patch for more
then 15 minutes before I figured this.
Thanks, Adam
> From 1700a5d7dbf6c36ce235091a449e13a5e18fbb8b Mon Sep 17 00:00:00 2001
> From: Petr Spacek <pspacek at redhat.com>
> Date: Thu, 21 Mar 2013 13:35:17 +0100
> Subject: [PATCH] Fix automatic reloading of invalid zone after each change in
> zone data.
>
> Reload wasn't done when serial_autoincrement feature was disabled.
>
> https://fedorahosted.org/bind-dyndb-ldap/ticket/102
>
> Signed-off-by: Petr Spacek <pspacek at redhat.com>
> ---
> src/ldap_helper.c | 7 +++++++
> 1 file changed, 7 insertions(+)
>
> diff --git a/src/ldap_helper.c b/src/ldap_helper.c
> index c10a23929c1536961a37d18e68d0669aa26539de..6f21b8407e8c01a98ae5b6f916c964432c651fd5 100644
> --- a/src/ldap_helper.c
> +++ b/src/ldap_helper.c
> @@ -3572,8 +3572,15 @@ update_restart:
> CHECK(zr_get_zone_settings(inst->zone_register, &origin, &zone_settings));
> CHECK(setting_get_bool("serial_autoincrement", zone_settings,
> &serial_autoincrement));
> +
> + /* Serial autoincrement does zone state check implicitly.
> + * Do explicit state check if serial autoincrement is disabled. */
> if (serial_autoincrement)
> CHECK(soa_serial_increment(mctx, inst, &origin));
> + else {
> + isc_uint32_t dummy;
> + CHECK(ldap_get_zone_serial(inst, &origin, &dummy));
> + }
> }
>
> cleanup:
> --
> 1.7.11.7
>
--
Adam Tkac, Red Hat, Inc.
More information about the Freeipa-devel
mailing list