summaryrefslogtreecommitdiff
path: root/modules/extra/m_ldap_authentication.cpp
diff options
context:
space:
mode:
authorSadie Powell <sadie@witchery.services>2024-11-11 16:46:56 +0000
committerSadie Powell <sadie@witchery.services>2024-11-11 16:46:56 +0000
commit96ccfe4cbe9948206ac2d6854778f9a268d2476f (patch)
tree1e2059a204088051b0155b58de56bacaa2bfd58e /modules/extra/m_ldap_authentication.cpp
parentee160842b3ccab37f005c16a8657781e0e0412fd (diff)
Fix using User::Account where User::IsIdentified should be used.
The former causes a dereference which cause a database update. This is not good for performance with db_sql_live on bigger networks.
Diffstat (limited to 'modules/extra/m_ldap_authentication.cpp')
-rw-r--r--modules/extra/m_ldap_authentication.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/modules/extra/m_ldap_authentication.cpp b/modules/extra/m_ldap_authentication.cpp
index 5340038dd..1d44c807f 100644
--- a/modules/extra/m_ldap_authentication.cpp
+++ b/modules/extra/m_ldap_authentication.cpp
@@ -143,7 +143,7 @@ class OnIdentifyInterface : public LDAPInterface
{
User *u = User::Find(uid);
- if (!u || !u->Account() || r.empty())
+ if (!u || !u->IsIdentified() || r.empty())
return;
try