[389-commits] ldap/admin

Richard Allen Megginson rmeggins at fedoraproject.org
Wed Mar 3 20:31:28 UTC 2010


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

New commits:
commit e8f50642bd3e19ad528b453850304611ab86506d
Author: Endi S. Dewata <edewata at redhat.com>
Date:   Wed Mar 3 13:25:45 2010 -0600

    Bug 545620 - Password cannot start with minus sign
    
    https://bugzilla.redhat.com/show_bug.cgi?id=545620
    
    Previously getopt would interpret initial '-' in root password
    as an option which could lead to setup failure. Now a special
    argument '--' has been added before the password to distinguish
    it from other options.

diff --git a/ldap/admin/src/scripts/DSUtil.pm.in b/ldap/admin/src/scripts/DSUtil.pm.in
index 7e846d7..79586db 100644
--- a/ldap/admin/src/scripts/DSUtil.pm.in
+++ b/ldap/admin/src/scripts/DSUtil.pm.in
@@ -736,7 +736,7 @@ sub getHashedPassword {
     if ($alg) {
         $cmd .= " -s $alg";
     }
-    $cmd .= " " . shellEscape($pwd);
+    $cmd .= " -- " . shellEscape($pwd);
     my $hashedpwd = `$cmd`;
     chomp($hashedpwd);
 




More information about the 389-commits mailing list