Bug 54971 - various UCR variables may break slapd.conf
various UCR variables may break slapd.conf
Status: NEW
Product: UCS
Classification: Unclassified
Component: LDAP
UCS 5.0
Other Linux
: P5 normal (vote)
: ---
Assigned To: UCS maintainers
UCS maintainers
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2022-07-11 17:40 CEST by Dirk Ahrnke
Modified: 2022-07-11 17:43 CEST (History)
5 users (show)

See Also:
What kind of report is it?: Bug Report
What type of bug is this?: 3: Simply Wrong: The implementation doesn't match the docu
Who will be affected by this bug?: 3: Will affect average number of installed domains
How will those affected feel about the bug?: 2: A Pain – users won’t like this once they notice it
User Pain: 0.103
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Dirk Ahrnke univentionstaff 2022-07-11 17:40:40 CEST
+++ This bug was initially created as a clone of Bug #31606 +++

It is possible to silently prevent slapd from starting by using comma and whitespace to separate IP addresses in ldap/acl/read/ips.

A simple strip() could prevent that from happening:

# diff -Nur 70univention-ldap-server_acl-master-end.orig 70univention-ldap-server_acl-master-end
--- 70univention-ldap-server_acl-master-end.orig        2013-06-03 10:27:49.000000000 +0200
+++ 70univention-ldap-server_acl-master-end     2013-06-03 10:28:01.000000000 +0200
@@ -142,7 +142,7 @@
        ldap_acl_read_anonymous_ips = configRegistry.get('ldap/acl/read/ips')
        if ldap_acl_read_anonymous_ips:
                for ip in ldap_acl_read_anonymous_ips.split(','):
-                       print '   by peername.ip=%s read' % ip
+                       print '   by peername.ip=%s read' % ip.strip()
 else:
        print '   by * read'
Comment 1 Dirk Ahrnke univentionstaff 2022-07-11 17:41:46 CEST
reproduced with 5.0-2 by unsetting ldap/threads