[389-commits] ldap/admin

ldap/admin/src/scripts/ns-accountstatus.pl.in | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit c39bed2e05cca1c73354dcca9f39f3d90594dc9a
Author: Mark Reynolds <mreynolds@redhat.com>
Date: Tue May 3 15:45:41 2016 -0400

Ticket 48815 - ns-accountstatus.sh does handle DN's with single quotes

Bug Description: When a DN is encountered that has a single quote in it
the script fails because of how the script quotes the DN.

Fix Description: There is no need to quote the DN when using it in a filter.

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

Reviewed by: nhosoi(Thanks!)

diff --git a/ldap/admin/src/scripts/ns-accountstatus.pl.in b/ldap/admin/src/scripts/ns-accountstatus.pl.in
index 5d92220..0d52ec9 100644
--- a/ldap/admin/src/scripts/ns-accountstatus.pl.in
+++ b/ldap/admin/src/scripts/ns-accountstatus.pl.in
@@ -643,7 +643,7 @@ sub getSuffix
#
debug("\tSuffix from the entry: #@suffixN#\n");
$info{base} = "cn=mapping tree, cn=config";
- $info{filter} = "cn=\"@suffixN\"";
+ $info{filter} = "cn=@suffix";
$info{scope} = "one";
$info{attrs} = "cn";
@mapping = DSUtil::ldapsrch_ext(%info);
@@ -849,7 +849,7 @@ for(my $i = 0; $i <= $#entries; $i++){
}

#
- # Gather the Account Ppoliy PLugin information(if available)
+ # Gather the Account Policy Plugin information (if available)
#
($acct_policy_enabled, $stateattr, $altstateattr, $limit) = getAcctPolicy(\%info, $entry);

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