[Fedora-directory-commits] ldapserver/ldap/admin/src/scripts FileConn.pm, 1.6, 1.7

Richard Allen Megginson rmeggins at fedoraproject.org
Thu Mar 12 22:13:01 UTC 2009


Author: rmeggins

Update of /cvs/dirsec/ldapserver/ldap/admin/src/scripts
In directory cvs1.fedora.phx.redhat.com:/tmp/cvs-serv22797

Modified Files:
	FileConn.pm 
Log Message:
Resolves: bug 480869
Description:  Bug 480869 -  DS console: Can not delete DS instance
Fix Description: Create instance was broken, so no instances could be created for purposes of deletion.  Create instance was printing the following error:
Unable to find Pass Through Authentication Plug-In config entry.
This is because the search for this entry in AdminUtil.pm was getting an incorrect error message - something other than "Success"
This is because the FileConn->getErrorString() method was returning "0" instead of "Success".
Platforms tested: RHEL4



Index: FileConn.pm
===================================================================
RCS file: /cvs/dirsec/ldapserver/ldap/admin/src/scripts/FileConn.pm,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- FileConn.pm	27 Feb 2009 14:33:12 -0000	1.6
+++ FileConn.pm	12 Mar 2009 22:12:59 -0000	1.7
@@ -213,7 +213,7 @@
 
 sub getErrorString {
     my $self = shift;
-    return ($self->{lastErrorCode} ? ldap_err2string($self->{lastErrorCode}) : LDAP_SUCCESS);
+    return ldap_err2string($self->{lastErrorCode});
 }
 
 #############################################################################




More information about the Fedora-directory-commits mailing list