extensions/Library_ldapbe2.mk | 1
extensions/source/config/ldap/ldapuserprofilebe.cxx | 41 +++++++++-----------
2 files changed, 21 insertions(+), 21 deletions(-)
New commits:
commit 5ed889e80805daf8471054786c1e424a73744ad3
Author: Mike Kaganski <mike.kagan...@collabora.com>
Date: Wed May 9 16:19:46 2018 +0300
Don't throw in LdapUserProfileBe ctor on LDAP misconfiguration
Otherwise LO crashes e.g. at opening Expert Configuration dialog.
Instead, log the event, create an empty backend, and return empty values.
Change-Id: I433fc89c003e7886dfc5242a4ef8defa46d643d3
Reviewed-on: https://gerrit.libreoffice.org/53958
Reviewed-by: Stephan Bergmann <sberg...@redhat.com>
Tested-by: Jenkins <c...@libreoffice.org>
diff --git a/extensions/Library_ldapbe2.mk b/extensions/Library_ldapbe2.mk
index 0364d58e8bd1..5ea5a9597101 100644
--- a/extensions/Library_ldapbe2.mk
+++ b/extensions/Library_ldapbe2.mk
@@ -28,6 +28,7 @@ $(eval $(call gb_Library_add_exception_objects,ldapbe2,\
))
$(eval $(call gb_Library_use_libraries,ldapbe2,\
+ comphelper \
cppuhelper \
cppu \
salhelper \
diff --git a/extensions/source/config/ldap/ldapuserprofilebe.cxx
b/extensions/source/config/ldap/ldapuserprofilebe.cxx
index 00fdf01e22df..a1208f4bd578 100644
--- a/extensions/source/config/ldap/ldapuserprofilebe.cxx
+++ b/extensions/source/config/ldap/ldapuserprofilebe.cxx
@@ -30,6 +30,7 @@
#include <com/sun/star/beans/NamedValue.hpp>
#include <com/sun/star/beans/Optional.hpp>
#include <com/sun/star/configuration/theDefaultProvider.hpp>
+#include <comphelper/scopeguard.hxx>
#include <cppuhelper/supportsservice.hxx>
#include <osl/security.hxx>
@@ -42,6 +43,8 @@ LdapUserProfileBe::LdapUserProfileBe( const
uno::Reference<uno::XComponentContex
{
LdapDefinition aDefinition;
OUString loggedOnUser;
+ // true initially to handle reentrant call; will become false if
readLdapConfiguration fails
+ bool bHaveLdapConfiguration = true;
// This whole rigmarole is to prevent an infinite recursion where reading
// the configuration for the backend would create another instance of the
@@ -55,30 +58,26 @@ LdapUserProfileBe::LdapUserProfileBe( const
uno::Reference<uno::XComponentContex
if (!bReentrantCall)
{
- try
- {
- bReentrantCall = true ;
- if (!readLdapConfiguration(
- xContext, &aDefinition, &loggedOnUser))
- {
- throw css::uno::RuntimeException(
- "LdapUserProfileBe- LDAP not configured",
- nullptr);
- }
-
- bReentrantCall = false ;
- }
- catch (...)
- {
- bReentrantCall = false;
- throw;
- }
+ bReentrantCall = true ;
+ comphelper::ScopeGuard aReentrantCallGuard([]() { bReentrantCall =
false; });
+ // Don't throw on fail: this will crash if LDAP is misconfigured,
and user opens
+ // Expert Configuration dialog. Instead, just don't fill data_,
which will make the
+ // backend return empty values. This happens in
SvtUserOptions::Impl::GetValue_Impl
+ // anyway even in throwing scenario, but doing it here also
improves performance
+ // because of avoiding repeated attempts to create the backend.
+ bHaveLdapConfiguration = readLdapConfiguration(
+ xContext, &aDefinition, &loggedOnUser);
+ if (!bHaveLdapConfiguration)
+ SAL_WARN("extensions.config", "LdapUserProfileBackend: LDAP
not configured");
}
}
- LdapConnection connection;
- connection.connectSimple(aDefinition);
- connection.getUserProfile(loggedOnUser, &data_);
+ if (bHaveLdapConfiguration)
+ {
+ LdapConnection connection;
+ connection.connectSimple(aDefinition);
+ connection.getUserProfile(loggedOnUser, &data_);
+ }
}
LdapUserProfileBe::~LdapUserProfileBe()
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits
