From 11f8ecf946c0b56d7d49f7b46429d153d5c19ee1 Mon Sep 17 00:00:00 2001 From: German Parente Date: Wed, 10 Oct 2018 10:24:08 +0200 Subject: [PATCH] Ticket #49946 upgrade of 389-ds-base could remove replication agreements. Bug Description: when a replication agreement starts with "cn=->...", the upgrade is removing the entry. Fix Description: a check is missing when re-building dse.ldif in "setup-ds.pl -u" that provoked this entry not to be re-added to the file. https://pagure.io/389-ds-base/issue/49946 Author: German Parente Review by: ??? --- ldap/admin/src/scripts/FileConn.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ldap/admin/src/scripts/FileConn.pm b/ldap/admin/src/scripts/FileConn.pm index dcea70904..8a2a1afe6 100644 --- a/ldap/admin/src/scripts/FileConn.pm +++ b/ldap/admin/src/scripts/FileConn.pm @@ -360,7 +360,7 @@ sub add { return $self->write(); } - if (exists($self->{$ndn})) { + if ($ndn && exists($self->{$ndn})) { $self->setErrorCode(LDAP_ALREADY_EXISTS); return 0; } -- 2.17.2