Full_Name: Michael Ströder
Version: HEAD
OS:
URL: ftp://ftp.openldap.org/incoming/
Submission from: (NULL) (217.92.147.99)
Feature request:
In accesslog the value of entryUUID of the entry affected by a write
operation should be logged in a separate attribute 'reqEntryUUID' so that this
write operations are also searchable in case an entry was renamed afterwards.
Searching in reqOld is not an option because for performance reasons one would
not index this multi-valued attribute which can contain lots of values.
Hello,
On Tue, Sep 21, 2010 at 12:12:54AM +0200, masarati(a)aero.polimi.it wrote:
> Fixed (differently) in HEAD; please test. Thanks for the report. p.
It works as expected! Thanks!
--=20
Johannes Wei=C3=9Fl
Full_Name: Pierangelo Masarati
Version: HEAD
OS: irrelevant
URL: ftp://ftp.openldap.org/incoming/
Submission from: (NULL) (188.153.50.138)
Submitted by: ando
lutil_atoix() is used to parse the version; since we only explicitly allow
LDIFv1, a string comparison suffices right now.
p.
Full_Name: Johannes Weißl
Version: HEAD
OS: Debian GNU/Linux AMD64
URL: ftp://ftp.openldap.org/incoming/
Submission from: (NULL) (93.104.127.153)
This is only a minor issue:
When URI is e.g. ldapi:// in ldap.conf and ldap_get_option() for
LDAP_OPT_HOST_NAME is called, ldap_url_list2hosts() [url.c:1416] segfaults
because the lud_host field in LDAPURLDesc is NULL. Adding an additional test in
the loop header fixes this bug:
diff -Naur openldap/libraries/libldap/url.c
openldap.new/libraries/libldap/url.c
--- openldap/libraries/libldap/url.c 2010-04-13 22:17:57.000000000 +0200
+++ openldap.new/libraries/libldap/url.c 2010-09-20 22:44:58.325892001 +0200
@@ -1424,7 +1424,7 @@
/* figure out how big the string is */
size = 1; /* nul-term */
- for (ludp = ludlist; ludp != NULL; ludp = ludp->lud_next) {
+ for (ludp = ludlist; ludp != NULL && ludp->lud_host != NULL; ludp =
ludp->lud_next) {
size += strlen(ludp->lud_host) + 1; /* host and space */
if (strchr(ludp->lud_host, ':')) /* will add [ ] below */
size += 2;
@@ -1436,7 +1436,7 @@
return NULL;
p = s;
- for (ludp = ludlist; ludp != NULL; ludp = ludp->lud_next) {
+ for (ludp = ludlist; ludp != NULL && ludp->lud_host != NULL; ludp =
ludp->lud_next) {
if (strchr(ludp->lud_host, ':')) {
p += sprintf(p, "[%s]", ludp->lud_host);
} else {