ldap/admin/src/scripts/ns-accountstatus.pl.in | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
New commits:
commit c39bed2e05cca1c73354dcca9f39f3d90594dc9a
Author: Mark Reynolds <mreynolds(a)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);