Bug#877508: ldap2zone: fails to check serial of zone

[Francois Masson]

Package: ldap2zone
Version: 0.2-9
Severity: important
Tags: patch

Dear Maintainer,

Using ldap2zone with a serial lead to infinite loop.

ex : ldap2zone "96.168.192.in-addr.arpa" ldap://localhost/ou=DNS,ou=Ressources,dc=example.com 259200 2017100200

Error come from using wrong var in loop. See attached patch.

-- System Information:
Debian Release: 9.1
  APT prefers stable-updates
  APT policy: (500, 'stable-updates'), (500, 'stable')
Architecture: amd64 (x86_64)

Kernel: Linux 4.9.0-3-amd64 (SMP w/1 CPU core)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8), LANGUAGE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)

Versions of packages ldap2zone depends on:
ii  bind9          1:9.10.3.dfsg.P4-12.3+deb9u3
ii  libc6          2.24-11+deb9u1
ii  libldap-2.4-2  2.4.44+dfsg-5

ldap2zone recommends no packages.

ldap2zone suggests no packages.

-- no debconf information
-------------- next part --------------
--- ldap2zone-0.2/ldap2zone.c	2017-10-02 11:27:59.000000000 +0200
+++ ldap2zone-0.2.b/ldap2zone.c	2017-10-02 11:29:53.551182216 +0200
@@ -301,7 +301,7 @@
 	if (msgid == -1)
 	    err(argv[0], "ldap_search() failed");
 
-	while ((rc = ldap_result(ld, msgid, 0, NULL, &res)) != LDAP_RES_SEARCH_RESULT ) {
+	while ((rc = ldap_result(ld, msgidp, 0, NULL, &res)) != LDAP_RES_SEARCH_RESULT ) {
 	    /* not supporting continuation references at present */
 	    if (rc != LDAP_RES_SEARCH_ENTRY)
 		err(argv[0], "ldap_result() returned cont.ref? Exiting");