Wednesday, April 27, 2016

[389-commits] ldap/admin

ldap/admin/src/scripts/DSUtil.pm.in | 1 +
1 file changed, 1 insertion(+)

New commits:
commit 0d1c21b0b96f3c42b334a74db0c5aa090a0339b9
Author: Noriko Hosoi <nhosoi@redhat.com>
Date: Wed Apr 27 11:31:49 2016 -0700

Ticket #48410 - 389-ds-base - Unable to remove / unregister a DS instance from admin server

Description: In check_and_add_entry (DSUtil.pm), if the operation is delete,
$subentry is exhausted. Thus, the do loop should be finished there without
calling $conn->nextEntry(), which causes the assertion failure in perl-Mozilla-
LDAP.

https://fedorahosted.org/389/ticket/48410

Reviewed by mreynolds@redhat.com (Thank you, Mark!!)

diff --git a/ldap/admin/src/scripts/DSUtil.pm.in b/ldap/admin/src/scripts/DSUtil.pm.in
index 3476d67..f53f0c0 100644
--- a/ldap/admin/src/scripts/DSUtil.pm.in
+++ b/ldap/admin/src/scripts/DSUtil.pm.in
@@ -463,6 +463,7 @@ sub check_and_add_entry
return 0;
}
debug(1, "Entry $aentry->{dn} is deleted\n");
+ $sentry = undef;
}
elsif ( 0 < $op ) # modify op
{

--
389-commits mailing list
389-commits@lists.fedoraproject.org
http://lists.fedoraproject.org/admin/lists/389-commits@lists.fedoraproject.org

No comments:

Post a Comment