SimpleSAML_Auth_LDAP: Also include the cause in exceptions.

git-svn-id: https://simplesamlphp.googlecode.com/svn/trunk@2919 44740490-163a-0410-bde0-09ae8108e29a

lib/SimpleSAML/Auth/LDAP.php

@@ -141,18 +141,21 @@ class SimpleSAML_Auth_LDAP {
 					return new SimpleSAML_Error_AuthSource('ldap', $description);
 			}
 		}else{
+			if ($errNo !== 0) {
+				$description .= '; cause: \'' . ldap_error($this->ldap) . '\' (0x' . dechex($errNo) . ')';
+			}
 			switch ($errNo){
 				case 0x20://LDAP_NO_SUCH_OBJECT
-					SimpleSAML_Logger::warning($description . '; cause: \'' . ldap_error($this->ldap) . '\' (0x' . dechex($errNo) . ')');
+					SimpleSAML_Logger::warning($description);
 					return new SimpleSAML_Error_UserNotFound($description, $errNo);
 				case 0x31://LDAP_INVALID_CREDENTIALS
-					SimpleSAML_Logger::info($description . '; cause: \'' . ldap_error($this->ldap) . '\' (0x' . dechex($errNo) . ')');
+					SimpleSAML_Logger::info($description);
 					return new SimpleSAML_Error_InvalidCredential($description, $errNo);
 				case -1://NO_SERVER_CONNECTION
-					SimpleSAML_Logger::error($description . '; cause: \'' . ldap_error($this->ldap) . '\' (0x' . dechex($errNo) . ')');
+					SimpleSAML_Logger::error($description);
 					return new SimpleSAML_Error_AuthSource('ldap', $description);
 				default:
-					SimpleSAML_Logger::error($description . '; cause: \'' . ldap_error($this->ldap) . '\' (0x' . dechex($errNo) . ')');
+					SimpleSAML_Logger::error($description);
 					return new SimpleSAML_Error_AuthSource('ldap', $description);
 			}
 		}