openldap-bugs July 2007

openldap-bugs@openldap.org

30 participants
126 discussions

Re: imporvement to slapo-constraint (ITS#4987)
by manu＠netbsd.org 07 Jul '07

07 Jul '07

> Did you see http://www.openldap.org/devel/contributing.html Right, but before moving foward with appropriate style, documentation, and so on, I'd like to have some input about the config syntax and the way it's implemented. I had no feedback at all yet on that. -- Emmanuel Dreyfus http://hcpnet.free.fr/pubz manu(a)netbsd.org

1 0

start_tls while chasing referrals
by Bin Lu 06 Jul '07

06 Jul '07

Hi, I noticed the following bug fix in referral chasing http://bugzilla.padl.com/show_bug.cgi?id=210 This seems only to take care of the usage with pam ldap lib. What if the ldap connection is not from the pam lib? In that case, when an ldap operation reaches a referral point, would the new connection be consistent if the original connection is using TLS(and the referral url is not using ldaps)? Our test shows it is not. Please advice, if that is also a security hole? Regards, Wenwu

2 1

Re: (ITS#5041) ldapwhoami takes unused "user" arg
by ando＠sys-net.it 05 Jul '07

05 Jul '07

h.b.furuseth(a)usit.uio.no wrote: > ldapwhoami and "ldapexop whoami" take an unused command line > argument "user". The code has been there since rev 1.1. could be a leftover or something like that? In any case, just trimmed. p. Ing. Pierangelo Masarati OpenLDAP Core Team SysNet s.r.l. via Dossi, 8 - 27100 Pavia - ITALIA http://www.sys-net.it --------------------------------------- Office: +39 02 23998309 Mobile: +39 333 4963172 Email: pierangelo.masarati(a)sys-net.it ---------------------------------------

1 0

(ITS#5041) ldapwhoami takes unused "user" arg
by h.b.furuseth＠usit.uio.no 05 Jul '07

05 Jul '07

Full_Name: Hallvard B Furuseth Version: 1.1->current OS: URL: Submission from: (NULL) (129.240.6.233) Submitted by: hallvard ldapwhoami and "ldapexop whoami" take an unused command line argument "user". The code has been there since rev 1.1.

1 0

(ITS#5040) modifyTimestamp being updated on login (bind) failure
by dan.cushing＠netideasinc.com 05 Jul '07

05 Jul '07

Full_Name: Dan Cushing Version: 2.3.36 OS: Solaris 9 URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (71.76.187.82) When running OpenLDAP with the ppolicy overlay, the modifyTimestamp for a user entry is updated if the user attempts to login (bind) with an incorrect password. This is happening because the password lockout feature is enabled and the operational attribute 'pwdFailureTime' is being updated. It seems like this results in a misleading modifyTimestamp. Is it intended that the modifyTimestamp attribute be updated when operational attributes are updated? To reproduce the issue, compile with the ppolicy overlay and enable password lockout. Below I've included the entries for a minimal policy.ldif file that I tested with. Then perform the following actions. 1) As the directory administrator execute the following search: ldapsearch -x -D "cn=Manager,o=testorg.com" -w password -b o=testorg.com uid=testuser pwdFailureTime modifyTimestamp. This will return something like "modifyTimestamp: 20070705142825Z" 2) As testuser execute the following search: ldapsearch -x -D "uid=testuser,o=testorg.com" -w correctPassword -b o=testorg.com uid=testuser modifyTimestamp. This will also return something like "modifyTimestamp: 20070705142825Z" 3) As testuser execute the following search (note the INCORRECT password): ldapsearch -x -D "uid=testuser,o=testorg.com" -w WRONGPassword -b o=testorg.com uid=testuser modifyTimestamp. This will also return something like "ldap_bind: Invalid credentials (49)" 4) As the directory administrator execute the following search: ldapsearch -x -D "cn=Manager,o=testorg.com" -w password -b o=testorg.com uid=testuser pwdFailureTime modifyTimestamp. This will return something like "pwdFailureTime: 20070705144735Z modifyTimestamp: 20070705144735Z". Note that the 'pwdFailureTime attribute has been created and the 'modifyTimestamp' attribute has been updated to the time that the bind attempt failed. Contents of my Standard policy file are ... dn: cn=Standard, ou=Policies, o=testorg.com pwdLockout: TRUE pwdMaxFailure: 3 objectClass: top objectClass: device objectClass: pwdPolicy pwdFailureCountInterval: 120 pwdLockoutDuration: 3600 cn: Standard pwdAttribute: userPassword

1 0

Re: (ITS#5035) Warning patrol
by ando＠sys-net.it 04 Jul '07

04 Jul '07

h.b.furuseth(a)usit.uio.no wrote: > #1. slapd/frontend.c:frontend_init() has the statement > if ( slap_known_controls ) > which is always true since slap_known_controls is an array. > Maybe it should be if ( slap_known_controls[0] )? > > Except... controls.c says slap_known_controls[0] == NULL, which is > not true - at least not at the time of the if-test above. What's > that about? Probably a leftover of a time when slap_known_controls was a pointer > #2. The return value from op->o_callback->sc_cleanup() is never > used. Should it be? I noticed because the pcache_op_cleanup() > function does not return anything. What should it return? two possibilities: a) ignore it: cleanup could occur too late, when response has already been returned to client; in this case a return code is there only because of the standard API b) handle it: cleanup return code could be usefully returned to caller if (a), then the return code can be safely ignored > #3. RE23 struct sync_cookie.rid and struct syncinfo_s.si_rid are > 'long', but are printed as 'int'. Could change the formats, but - > any reason not to instead change their types to 'int' as in HEAD? > I haven't looked at the HEAD syncrepl changes, so I don't know > what other issues might be involved. There should be no impact, since they're both limited to 0,999 > #4. bconfig.c: Integer overflow in (1 << (sizeof(int)*8 - 1)) in > slap_loglevel_get(). If anything else have been trusting signed > integer overflow to resemble unsigned integers, it's time to cease > that. The behaviour is undefined, and compilers make use of this > fact to optimize. > > The loglevel functions are confused about types anyway. They > collect masks in slap_mask_t, but the API uses an int for the > masks. Agree > Either the API should use slap_mask_t Agree > or masks should be > stored in (unsigned) ints. Not for 2.3, I think. Agree p. Ing. Pierangelo Masarati OpenLDAP Core Team SysNet s.r.l. via Dossi, 8 - 27100 Pavia - ITALIA http://www.sys-net.it --------------------------------------- Office: +39 02 23998309 Mobile: +39 333 4963172 Email: pierangelo.masarati(a)sys-net.it ---------------------------------------

1 0

Re: (ITS#5036) test034-translucent fails on HEAD
by ghenry＠suretecsystems.com 04 Jul '07

04 Jul '07

<quote who="hyc(a)symas.com"> > hyc(a)symas.com wrote: >> h.b.furuseth(a)usit.uio.no wrote: >>> Full_Name: Hallvard B Furuseth >>> Version: HEAD >>> OS: Linux >>> URL: ftp://ftp.openldap.org/incoming/Hallvard-Furuseth.070703.tgz.1 >>> Submission from: (NULL) (129.240.202.105) >>> Submitted by: hallvard >>> >>> >>> test034-translucent fails. Output: >> >>> Configuration: --disable-dynamic --enable-crypt --enable-lmpasswd >>> --enable-spasswd --enable-rlookups --enable-slapi --enable-wrappers >>> --disable-backends --enable-bdb --enable-ldap --enable-relay >>> --enable-null >>> --enable-meta --enable-overlays CC=gcc-4.2.0 >>> 'CPPFLAGS=-DLDAP_DEPRECATED >>> -DLDAP_THREAD_DEBUG -DLDAP_RDWR_DEBUG' >> >> Strange. I got the same result with those options. But with this >> configuration, >> the test works fine: >> >> --enable-modules --enable-backends=mod --enable-overlays=mod >> --enable-slapi >> --disable-sql >> >> The test also fails with your options, but without any of the CPPFLAGS >> set. >> > Duh... In your test back-monitor is missing. Usually the monitor would be > olcDatabase={1} in the config, so the bdb instance would be > olcDatabase={2}, > and that's the name that's coded into the script. > I'm trying to get push based Syncrepl going for the docs and this test is also failing for me with HEAD updated 2 hrs ago. My options: env CC=gcc CPPFLAGS="-I/usr/local/BerkeleyDB.4.2/include" \ LDFLAGS="-L/usr/local/BerkeleyDB.4.2/lib" \ LD_LIBRARY_PATH="/usr/local/BerkeleyDB.4.2/lib" ./configure \ --prefix=/opt/suretec/ldap --enable-slapd --enable-syslog --with-cyrus-sasl=yes \ --enable-dynamic --enable-rewrite --disable-ipv6 --disable-shell \ --disable-sql --with-threads --enable-modules --enable-backends=mod \ --enable-overlays=mod --with-tls --enable-wrappers Gavin.

1 0

Re: (ITS#5032) cannot build on x86_64, needs newer libtool
by h.b.furuseth＠usit.uio.no 04 Jul '07

04 Jul '07

Buchan Milne writes: >> libtool translates -ldb-4.3 to /usr/lib/libdb-4.3.so, which breaks. >> "cc -ldb-4.3" uses /lib64/libdb-4.3.so. > > Only if the 32bit package is installed (which unfortunately happens by > default e.g. with 'yum install db4-devel' you get both i386 and > x86_64). OK. A libtool bug triggered by a RedHat irritant (but still non-bug). > On RHEL5: > # rpm -e db4-devel-4.3.29-9.fc6.i386 > works around this. Thanks. The system folks here say we need some of the 32-bit stuff, though:-( I wrote: > Is OpenLDAP using a patched libtool? One of the differences is that > ltmain.sh contains VERSION=1.5.22-OpenLDAP, while I get VERSION=1.5.22. Er, yes, it seems so. I was even one of the sinners, though those patches are gone now. It doesn't mention that it is a generated file. -- Regards, Hallvard

1 0

(ITS#5039) Support config for start_tls in ldap.conf
by ahasenack＠terra.com.br 04 Jul '07

04 Jul '07

Full_Name: Andreas Hasenack Version: 2.3.36 OS: Linux URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (200.140.247.98) This is an enhancement request. It would be helpfull if there were some configuration option for /etc/openldap/ldap.conf to mimic the -ZZ command line, that is, behave as if the user added -ZZ to it. Perhaps something along the lines of the sasl secprops, or the server's "security" keyword. My scenario is that it doesn't matter if I block clear text communication with the ldap server via ACL or security: if the client initiates a simple bind operation in clear text, the password is exposed even if the server rejects the operation. The point here is to avoid accidents, like leaving out the -ZZ option when doing command line operations. It would be like an initial default. It also saves typing, of course ;) I can easily workaround this with shell aliases, or wrapper scripts, of course. That's why this is an enhancement request.

1 0

Re: (ITS#5037) slapd-bdb filterindex.c: out of order if statement in filter
by quanah＠zimbra.com 03 Jul '07

03 Jul '07

--On Wednesday, July 04, 2007 1:30 AM +0000 openldap-its(a)OpenLDAP.org wrote: > > *** THIS IS AN AUTOMATICALLY GENERATED REPLY *** nm, the function call sets db. But still the if needs to be re-ordered. ;) --Quanah -- Quanah Gibson-Mount Principal Software Engineer Zimbra, Inc -------------------- Zimbra :: the leader in open source messaging and collaboration

1 0

← Newer
1
...
7
8
9
10
11
12
13
Older →

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

openldap-bugs July 2007