openldap-bugs

openldap-bugs@openldap.org

1 participants
20962 discussions

Re: (ITS#5342) DN and naming attribute mismatch
by hyc＠symas.com 30 Jan '08

30 Jan '08

Aaron Richton wrote: >> after it. ("Last" is obviously imprecise. But the point is you would have had >> to have created some new entries after the upgrade whose DNs would cause >> their records to fall on the same DB page as this record, meaning they're >> close in the sort order and such.) > > As for "close pagewise," there's a lot of churn here since it's the > beginning of the semester. It's not uncommon to see the same DN DEL/ADD > within ~30-60 minutes. > >> And for most err=80 situations, there ought to be BerkeleyDB error messages >> in the log as well. > > Agreed. On that end, I pulled out two weeks of logs overnight, and grep > for "bdb(" found precisely zero hits. > > > While I had the logs out, I looked back, and it turns out the first time I > ever saw a problem with 2.3.40 actually WASN'T err=80: > > Jan 24 19:03:31 op=4 DEL dn="cn=172.23.132.58,cn..." > Jan 24 19:03:31 conn=3252 op=4 RESULT tag=107 err=32 text= > > > > This was added under 2.3.37: > > Jan 22 18:33:28 op=4 ADD dn="cn=172.23.132.58,cn=..." > > Now of course you can blame stuff between Jan 22-24 as moving that entry. > But I've got nightly slapcat's showing: > > dn: cn=172.23.132.58,cn=Clients,cn=ResNet Service Config,ou=Host Config,o=Rutg > entryCSN: 20080122233328Z#000004#00#000000 > > As a matter of fact, I can even run that right this second. > > # date;slapcat -b "o=Rutgers CCF,c=US" | /usr/sfw/bin/ggrep -A12 cn=172.23.132.58 | egrep '^cn|^entryCSN' > Wed Jan 30 09:29:03 EST 2008 > cn: 172.23.132.58 > entryCSN: 20080122233328Z#000004#00#000000 > > So notfound is.....dubious. (And no, I'm not manually tweaking it, and I > don't even know how to spell managedSAit. ;) That's the first time I saw > 2.3.40 go weird. Well, one of the big patches in 2.3.40 (relative to 2.3.39) was a fix for some caching bugs which showed up if an entry was being deleted and added in rapid succession, and being searched for. In that case, it was possible for the old dn2id value to be read back into the cache from disk before it got deleted off the disk. So it's conceivable that this patch caused your err=32 problem somehow. But nothing explains the errant bit getting set in the DN. Have you got any servers running 2.3.39, or all 2.3.37? -- -- Howard Chu Chief Architect, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

Re: (ITS#5343) cn=config crash when deleting attrs
by h.b.furuseth＠usit.uio.no 30 Jan '08

30 Jan '08

Ralf Haferkamp writes: > This should be fixed in HEAD now. Yes. Instead test034-translucent fails: >>>>> Starting test034-translucent ... ... Dynamically configuring local slapd without translucent_no_glue... ldap_modify: No such attribute (16) additional info: modify/delete: entryCSN: no such attribute modifying entry "olcOverlay={0}translucent,olcDatabase={2}bdb,cn=config" ldapmodify of dynamic config failed (16) >>>>> scripts/test034-translucent with bdb failed (exit 1) -- Hallvard

1 0

Re: (ITS#5342) DN and naming attribute mismatch
by richton＠nbcs.rutgers.edu 30 Jan '08

30 Jan '08

> after it. ("Last" is obviously imprecise. But the point is you would have had > to have created some new entries after the upgrade whose DNs would cause > their records to fall on the same DB page as this record, meaning they're > close in the sort order and such.) As for "close pagewise," there's a lot of churn here since it's the beginning of the semester. It's not uncommon to see the same DN DEL/ADD within ~30-60 minutes. > And for most err=80 situations, there ought to be BerkeleyDB error messages > in the log as well. Agreed. On that end, I pulled out two weeks of logs overnight, and grep for "bdb(" found precisely zero hits. While I had the logs out, I looked back, and it turns out the first time I ever saw a problem with 2.3.40 actually WASN'T err=80: Jan 24 19:03:31 op=4 DEL dn="cn=172.23.132.58,cn..." Jan 24 19:03:31 conn=3252 op=4 RESULT tag=107 err=32 text= This was added under 2.3.37: Jan 22 18:33:28 op=4 ADD dn="cn=172.23.132.58,cn=..." Now of course you can blame stuff between Jan 22-24 as moving that entry. But I've got nightly slapcat's showing: dn: cn=172.23.132.58,cn=Clients,cn=ResNet Service Config,ou=Host Config,o=Rutg entryCSN: 20080122233328Z#000004#00#000000 As a matter of fact, I can even run that right this second. # date;slapcat -b "o=Rutgers CCF,c=US" | /usr/sfw/bin/ggrep -A12 cn=172.23.132.58 | egrep '^cn|^entryCSN' Wed Jan 30 09:29:03 EST 2008 cn: 172.23.132.58 entryCSN: 20080122233328Z#000004#00#000000 So notfound is.....dubious. (And no, I'm not manually tweaking it, and I don't even know how to spell managedSAit. ;) That's the first time I saw 2.3.40 go weird.

1 0

Re: (ITS#5343) cn=config crash when deleting attrs
by rhafer＠suse.de 30 Jan '08

30 Jan '08

On Mittwoch, 30. Januar 2008, h.b.furuseth(a)usit.uio.no wrote: [..] > > back-config crashes if I delete some attribute, even some which > does not exist, and then olcAttributeOptions: > > bash$ cat slapd.conf > include servers/slapd/schema/core.schema > database config > rootdn cn=config > rootpw secret > > bash$ ldapmodify -xh:3890 -Dcn=config -wsecret > dn: cn=config > changetype: modify > # or use "delete: o" > delete: olcAllows > - > delete: olcAttributeOptions > olcAttributeOptions: foo > - > > Crash in bconfig.c:config_modify_internal line 4843 because d == > NULL: 4842 if ( ml->sml_values ) { > 4843 d = d->next; > 4844 ch_free( dels ); > 4845 dels = d; > 4846 } [..] This should be fixed in HEAD now. The code did not check the return code of modify_delete_vindex() for failures. > Also the error message from deleting just olcAttributeOptions is bad: > > bash$ ldapmodify -xh:3890 -Dcn=config -wsecret > dn: cn=config > changetype: modify > replace: olcAttributeOptions > olcAttributeOptions: foo > > ldap_modify: Internal (implementation specific) error (80) > additional info: modify/delete: modifyTimestamp: no such attribute > modifying entry "cn=config" > > Finally "replace:" only needs to fail if there are options to delete, > not if one adds a line with just "attributeOptions" to slapd.conf. > That removes the default, "lang-". Haven't looked into this yet. -- Ralf

1 0

Re: (ITS#5344) Wrong check for bad Modify DN
by h.b.furuseth＠usit.uio.no 30 Jan '08

30 Jan '08

hyc(a)symas.com writes: > Probably should look at adding the dest_dn to the op struct, so each > backend doesn't have to rebuild it. Third-party overlays which modify newrdn or newSuperior would become incorrect, so maybe this change should wait for OpenLDAP 2.5. -- Hallvard

1 0

Re: (ITS#5344) Wrong check for bad Modify DN
by h.b.furuseth＠usit.uio.no 30 Jan '08

30 Jan '08

hyc(a)symas.com writes: >> The slapd/modrdn.c check for affectsMultipleDSAs is insufficient, it >> requires newSuperior to be in the same backend. That does not catch >> moving "cn=x,cn=y" to another database's suffix "cn=z,cn=y". > > I don't see how it can miss this. It checks if newSuperior is in another backend, but that move doesn't need a newSuperior. It keeps the same parent: database 1: suffix cn=y database 2: suffix cn=z,cn=y modify rdn: dn "cn=x,cn=y"; newrdn cn=z; newSuperior (if any) cn=y. > Probably should look at adding the dest_dn to the op struct, so each > backend doesn't have to rebuild it. Indeed. And dest_ndn. -- Hallvard

1 0

Re: (ITS#5344) Wrong check for bad Modify DN
by hyc＠symas.com 30 Jan '08

30 Jan '08

h.b.furuseth(a)usit.uio.no wrote: > Full_Name: Hallvard B Furuseth > Version: HEAD > OS: > URL: http://folk.uio.no/hbf/OpenLDAP/modrdn.diff > Submission from: (NULL) (129.240.6.233) > Submitted by: hallvard > > > The slapd/modrdn.c check for affectsMultipleDSAs is insufficient, it > requires newSuperior to be in the same backend. That does not catch > moving "cn=x,cn=y" to another database's suffix "cn=z,cn=y". I don't see how it can miss this. > Also > if a database has multiple suffixes it prevents moving an entry > to one of the suffixes. OK, I see that. > slapd/modrdn.c can catch attempts to place an entry above/below itself. > It doesn't need to send those to the database and hope that catches it. > This patch puts it in the frontend (fe_op_modrdn); that needed to > generate the destination DN anyway for the affectsMultipleDSAs check. > Not sure if that's right or if it should go in do_modrdn() instead. > (I'm thinking of DN rewriting in rwm, vs. global overlays.) Probably should look at adding the dest_dn to the op struct, so each backend doesn't have to rebuild it. > In test005-modrdn, the "modrdn with newSuperior as child of target" check > would not catch anything: It expects failure, but the newSuperior did > not exist and would fail with noSuchObject in any case. This patch uses > newSuperior=target instead, and expects unwillingToPerform. > I'll apply the patch later, unless someone thinks it should be done > differently. -- -- Howard Chu Chief Architect, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

(ITS#5344) Wrong check for bad Modify DN
by h.b.furuseth＠usit.uio.no 30 Jan '08

30 Jan '08

Full_Name: Hallvard B Furuseth Version: HEAD OS: URL: http://folk.uio.no/hbf/OpenLDAP/modrdn.diff Submission from: (NULL) (129.240.6.233) Submitted by: hallvard The slapd/modrdn.c check for affectsMultipleDSAs is insufficient, it requires newSuperior to be in the same backend. That does not catch moving "cn=x,cn=y" to another database's suffix "cn=z,cn=y". Also if a database has multiple suffixes it prevents moving an entry to one of the suffixes. slapd/modrdn.c can catch attempts to place an entry above/below itself. It doesn't need to send those to the database and hope that catches it. This patch puts it in the frontend (fe_op_modrdn); that needed to generate the destination DN anyway for the affectsMultipleDSAs check. Not sure if that's right or if it should go in do_modrdn() instead. (I'm thinking of DN rewriting in rwm, vs. global overlays.) In test005-modrdn, the "modrdn with newSuperior as child of target" check would not catch anything: It expects failure, but the newSuperior did not exist and would fail with noSuchObject in any case. This patch uses newSuperior=target instead, and expects unwillingToPerform. I'll apply the patch later, unless someone thinks it should be done differently.

1 0

Re: (ITS#5330) back-null upgrade
by h.b.furuseth＠usit.uio.no 30 Jan '08

30 Jan '08

hyc(a)symas.com writes: > h.b.furuseth(a)usit.uio.no wrote: >> I'll be upgrading back-null a bit. At the moment just support for >> Back-config, Modify-Increment Extension and empty rootpw. > >> Maybe grow it to a useful template of a nontrivial backend later >> (after I learn how to write that), with access control, referrals, >> various extensions... Might be almost-unused and thus subject to >> code rot, but less so than documentation of the backend API. >> (...) > > I don't believe back-null should ever handle access controls or > referrals. OK. However it shouldn't need to _handle_ referrals, since it can have no referral objects. I think the complete implementation would be to set SLAP_BFLAG_REFERRALS and LDAP_CONTROL_MANAGEDSAIT in the init function. Assuming the default_referral code in BDB which neither of us understand (ITS#5339) is unnecessary. SLAP_BFLAG_INCREMENT and (I think) LDAP_CONTROL_X_PERMISSIVE_MODIFY can be set without needing any extra code, and LDAP_CONTROL_NOOP will need one line for the update functions. > It shouldn't have any config options at all, really. Does that mean I shouldn't add back-config support either? I do find "bind on" useful. And it would be convenient to accept and ignore the "directory" option, so it can be used as a drop-in replacement for a minimal back-bdb/back-ldif config. My current motivation is nothing major, just to push back-null through "cd tests/ && ./run -b null all". > Create some other backend if you want a dummy template that shows how > all the other APIs are tied in. OK. Just need to think of something for it to do... -- Hallvard

1 0

(ITS#5343) cn=config crash when deleting attrs
by h.b.furuseth＠usit.uio.no 30 Jan '08

30 Jan '08

Full_Name: Hallvard B Furuseth Version: HEAD OS: Linux URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (129.240.203.232) Submitted by: hallvard back-config crashes if I delete some attribute, even some which does not exist, and then olcAttributeOptions: bash$ cat slapd.conf include servers/slapd/schema/core.schema database config rootdn cn=config rootpw secret bash$ ldapmodify -xh:3890 -Dcn=config -wsecret dn: cn=config changetype: modify # or use "delete: o" delete: olcAllows - delete: olcAttributeOptions olcAttributeOptions: foo - Crash in bconfig.c:config_modify_internal line 4843 because d == NULL: 4842 if ( ml->sml_values ) { 4843 d = d->next; 4844 ch_free( dels ); 4845 dels = d; 4846 } Backtrace: #0 0x000000000042db19 in config_modify_internal (ce=0x9cb710, op=0x9d7330, rs=0x41801cb0, ca=0x417ff750) at bconfig.c:4843 #1 0x000000000042e34a in config_back_modify (op=0x9d7330, rs=0x41801cb0) at bconfig.c:4991 #2 0x0000000000461107 in fe_op_modify (op=0x9d7330, rs=0x41801cb0) at modify.c:300 #3 0x00000000004609b0 in do_modify (op=0x9d7330, rs=0x41801cb0) at modify.c:175 #4 0x0000000000440dd8 in connection_operation (ctx=0x41801e00, arg_v=0x9d7330) at connection.c:1087 #5 0x0000000000441306 in connection_read_thread (ctx=0x41801e00, argv=0x9) at connection.c:1214 #6 0x00002b100d4c354f in ldap_int_thread_pool_wrapper (xpool=0x927a10) at tpool.c:625 #7 0x00002b100e1e6317 in start_thread () from /lib/libpthread.so.0 #8 0x00002b100f935d5d in clone () from /lib/libc.so.6 #9 0x0000000000000000 in ?? () Also the error message from deleting just olcAttributeOptions is bad: bash$ ldapmodify -xh:3890 -Dcn=config -wsecret dn: cn=config changetype: modify replace: olcAttributeOptions olcAttributeOptions: foo ldap_modify: Internal (implementation specific) error (80) additional info: modify/delete: modifyTimestamp: no such attribute modifying entry "cn=config" Finally "replace:" only needs to fail if there are options to delete, not if one adds a line with just "attributeOptions" to slapd.conf. That removes the default, "lang-".

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

openldap-bugs