diff options
| author | Rich Megginson <rmeggins@redhat.com> | 2009-03-13 14:27:50 +0000 |
|---|---|---|
| committer | Rich Megginson <rmeggins@redhat.com> | 2009-03-13 14:27:50 +0000 |
| commit | a0ab21a70e13be07542ea92374f278f56e29ccfd (patch) | |
| tree | de31d25388d32ef9ede56f81052396bafb6b4089 /ldap/admin/src/scripts/SetupLog.pm | |
| parent | 1b87897a086bbfcd621cfdba35b4890f1ece3e83 (diff) | |
| download | ds-a0ab21a70e13be07542ea92374f278f56e29ccfd.tar.gz ds-a0ab21a70e13be07542ea92374f278f56e29ccfd.tar.xz ds-a0ab21a70e13be07542ea92374f278f56e29ccfd.zip | |
Resolves: bug 489360
Bug Description: Replication Bind Failure After Migration from DS 7.1
Reviewed by: nkinder (Thanks!)
Fix Description: We have to quote shell metacharacters before passing them to the shell. I added a new function shellEscape to use for this purpose. We really should shell escape anything passed to system() or back ticks ``. Certainly passwords should contain shell meta characters so I changed places where we use passwords to use shellEscape to pass them to pwdhash or migratecred. I also chomp() the output of migratecred to remove the trailing newline. With the fix, I was able to run setup with a root password of `~!@#$%^&*()\\|[]{}:;<>?/"\ and successfully authenticate.
Platforms tested: RHEL5
Flag Day: no
Doc impact: no
Diffstat (limited to 'ldap/admin/src/scripts/SetupLog.pm')
0 files changed, 0 insertions, 0 deletions