- openldap-bugs - openldap.org

Re: (ITS#6045) Use copy of db in syncprov_matchops
by hyc＠symas.com 03 Apr '09

03 Apr '09

rein(a)OpenLDAP.org wrote: > Full_Name: Rein Tollevik > Version: CVS HEAD/RE_24 > OS: Irrelevant > URL: > Submission from: (NULL) (81.93.160.250) > > > I had a couple of seg. faults when resync'ing my servers after upgrading to the > upcoming 2.4.16 release. Looks as if a copy of the backend must be used when > testing the filter in syncprov_matchops. See the gdb output at the end. Note, > some function names are incorrect due to optimization. A fix is coming. The fix makes no sense, or the problem has not yet been analyzed sufficiently. Nobody in that call chain should be zeroing out bd_info. And if someone *is*, then it will happen in *whatever* BackendDB structure is currently being used. Explain the real cause of the problem, and why the fix is correct. -- -- Howard Chu CTO, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

(ITS#6045) Use copy of db in syncprov_matchops
by rein＠OpenLDAP.org 03 Apr '09

03 Apr '09

Full_Name: Rein Tollevik Version: CVS HEAD/RE_24 OS: Irrelevant URL: Submission from: (NULL) (81.93.160.250) I had a couple of seg. faults when resync'ing my servers after upgrading to the upcoming 2.4.16 release. Looks as if a copy of the backend must be used when testing the filter in syncprov_matchops. See the gdb output at the end. Note, some function names are incorrect due to optimization. A fix is coming. Rein Tollevik Basefarm AS Program terminated with signal 11, Segmentation fault. #0 0x080989a1 in access_allowed_mask (op=0x8c98dc54, e=0x83f827c, desc=0x81f2008, val=0x0, access=ACL_SEARCH, state=0x0, maskp=0x0) at acl.c:454 454 if ( op->o_bd->bd_info->bi_access_allowed ) { (gdb) where #0 0x080989a1 in access_allowed_mask (op=0x8c98dc54, e=0x83f827c, desc=0x81f2008, val=0x0, access=ACL_SEARCH, state=0x0, maskp=0x0) at acl.c:454 #1 0x0809603b in test_filter (op=0x8c98dc54, e=0x83f827c, f=0x82ffb0ac) at filterentry.c:821 #2 0xb7a3d740 in syncprov_matchops (op=0x8c98edc8, opc=0x7ae2d620, saveit=0) at syncprov.c:1257 #3 0xb7a3e5cd in syncprov_op_response (op=0x8c98edc8, rs=0x8c98e890) at syncprov.c:1801 #4 0x08084e5d in slap_response_play (op=0x8c98edc8, rs=0x8c98e890) at result.c:350 #5 0x0808821c in send_ldap_response (op=0x8c98edc8, rs=0x8c98e890) at result.c:424 #6 0x080893b3 in slap_send_ldap_result (op=0x8c98edc8, rs=0x8c98e890) at result.c:693 #7 0x08109928 in bdb_add (op=0x8c98edc8, rs=0x8c98e890) at add.c:494 #8 0x080e13cf in overlay_op_walk (op=0x8c98edc8, rs=0x8c98e890, which=op_add, oi=0x829a000, on=0x829b2b0) at backover.c:669 #9 0x080e19b5 in over_op_func (op=0x8c98edc8, rs=0x8c98e890, which=op_add) at backover.c:721 #10 0x080e1a95 in over_op_add (op=0x8c98edc8, rs=0x8c98e890) at backover.c:767 #11 0x080df61f in glue_op_func (op=0x8c98edc8, rs=0x8c98e890) at backglue.c:238 #12 0x080e1379 in overlay_op_walk (op=0x8c98edc8, rs=0x8c98e890, which=op_add, oi=0x82a3098, on=0x82a3298) at backover.c:659 #13 0x080e19b5 in over_op_func (op=0x8c98edc8, rs=0x8c98e890, which=op_add) at backover.c:721 #14 0x080e1a95 in over_op_add (op=0x8c98edc8, rs=0x8c98e890) at backover.c:767 #15 0x080d962f in syncrepl_entry (si=0x829b140, op=0x8c98edc8, entry=0x83f827c, modlist=0x8c98ebf8, syncstate=1, syncUUID=0x8c98ec50, syncCSN=0x0) at syncrepl.c:2187 #16 0x080dca39 in do_syncrep2 (op=0x8c98edc8, si=0x829b140) at syncrepl.c:892 #17 0x080de907 in do_syncrepl (ctx=0x8c98f2b8, arg=0x829a760) at syncrepl.c:1361 #18 0x08075d59 in connection_read_thread (ctx=0x8c98f2b8, argv=0x4d) at connection.c:1225 #19 0xb7fb3a20 in ldap_int_thread_pool_wrapper (xpool=0x81f8cf0) at tpool.c:663 #20 0xb7c53371 in start_thread () from /lib/tls/libpthread.so.0 #21 0xb7beaffe in clone () from /lib/tls/libc.so.6 (gdb) print op->o_bd->bd_info $1 = (BackendInfo *) 0x0 (gdb) print op->o_bd $2 = (BackendDB *) 0x8a4c4814 (gdb) frame 2 #2 0xb7a3d740 in syncprov_matchops (op=0x8c98edc8, opc=0x7ae2d620, saveit=0) at syncprov.c:1257 1257 rc = test_filter( &op2, e, ss->s_op->ors_filter ); (gdb) print op2.o_bd $3 = (BackendDB *) 0x8a4c4814 (gdb) print *op2.o_bd $4 = {bd_info = 0x0, bd_self = 0x0, be_ctrls = "\000\000\000\000\214\221\030\b�\001�\204\210HL\212\201�\f\b ", '\0' <repeats 11 times>, be_flags = 0, be_restrictops = 0, be_requires = 0, be_ssf_set = {sss_ssf = 0, sss_transport = 0, sss_tls = 0, sss_sasl = 1, sss_update_ssf = 3086609409, sss_update_transport = 2142916662, sss_update_tls = 2320255160, sss_update_sasl = 1, sss_simple_bind = 0}, be_suffix = 0x0, be_nsuffix = 0x1, be_schemadn = {bv_len = 32, bv_val = 0xb7fa418c "\204�"}, be_schemandn = {bv_len = 0, bv_val = 0x8a4c4a0c "\002"}, be_rootdn = {bv_len = 2320255176, bv_val = 0xb7f9c44b "\203�,[^_]�U\211�\203�8\211]�\211u�\211}��w���\201�%}"}, be_rootndn = {bv_len = 2320255500, bv_val = 0x8a4c48b8 "\004\001�\204\214A��\fJL\212\fJL\212�HL\212?���\001"}, be_rootpw = {bv_len = 1, bv_val = 0x0}, be_max_deref_depth = 2229404096, be_def_limit = {lms_t_soft = 1, lms_t_hard = 0, lms_s_soft = -1208355505, lms_s_hard = 1, lms_s_unchecked = 24, lms_s_pr = -2065563388, lms_s_pr_hide = -1208335988, lms_s_pr_total = -1974711796}, be_limits = 0x8a4c4a0c, be_acl = 0x8a4c48f8, be_dfltaccess = -1208367041, be_update_ndn = {bv_len = 1, bv_val = 0x18 ""}, be_update_refs = 0x84e201c0, be_pending_csn_list = 0x818918c, be_pcl_mutex = {__m_reserved = 0, __m_count = 48, __m_owner = 0x7fba5030, __m_kind = -1208335988, __m_lock = {__status = -1, __spinlock = -1974711796}}, be_syncinfo = 0x8a4c4918, be_pb = 0xb7f9c961, be_cf_ocs = 0x8a4c4a0c, be_private = 0x81f24f8, be_next = {stqe_next = 0xf}}

1 0

Re: (ITS#6021) slapo-pcache/back-ldap slapd crashes
by toby＠inf.ed.ac.uk 03 Apr '09

03 Apr '09

Hi there, I've been collecting a bit more information on these crashes... Since we started running 2.4.15: 26 core files, of these, 15 have crashed in memory.c:152 It looks like the majority of these were removing stale entries from the cache when slapd crashed, which wasn't the case with the original ticket I submitted. Also, the last thing logged by slapd in quite a few of these cases was along this lines of ... Mar 30 20:50:16 albany slapd[14239]: DELETING ENTRY TEMPLATE=db78c810-0c2a-49de-96af-f5796bbe8ca3 Mar 30 20:50:17 albany last message repeated 14 times ... I don't know whether this is useful? Here's the details of the latest crash, slapd log and backtrace: Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a5140 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a5140 QUERIES-- 0 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a5140 Mar 30 20:50:16 albany slapd[14239]: DELETING ENTRY TEMPLATE=e7ccb20c-d263-4d6c-ba23-62afe55f0158 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=1 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 30 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 15 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 29 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 14 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 28 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 13 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 27 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 12 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 26 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 11 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 25 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 10 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 24 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 9 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 23 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 8 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 22 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 7 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 21 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 6 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 20 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 5 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 19 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 4 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 18 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 3 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 17 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 2 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 16 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 1 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 15 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e90 QUERIES-- 0 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e90 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, SIZE=0 Mar 30 20:50:16 albany slapd[14239]: STORED QUERIES = 14 Mar 30 20:50:16 albany slapd[14239]: STALE QUERY REMOVED, CACHE =22 entries Mar 30 20:50:16 albany slapd[14239]: Lock CR index = 0x83a4e28 Mar 30 20:50:16 albany slapd[14239]: TEMPLATE 0x83a4e28 QUERIES-- 0 Mar 30 20:50:16 albany slapd[14239]: Unlock CR index = 0x83a4e28 Mar 30 20:50:16 albany slapd[14239]: DELETING ENTRY TEMPLATE=db78c810-0c2a-49de-96af-f5796bbe8ca3 Mar 30 20:50:17 albany last message repeated 14 times Program terminated with signal 6, Aborted. #0 0x00185402 in __kernel_vsyscall () (gdb) bt #0 0x00185402 in __kernel_vsyscall () #1 0x00476d20 in raise () from /lib/libc.so.6 #2 0x00478631 in abort () from /lib/libc.so.6 #3 0x004aee6b in __libc_message () from /lib/libc.so.6 #4 0x004b6b16 in _int_free () from /lib/libc.so.6 #5 0x004ba070 in free () from /lib/libc.so.6 #6 0x081d5746 in ber_memfree_x (p=0x841e688, ctx=0x0) at memory.c:152 #7 0x081d630c in ber_bvarray_free_x (a=0x843af58, ctx=0x0) at memory.c:731 #8 0x081d6343 in ber_bvarray_free (a=0x843af58) at memory.c:741 #9 0x0807979c in attr_clean (a=0xb62934ec) at attr.c:146 #10 0x0807983b in attrs_free (a=0xb62934ec) at attr.c:196 #11 0x0807c059 in entry_clean (e=0xb6727584) at entry.c:504 #12 0x0807c080 in entry_free (e=0xb6727584) at entry.c:514 #13 0x08140030 in bdb_entry_return (e=0xb6727584) at id2entry.c:229 #14 0x08134c0c in bdb_cache_delete_cleanup (cache=0x838956c, ei=0xb332a060) at cache.c:1316 #15 0x0813a824 in bdb_delete (op=0xb32fed5c, rs=0xb32febec) at delete.c:575 #16 0x08177be1 in remove_query_data (op=0xb32fed5c, rs=0xb32fecf8, query_uuid=0x83ff288) at pcache.c:1460 #17 0x0817af8f in consistency_check (ctx=0xb32ff1d0, arg=0x83f57e8) at pcache.c:2611 #18 0x0819ecad in ldap_int_thread_pool_wrapper (xpool=0x8366f90) at tpool.c:663 #19 0x005c746b in start_thread () from /lib/libpthread.so.0 #20 0x0051edbe in clone () from /lib/libc.so.6 (gdb) Also, I'll include details of another type of crash that I've seen 3 times since running 2.4.15 in the lab, at the risk of overloading information into this ticket. I'm including it here as the slapd behaviour at time of crash was similar to that noted above, i.e. removing stale entries from the cache: Mar 13 04:37:48 pemberton slapd[6767]: Lock CR index = 0x9ac9188 Mar 13 04:37:48 pemberton slapd[6767]: TEMPLATE 0x9ac9188 QUERIES-- 2 Mar 13 04:37:48 pemberton slapd[6767]: Unlock CR index = 0x9ac9188 Mar 13 04:37:48 pemberton slapd[6767]: DELETING ENTRY TEMPLATE=febcaede-1a2f-42ef-af2e-9206f0a4f678 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, SIZE=1 Mar 13 04:37:48 pemberton slapd[6767]: STORED QUERIES = 32 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, CACHE =19 entries Mar 13 04:37:48 pemberton slapd[6767]: Lock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: TEMPLATE 0x9ac8eb0 QUERIES-- 15 Mar 13 04:37:48 pemberton slapd[6767]: Unlock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, SIZE=0 Mar 13 04:37:48 pemberton slapd[6767]: STORED QUERIES = 31 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, CACHE =19 entries Mar 13 04:37:48 pemberton slapd[6767]: Lock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: TEMPLATE 0x9ac8eb0 QUERIES-- 14 Mar 13 04:37:48 pemberton slapd[6767]: Unlock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, SIZE=0 Mar 13 04:37:48 pemberton slapd[6767]: STORED QUERIES = 30 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, CACHE =19 entries Mar 13 04:37:48 pemberton slapd[6767]: Lock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: TEMPLATE 0x9ac8eb0 QUERIES-- 13 Mar 13 04:37:48 pemberton slapd[6767]: Unlock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, SIZE=0 Mar 13 04:37:48 pemberton slapd[6767]: STORED QUERIES = 29 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, CACHE =19 entries Mar 13 04:37:48 pemberton slapd[6767]: Lock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: TEMPLATE 0x9ac8eb0 QUERIES-- 12 Mar 13 04:37:48 pemberton slapd[6767]: Unlock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, SIZE=0 Mar 13 04:37:48 pemberton slapd[6767]: STORED QUERIES = 28 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, CACHE =19 entries Mar 13 04:37:48 pemberton slapd[6767]: Lock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: TEMPLATE 0x9ac8eb0 QUERIES-- 11 Mar 13 04:37:48 pemberton slapd[6767]: Unlock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, SIZE=0 Mar 13 04:37:48 pemberton slapd[6767]: STORED QUERIES = 27 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, CACHE =19 entries Mar 13 04:37:48 pemberton slapd[6767]: Lock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: TEMPLATE 0x9ac8eb0 QUERIES-- 10 Mar 13 04:37:48 pemberton slapd[6767]: Unlock CR index = 0x9ac8eb0 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, SIZE=0 Mar 13 04:37:48 pemberton slapd[6767]: STORED QUERIES = 26 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, CACHE =19 entries Mar 13 04:37:48 pemberton slapd[6767]: Lock CR index = 0x9ac8e48 Mar 13 04:37:48 pemberton slapd[6767]: TEMPLATE 0x9ac8e48 QUERIES-- 1 Mar 13 04:37:48 pemberton slapd[6767]: Unlock CR index = 0x9ac8e48 Mar 13 04:37:48 pemberton slapd[6767]: DELETING ENTRY TEMPLATE=3eff8b71-4ca6-4918-b760-c32023dbe215 Mar 13 04:37:48 pemberton last message repeated 12 times Mar 13 04:37:48 pemberton slapd[6767]: REMOVING TEMP ATTR : TEMPLATE=3eff8b71-4ca6-4918-b760-c32023dbe215 Mar 13 04:37:48 pemberton last message repeated 2 times Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, SIZE=13 Mar 13 04:37:48 pemberton slapd[6767]: STORED QUERIES = 25 Mar 13 04:37:48 pemberton slapd[6767]: STALE QUERY REMOVED, CACHE =6 entries Mar 13 04:37:48 pemberton slapd[6767]: Lock CR index = 0x9ac8da8 Program terminated with signal 11, Segmentation fault. #0 0x081772d2 in remove_from_template (qc=0xaef209a8, template=0x9ac8da8) at pcache.c:1297 1297 qc->prev->next = qc->next; (gdb) bt #0 0x081772d2 in remove_from_template (qc=0xaef209a8, template=0x9ac8da8) at pcache.c:1297 #1 0x0817adc8 in consistency_check (ctx=0xaeeff1d0, arg=0x9b19860) at pcache.c:2594 #2 0x0819ecad in ldap_int_thread_pool_wrapper (xpool=0x9a8afa0) at tpool.c:663 #3 0x00bb046b in start_thread () from /lib/libpthread.so.0 #4 0x00b07dbe in clone () from /lib/libc.so.6 (gdb) (gdb) p qc->prev->next Cannot access memory at address 0x54 (gdb) p qc->prev $2 = (struct cached_query_s *) 0x30 (gdb) p *qc->prev Cannot access memory at address 0x30 (gdb) As always, if there's any more information that would be useful, let me know... Cheers Toby -- The University of Edinburgh is a charitable body, registered in Scotland, with registration number SC005336.

1 0

Re: (ITS#6043) slapd segfaults in bdb_rdn_cmp
by hyc＠symas.com 03 Apr '09

03 Apr '09

Luca Scamoni wrote: > (gdb) thread apply all bt full > > Thread 4 (process 29343): > #0 0x006487a2 in _dl_sysinfo_int80 () from /lib/ld-linux.so.2 > No symbol table info available. > #1 0x001271d7 in pthread_join () from /lib/tls/libpthread.so.0 > No symbol table info available. > #2 0x008f8926 in ldap_pvt_thread_join (thread=4294967292, > thread_return=0xfffffffc) > at ../../../libraries/libldap_r/thr_posix.c:197 > No locals. > #3 0x080704d7 in slapd_daemon () > No symbol table info available. > #4 0x0805858e in main () > No symbol table info available. > > Thread 3 (process 29344): > #0 0x006487a2 in _dl_sysinfo_int80 () from /lib/ld-linux.so.2 > No symbol table info available. > #1 0x007372be in epoll_wait () from /lib/tls/libc.so.6 > No symbol table info available. > #2 0x0806f3fb in slapd_daemon_destroy () This trace is bogus, there's no call from slapd_daemon_destroy() to epoll_wait() in the code. And you wouldn't have gotten to slapd_daemon_destroy() unless you were shutting down. > No symbol table info available. > #3 0x001263cc in start_thread () from /lib/tls/libpthread.so.0 > No symbol table info available. > #4 0x00736c3e in clone () from /lib/tls/libc.so.6 > No symbol table info available. > > Thread 2 (process 29345): > #0 0x006487a2 in _dl_sysinfo_int80 () from /lib/ld-linux.so.2 > No symbol table info available. > #1 0x00128c26 in pthread_cond_wait@@GLIBC_2.3.2 () from > /lib/tls/libpthread.so.0 > No symbol table info available. > #2 0x008f8a2b in ldap_pvt_thread_cond_wait (cond=0xfffffffc, > mutex=0xfffffffc) > at ../../../libraries/libldap_r/thr_posix.c:277 > No locals. > #3 0x008f7d5a in ldap_int_thread_pool_wrapper (xpool=0x96eeb98) > at ../../../libraries/libldap_r/tpool.c:654 > task = (ldap_int_thread_task_t *) 0x0 > work_list = Variable "work_list" is not available. > > Thread 1 (process 29346): > #0 0x006d7860 in strncmp () from /lib/tls/libc.so.6 > No symbol table info available. > #1 0x0019085e in bdb_rdn_cmp (v_e1=0x17d7270, v_e2=0x9837d58) at > cache.c:301 > rc = 5 > #2 0x080fc1f5 in avl_find () > No symbol table info available. > #3 0x00190abd in hdb_cache_find_ndn (op=0x17d7460, txn=0x9837aa8, > ndn=0x17d73ec, res=0x17d7318) > at cache.c:433 > bdb = (struct bdb_info *) 0x97bf5d8 > ei = {bei_parent = 0x97bf624, bei_id = 0, bei_lockpad = 0, > bei_state = 0, > bei_finders = 0, bei_nrdn = {bv_len = 23, > bv_val = 0xffffffe9<Address 0xffffffe9 out of bounds>}, bei_rdn = > {bv_len = 0, > bv_val = 0x0}, bei_modrdns = 0, bei_ckids = 0, bei_dkids = 0, bei_e > = 0x0, > bei_kids = 0xb7f43994, bei_kids_mutex = {__m_reserved = 0, __m_count = > 0, __m_owner = 0x0, > __m_kind = 0, __m_lock = {__status = 0, __spinlock = 135208867}}, > bei_lrunext = 0x55313c, > bei_lruprev = 0x542022fc} > eip = (EntryInfo *) 0x97bf624 > ei2 = (EntryInfo *) 0x0 > rc = 0 > ptr = Variable "ptr" is not available. This is also bogus; is this an optimized compile? It's always hard to tell how much to trust the debugger here; if there are no further stack frames then possibly the stack was overwritten. But if you compiled with any type of optimization, and didn't use -fno-omit-frame-pointer, then that may be all that's happening here. -- -- Howard Chu CTO, 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#6043) slapd segfaults in bdb_rdn_cmp
by luca.scamoni＠sys-net.it 02 Apr '09

02 Apr '09

Howard Chu ha scritto: > Do you still have the data that caused this crash? Will it continue to > re-occur each time you restart a down server? (Can you continue to > reproduce the event...) I still have the data but the behaviour doesn't seem to be reproducible. It happened only on first run. After that we had to switch back to the old instance (it wasn't a real test, more an attempt to push it into production). I started slapd listening on another port after that and it didn't crash anymore (but it's also not receiving modifications). > > Please print *op and *eip as well. Also, what's in frame 4 or higher, > what function called hdb_cache_find_ndn(), does the stack trace stop > there or is there more? > (gdb) thread apply all bt full Thread 4 (process 29343): #0 0x006487a2 in _dl_sysinfo_int80 () from /lib/ld-linux.so.2 No symbol table info available. #1 0x001271d7 in pthread_join () from /lib/tls/libpthread.so.0 No symbol table info available. #2 0x008f8926 in ldap_pvt_thread_join (thread=4294967292, thread_return=0xfffffffc) at ../../../libraries/libldap_r/thr_posix.c:197 No locals. #3 0x080704d7 in slapd_daemon () No symbol table info available. #4 0x0805858e in main () No symbol table info available. Thread 3 (process 29344): #0 0x006487a2 in _dl_sysinfo_int80 () from /lib/ld-linux.so.2 No symbol table info available. #1 0x007372be in epoll_wait () from /lib/tls/libc.so.6 No symbol table info available. #2 0x0806f3fb in slapd_daemon_destroy () No symbol table info available. #3 0x001263cc in start_thread () from /lib/tls/libpthread.so.0 No symbol table info available. #4 0x00736c3e in clone () from /lib/tls/libc.so.6 No symbol table info available. Thread 2 (process 29345): #0 0x006487a2 in _dl_sysinfo_int80 () from /lib/ld-linux.so.2 No symbol table info available. #1 0x00128c26 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/libpthread.so.0 No symbol table info available. #2 0x008f8a2b in ldap_pvt_thread_cond_wait (cond=0xfffffffc, mutex=0xfffffffc) at ../../../libraries/libldap_r/thr_posix.c:277 No locals. #3 0x008f7d5a in ldap_int_thread_pool_wrapper (xpool=0x96eeb98) at ../../../libraries/libldap_r/tpool.c:654 task = (ldap_int_thread_task_t *) 0x0 work_list = Variable "work_list" is not available. Thread 1 (process 29346): #0 0x006d7860 in strncmp () from /lib/tls/libc.so.6 No symbol table info available. #1 0x0019085e in bdb_rdn_cmp (v_e1=0x17d7270, v_e2=0x9837d58) at cache.c:301 rc = 5 #2 0x080fc1f5 in avl_find () No symbol table info available. #3 0x00190abd in hdb_cache_find_ndn (op=0x17d7460, txn=0x9837aa8, ndn=0x17d73ec, res=0x17d7318) at cache.c:433 bdb = (struct bdb_info *) 0x97bf5d8 ei = {bei_parent = 0x97bf624, bei_id = 0, bei_lockpad = 0, bei_state = 0, bei_finders = 0, bei_nrdn = {bv_len = 23, bv_val = 0xffffffe9 <Address 0xffffffe9 out of bounds>}, bei_rdn = {bv_len = 0, bv_val = 0x0}, bei_modrdns = 0, bei_ckids = 0, bei_dkids = 0, bei_e = 0x0, bei_kids = 0xb7f43994, bei_kids_mutex = {__m_reserved = 0, __m_count = 0, __m_owner = 0x0, __m_kind = 0, __m_lock = {__status = 0, __spinlock = 135208867}}, bei_lrunext = 0x55313c, bei_lruprev = 0x542022fc} eip = (EntryInfo *) 0x97bf624 ei2 = (EntryInfo *) 0x0 rc = 0 ptr = Variable "ptr" is not available. Nothing else... variables contents follow in a private mail... Ing. Luca Scamoni Responsabile Ricerca e Sviluppo SysNet s.r.l. via Dossi, 8 - 27100 Pavia - ITALIA http://www.sys-net.it ----------------------------------- Office: +39 0382 573859 (137) Mobile: +39 347 1014425 Fax: +39 0382 476497 Email: luca.scamoni(a)sys-net.it -----------------------------------

1 0

(ITS#6044) AnonCVS not working
by evaristojosec＠yahoo.es 02 Apr '09

02 Apr '09

Full_Name: Evaristo Camarero Version: Not relevant OS: Fedora Core 10 URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (194.237.142.7) I am trying to access to the OpenLDAP repository but it is not possible doing it cvs -d :pserver:anonymous@cvs.OpenLDAP.org:/repo/OpenLDAP login Logging in to :pserver:anonymous@cvs.openldap.org:2401/repo/OpenLDAP CVS password: cvs [login aborted]: end of file from server (consult above messages if any) Checking with wireshark (old ethereal), I have checked that my cvs client is able to send the authentication request, but the openLDAP cvs server request the end of the TCP connection BEGIN VERIFICATION REQUEST /repo/OpenLDAP anonymous A#:d=*.9} END VERIFICATION REQUEST I have also checked that the same client is working fine with other CVS servers and I had no problem at all cvs -d :pserver:anonymous@cvs.fedoraproject.org:/cvs/pkgs login Logging in to :pserver:anonymous@cvs.fedoraproject.org:2401/cvs/pkgs CVS password:

1 0

Re: (ITS#6043) slapd segfaults in bdb_rdn_cmp
by hyc＠symas.com 02 Apr '09

02 Apr '09

luca(a)OpenLDAP.org wrote: > Full_Name: Luca Scamoni > Version: RELENG > OS: Linux > URL: ftp://ftp.openldap.org/incoming/ > Submission from: (NULL) (82.63.140.131) > > > Testing releng got this segfaults in a master-slave setup on both machines. Both > traces show the same: > > (gdb) bt full > #0 0x006d7860 in strncmp () from /lib/tls/libc.so.6 > No symbol table info available. > #1 0x0019085e in bdb_rdn_cmp (v_e1=0x17d7270, v_e2=0x9837d58) at cache.c:301 > rc = 5 > #2 0x080fc1f5 in avl_find () > No symbol table info available. > #3 0x00190abd in hdb_cache_find_ndn (op=0x17d7460, txn=0x9837aa8, > ndn=0x17d73ec, res=0x17d7318) at cache.c:433 > bdb = (struct bdb_info *) 0x97bf5d8 > ei = {bei_parent = 0x97bf624, bei_id = 0, bei_lockpad = 0, bei_state = > 0, bei_finders = 0, bei_nrdn = {bv_len = 23, > bv_val = 0xffffffe9<Address 0xffffffe9 out of bounds>}, bei_rdn = {bv_len = > 0, bv_val = 0x0}, bei_modrdns = 0, bei_ckids = 0, bei_dkids = 0, > bei_e = 0x0, bei_kids = 0xb7f43994, bei_kids_mutex = {__m_reserved = 0, > __m_count = 0, __m_owner = 0x0, __m_kind = 0, __m_lock = {__status = 0, > __spinlock = 135208867}}, bei_lrunext = 0x55313c, bei_lruprev = > 0x542022fc} > eip = (EntryInfo *) 0x97bf624 > ei2 = (EntryInfo *) 0x0 > rc = 0 > ptr = Variable "ptr" is not available. > > The core happened on master shortly after starting the slave, then on slave > shortly after staring the master again. > Using HDB and syncrepl (two databases per instance, different suffixes) > > I have access to the core(s) for any further information needed > > Do you still have the data that caused this crash? Will it continue to re-occur each time you restart a down server? (Can you continue to reproduce the event...) Please print *op and *eip as well. Also, what's in frame 4 or higher, what function called hdb_cache_find_ndn(), does the stack trace stop there or is there more? -- -- Howard Chu CTO, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

(ITS#6043) slapd segfaults in bdb_rdn_cmp
by luca＠OpenLDAP.org 02 Apr '09

02 Apr '09

Full_Name: Luca Scamoni Version: RELENG OS: Linux URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (82.63.140.131) Testing releng got this segfaults in a master-slave setup on both machines. Both traces show the same: (gdb) bt full #0 0x006d7860 in strncmp () from /lib/tls/libc.so.6 No symbol table info available. #1 0x0019085e in bdb_rdn_cmp (v_e1=0x17d7270, v_e2=0x9837d58) at cache.c:301 rc = 5 #2 0x080fc1f5 in avl_find () No symbol table info available. #3 0x00190abd in hdb_cache_find_ndn (op=0x17d7460, txn=0x9837aa8, ndn=0x17d73ec, res=0x17d7318) at cache.c:433 bdb = (struct bdb_info *) 0x97bf5d8 ei = {bei_parent = 0x97bf624, bei_id = 0, bei_lockpad = 0, bei_state = 0, bei_finders = 0, bei_nrdn = {bv_len = 23, bv_val = 0xffffffe9 <Address 0xffffffe9 out of bounds>}, bei_rdn = {bv_len = 0, bv_val = 0x0}, bei_modrdns = 0, bei_ckids = 0, bei_dkids = 0, bei_e = 0x0, bei_kids = 0xb7f43994, bei_kids_mutex = {__m_reserved = 0, __m_count = 0, __m_owner = 0x0, __m_kind = 0, __m_lock = {__status = 0, __spinlock = 135208867}}, bei_lrunext = 0x55313c, bei_lruprev = 0x542022fc} eip = (EntryInfo *) 0x97bf624 ei2 = (EntryInfo *) 0x0 rc = 0 ptr = Variable "ptr" is not available. The core happened on master shortly after starting the slave, then on slave shortly after staring the master again. Using HDB and syncrepl (two databases per instance, different suffixes) I have access to the core(s) for any further information needed

1 0

Access to pseudoattribute "entry" and rewrite overlay
by Konovalov Andrey 01 Apr '09

01 Apr '09

Hi4All! :) I notice that active RWM/Remap overlay affects ACL-subsystem when ACL checks access to pseudoatribute "entry" and this strange situation occurs even if i not use any rules for rewrite/remap. Нerewith without the loaded overlay RWM all works correctly... In debug mode slapd with active RWM (no rewrite rules!) deny all access to attribute entry except for "root" user => access_allowed: search access to "uid=akkerman,cn=Directory Server Admins,ou=Groups,dc=r2,dc=money,dc=ge,dc=com" "objectClass" requested <= test_filter 5 => acl_get: [13] attr entry => slap_access_allowed: result not in cache (entry) => acl_mask: access to entry "uid=akkerman,cn=Directory Server Admins,ou=Groups,dc=r2,dc=money,dc=ge,dc=com", attr "entry" requested => acl_mask: to all values by "", (none(=0)) <= check a_dn_pat: * <= acl_mask: [1] applying none(=0) (stop) <= acl_mask: [1] mask: none(=0) => slap_access_allowed: read access denied by none(=0) This problem may be solved by adding radically liberate rule to the beginning of olcAccess sequence in cn=config: olcAccess: {1}to * attrs=entry by * read Is it a bug?

2 1

Re: (ITS#6042) Microsoft compatibility patch
by ghenry＠suretecsystems.com 01 Apr '09

01 Apr '09

----- hyc(a)OpenLDAP.org wrote: > Full_Name: Howard Chu > Version: HEAD > OS: Linux > URL: ftp://ftp.openldap.org/incoming/hyc-mscompat.txt > Submission from: (NULL) (76.91.220.157) > > > This patch adds a new olcMsCompat config keyword which enables > ActiveDirectory emulation mode in slapd. It's still experimental, any > feedback > is appreciated. Can we fast track this patch in, as we really need this feature to get more breakfix requests. Stock slapd never goes down enough! -- Kind Regards, Gavin Henry. Managing Director. T +44 (0) 1224 279484 M +44 (0) 7930 323266 F +44 (0) 1224 824887 E ghenry(a)suretecsystems.com Open Source. Open Solutions(tm). http://www.suretecsystems.com/ Suretec Systems is a limited company registered in Scotland. Registered number: SC258005. Registered office: 13 Whiteley Well Place, Inverurie, Aberdeenshire, AB51 4FP. Subject to disclaimer at http://www.suretecgroup.com/disclaimer.html

1 0

(ITS#6042) Microsoft compatibility patch
by hyc＠OpenLDAP.org 01 Apr '09

01 Apr '09

Full_Name: Howard Chu Version: HEAD OS: Linux URL: ftp://ftp.openldap.org/incoming/hyc-mscompat.txt Submission from: (NULL) (76.91.220.157) This patch adds a new olcMsCompat config keyword which enables ActiveDirectory emulation mode in slapd. It's still experimental, any feedback is appreciated.

1 0

Re: (ITS#6041) opendir() and closedir() bugs on Windows
by hyc＠symas.com 30 Mar '09

30 Mar '09

test.007(a)seznam.cz wrote: > Full_Name: Test Seven > Version: 2.4.15 > OS: Windows > URL: > Submission from: (NULL) (195.113.184.20) Thanks for the patch. The Windows code is known to work fine using gcc. Use of MSVC is discouraged since there have been so many incompatible CRT version changes in such a short span of time. (Actually, use of Microsoft-anything is generally discouraged; the same OpenLDAP code is at least 2-3x faster on the same hardware when using Linux instead of Windows.) > Two problems solved in the patch: > 1. closedir()'s retval is used, thus it must not be void. Windows uses implicit > declaration ``int closedir(int)'' because the function is not declared in > headers, thus the call in ldif.c compiles, but uses random value in EAX. > > 2. opendir() must set errno, because ldif_readdir() reads it. The patch converts > the most common Windows error codes to errno codes after FindFirstFile() WinAPI > call. > > The bugs prevent me from loading cn=config configuration. > > --- utils.c.orig 2009-01-22 01:00:58.000000000 +0100 > +++ utils.c 2009-03-27 14:28:33.500000000 +0100 > @@ -466,6 +466,40 @@ int mkstemp( char * template ) > #endif > > #ifdef _MSC_VER > +/* Equivalent of MS CRT's _dosmaperr(). > + * @param lastError[in] Result of GetLastError(). > + */ > +static errno_t win2errno(DWORD lastError) > +{ > + const struct { > + DWORD windows_code; > + errno_t errno_code; > + } WIN2ERRNO_TABLE[] = { > + { ERROR_SUCCESS, 0 }, > + { ERROR_FILE_NOT_FOUND, ENOENT }, > + { ERROR_PATH_NOT_FOUND, ENOENT }, > + { ERROR_TOO_MANY_OPEN_FILES, EMFILE }, > + { ERROR_ACCESS_DENIED, EACCES }, > + { ERROR_INVALID_HANDLE, EBADF }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_LOCK_VIOLATION, EACCES }, > + { ERROR_FILE_EXISTS, EEXIST }, > + { ERROR_INVALID_PARAMETER, EINVAL }, > + { ERROR_FILENAME_EXCED_RANGE, ENAMETOOLONG }, > + }; > + const unsigned int WIN2ERRNO_TABLE_SIZE = sizeof(WIN2ERRNO_TABLE) / > sizeof(WIN2ERRNO_TABLE[0]); > + const errno_t DEFAULT_ERRNO_ERROR = -1; > + unsigned int i; > + > + for (i = 0; i< WIN2ERRNO_TABLE_SIZE; ++i) { > + if (WIN2ERRNO_TABLE[i].windows_code == lastError) { > + return WIN2ERRNO_TABLE[i].errno_code; > + } > + } > + return DEFAULT_ERRNO_ERROR; > +} > + > struct dirent { > char *d_name; > }; > @@ -493,9 +527,10 @@ DIR *opendir( char *path ) > > h = FindFirstFile( tmp,&data ); > > - if ( h == INVALID_HANDLE_VALUE ) > + if ( h == INVALID_HANDLE_VALUE ) { > + errno = win2errno(GetLastError()); > return NULL; > - > + } > d = ber_memalloc( sizeof(DIR) ); > if ( !d ) > return NULL; > @@ -518,10 +553,11 @@ struct dirent *readdir(DIR *dir) > } > return&dir->data; > } > -void closedir(DIR *dir) > +int closedir(DIR *dir) > { > FindClose(dir->dir); > ber_memfree(dir); > + return 0; > } > #endif > > > -- -- Howard Chu CTO, 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#6041) opendir() and closedir() bugs on Windows
by hyc＠symas.com 30 Mar '09

30 Mar '09

test.007(a)seznam.cz wrote: > Full_Name: Test Seven > Version: 2.4.15 > OS: Windows > URL: > Submission from: (NULL) (195.113.184.20) Thanks for the patch. The Windows code is known to work fine using gcc. Use of MSVC is discouraged since there have been so many incompatible CRT version changes in such a short span of time. (Actually, use of Microsoft-anything is generally discouraged; the same OpenLDAP code is at least 2-3x faster on the same hardware when using Linux instead of Windows.) > Two problems solved in the patch: > 1. closedir()'s retval is used, thus it must not be void. Windows uses implicit > declaration ``int closedir(int)'' because the function is not declared in > headers, thus the call in ldif.c compiles, but uses random value in EAX. > > 2. opendir() must set errno, because ldif_readdir() reads it. The patch converts > the most common Windows error codes to errno codes after FindFirstFile() WinAPI > call. > > The bugs prevent me from loading cn=config configuration. > > --- utils.c.orig 2009-01-22 01:00:58.000000000 +0100 > +++ utils.c 2009-03-27 14:28:33.500000000 +0100 > @@ -466,6 +466,40 @@ int mkstemp( char * template ) > #endif > > #ifdef _MSC_VER > +/* Equivalent of MS CRT's _dosmaperr(). > + * @param lastError[in] Result of GetLastError(). > + */ > +static errno_t win2errno(DWORD lastError) > +{ > + const struct { > + DWORD windows_code; > + errno_t errno_code; > + } WIN2ERRNO_TABLE[] = { > + { ERROR_SUCCESS, 0 }, > + { ERROR_FILE_NOT_FOUND, ENOENT }, > + { ERROR_PATH_NOT_FOUND, ENOENT }, > + { ERROR_TOO_MANY_OPEN_FILES, EMFILE }, > + { ERROR_ACCESS_DENIED, EACCES }, > + { ERROR_INVALID_HANDLE, EBADF }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_LOCK_VIOLATION, EACCES }, > + { ERROR_FILE_EXISTS, EEXIST }, > + { ERROR_INVALID_PARAMETER, EINVAL }, > + { ERROR_FILENAME_EXCED_RANGE, ENAMETOOLONG }, > + }; > + const unsigned int WIN2ERRNO_TABLE_SIZE = sizeof(WIN2ERRNO_TABLE) / > sizeof(WIN2ERRNO_TABLE[0]); > + const errno_t DEFAULT_ERRNO_ERROR = -1; > + unsigned int i; > + > + for (i = 0; i< WIN2ERRNO_TABLE_SIZE; ++i) { > + if (WIN2ERRNO_TABLE[i].windows_code == lastError) { > + return WIN2ERRNO_TABLE[i].errno_code; > + } > + } > + return DEFAULT_ERRNO_ERROR; > +} > + > struct dirent { > char *d_name; > }; > @@ -493,9 +527,10 @@ DIR *opendir( char *path ) > > h = FindFirstFile( tmp,&data ); > > - if ( h == INVALID_HANDLE_VALUE ) > + if ( h == INVALID_HANDLE_VALUE ) { > + errno = win2errno(GetLastError()); > return NULL; > - > + } > d = ber_memalloc( sizeof(DIR) ); > if ( !d ) > return NULL; > @@ -518,10 +553,11 @@ struct dirent *readdir(DIR *dir) > } > return&dir->data; > } > -void closedir(DIR *dir) > +int closedir(DIR *dir) > { > FindClose(dir->dir); > ber_memfree(dir); > + return 0; > } > #endif > > > -- -- Howard Chu CTO, 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#6041) opendir() and closedir() bugs on Windows
by hyc＠symas.com 30 Mar '09

30 Mar '09

test.007(a)seznam.cz wrote: > Full_Name: Test Seven > Version: 2.4.15 > OS: Windows > URL: > Submission from: (NULL) (195.113.184.20) Thanks for the patch. The Windows code is known to work fine using gcc. Use of MSVC is discouraged since there have been so many incompatible CRT version changes in such a short span of time. (Actually, use of Microsoft-anything is generally discouraged; the same OpenLDAP code is at least 2-3x faster on the same hardware when using Linux instead of Windows.) > Two problems solved in the patch: > 1. closedir()'s retval is used, thus it must not be void. Windows uses implicit > declaration ``int closedir(int)'' because the function is not declared in > headers, thus the call in ldif.c compiles, but uses random value in EAX. > > 2. opendir() must set errno, because ldif_readdir() reads it. The patch converts > the most common Windows error codes to errno codes after FindFirstFile() WinAPI > call. > > The bugs prevent me from loading cn=config configuration. > > --- utils.c.orig 2009-01-22 01:00:58.000000000 +0100 > +++ utils.c 2009-03-27 14:28:33.500000000 +0100 > @@ -466,6 +466,40 @@ int mkstemp( char * template ) > #endif > > #ifdef _MSC_VER > +/* Equivalent of MS CRT's _dosmaperr(). > + * @param lastError[in] Result of GetLastError(). > + */ > +static errno_t win2errno(DWORD lastError) > +{ > + const struct { > + DWORD windows_code; > + errno_t errno_code; > + } WIN2ERRNO_TABLE[] = { > + { ERROR_SUCCESS, 0 }, > + { ERROR_FILE_NOT_FOUND, ENOENT }, > + { ERROR_PATH_NOT_FOUND, ENOENT }, > + { ERROR_TOO_MANY_OPEN_FILES, EMFILE }, > + { ERROR_ACCESS_DENIED, EACCES }, > + { ERROR_INVALID_HANDLE, EBADF }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_LOCK_VIOLATION, EACCES }, > + { ERROR_FILE_EXISTS, EEXIST }, > + { ERROR_INVALID_PARAMETER, EINVAL }, > + { ERROR_FILENAME_EXCED_RANGE, ENAMETOOLONG }, > + }; > + const unsigned int WIN2ERRNO_TABLE_SIZE = sizeof(WIN2ERRNO_TABLE) / > sizeof(WIN2ERRNO_TABLE[0]); > + const errno_t DEFAULT_ERRNO_ERROR = -1; > + unsigned int i; > + > + for (i = 0; i< WIN2ERRNO_TABLE_SIZE; ++i) { > + if (WIN2ERRNO_TABLE[i].windows_code == lastError) { > + return WIN2ERRNO_TABLE[i].errno_code; > + } > + } > + return DEFAULT_ERRNO_ERROR; > +} > + > struct dirent { > char *d_name; > }; > @@ -493,9 +527,10 @@ DIR *opendir( char *path ) > > h = FindFirstFile( tmp,&data ); > > - if ( h == INVALID_HANDLE_VALUE ) > + if ( h == INVALID_HANDLE_VALUE ) { > + errno = win2errno(GetLastError()); > return NULL; > - > + } > d = ber_memalloc( sizeof(DIR) ); > if ( !d ) > return NULL; > @@ -518,10 +553,11 @@ struct dirent *readdir(DIR *dir) > } > return&dir->data; > } > -void closedir(DIR *dir) > +int closedir(DIR *dir) > { > FindClose(dir->dir); > ber_memfree(dir); > + return 0; > } > #endif > > > -- -- Howard Chu CTO, 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#6041) opendir() and closedir() bugs on Windows
by hyc＠symas.com 30 Mar '09

30 Mar '09

test.007(a)seznam.cz wrote: > Full_Name: Test Seven > Version: 2.4.15 > OS: Windows > URL: > Submission from: (NULL) (195.113.184.20) Thanks for the patch. The Windows code is known to work fine using gcc. Use of MSVC is discouraged since there have been so many incompatible CRT version changes in such a short span of time. (Actually, use of Microsoft-anything is generally discouraged; the same OpenLDAP code is at least 2-3x faster on the same hardware when using Linux instead of Windows.) > Two problems solved in the patch: > 1. closedir()'s retval is used, thus it must not be void. Windows uses implicit > declaration ``int closedir(int)'' because the function is not declared in > headers, thus the call in ldif.c compiles, but uses random value in EAX. > > 2. opendir() must set errno, because ldif_readdir() reads it. The patch converts > the most common Windows error codes to errno codes after FindFirstFile() WinAPI > call. > > The bugs prevent me from loading cn=config configuration. > > --- utils.c.orig 2009-01-22 01:00:58.000000000 +0100 > +++ utils.c 2009-03-27 14:28:33.500000000 +0100 > @@ -466,6 +466,40 @@ int mkstemp( char * template ) > #endif > > #ifdef _MSC_VER > +/* Equivalent of MS CRT's _dosmaperr(). > + * @param lastError[in] Result of GetLastError(). > + */ > +static errno_t win2errno(DWORD lastError) > +{ > + const struct { > + DWORD windows_code; > + errno_t errno_code; > + } WIN2ERRNO_TABLE[] = { > + { ERROR_SUCCESS, 0 }, > + { ERROR_FILE_NOT_FOUND, ENOENT }, > + { ERROR_PATH_NOT_FOUND, ENOENT }, > + { ERROR_TOO_MANY_OPEN_FILES, EMFILE }, > + { ERROR_ACCESS_DENIED, EACCES }, > + { ERROR_INVALID_HANDLE, EBADF }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_LOCK_VIOLATION, EACCES }, > + { ERROR_FILE_EXISTS, EEXIST }, > + { ERROR_INVALID_PARAMETER, EINVAL }, > + { ERROR_FILENAME_EXCED_RANGE, ENAMETOOLONG }, > + }; > + const unsigned int WIN2ERRNO_TABLE_SIZE = sizeof(WIN2ERRNO_TABLE) / > sizeof(WIN2ERRNO_TABLE[0]); > + const errno_t DEFAULT_ERRNO_ERROR = -1; > + unsigned int i; > + > + for (i = 0; i< WIN2ERRNO_TABLE_SIZE; ++i) { > + if (WIN2ERRNO_TABLE[i].windows_code == lastError) { > + return WIN2ERRNO_TABLE[i].errno_code; > + } > + } > + return DEFAULT_ERRNO_ERROR; > +} > + > struct dirent { > char *d_name; > }; > @@ -493,9 +527,10 @@ DIR *opendir( char *path ) > > h = FindFirstFile( tmp,&data ); > > - if ( h == INVALID_HANDLE_VALUE ) > + if ( h == INVALID_HANDLE_VALUE ) { > + errno = win2errno(GetLastError()); > return NULL; > - > + } > d = ber_memalloc( sizeof(DIR) ); > if ( !d ) > return NULL; > @@ -518,10 +553,11 @@ struct dirent *readdir(DIR *dir) > } > return&dir->data; > } > -void closedir(DIR *dir) > +int closedir(DIR *dir) > { > FindClose(dir->dir); > ber_memfree(dir); > + return 0; > } > #endif > > > -- -- Howard Chu CTO, 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#6041) opendir() and closedir() bugs on Windows
by hyc＠symas.com 30 Mar '09

30 Mar '09

test.007(a)seznam.cz wrote: > Full_Name: Test Seven > Version: 2.4.15 > OS: Windows > URL: > Submission from: (NULL) (195.113.184.20) Thanks for the patch. The Windows code is known to work fine using gcc. Use of MSVC is discouraged since there have been so many incompatible CRT version changes in such a short span of time. (Actually, use of Microsoft-anything is generally discouraged; the same OpenLDAP code is at least 2-3x faster on the same hardware when using Linux instead of Windows.) > Two problems solved in the patch: > 1. closedir()'s retval is used, thus it must not be void. Windows uses implicit > declaration ``int closedir(int)'' because the function is not declared in > headers, thus the call in ldif.c compiles, but uses random value in EAX. > > 2. opendir() must set errno, because ldif_readdir() reads it. The patch converts > the most common Windows error codes to errno codes after FindFirstFile() WinAPI > call. > > The bugs prevent me from loading cn=config configuration. > > --- utils.c.orig 2009-01-22 01:00:58.000000000 +0100 > +++ utils.c 2009-03-27 14:28:33.500000000 +0100 > @@ -466,6 +466,40 @@ int mkstemp( char * template ) > #endif > > #ifdef _MSC_VER > +/* Equivalent of MS CRT's _dosmaperr(). > + * @param lastError[in] Result of GetLastError(). > + */ > +static errno_t win2errno(DWORD lastError) > +{ > + const struct { > + DWORD windows_code; > + errno_t errno_code; > + } WIN2ERRNO_TABLE[] = { > + { ERROR_SUCCESS, 0 }, > + { ERROR_FILE_NOT_FOUND, ENOENT }, > + { ERROR_PATH_NOT_FOUND, ENOENT }, > + { ERROR_TOO_MANY_OPEN_FILES, EMFILE }, > + { ERROR_ACCESS_DENIED, EACCES }, > + { ERROR_INVALID_HANDLE, EBADF }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_LOCK_VIOLATION, EACCES }, > + { ERROR_FILE_EXISTS, EEXIST }, > + { ERROR_INVALID_PARAMETER, EINVAL }, > + { ERROR_FILENAME_EXCED_RANGE, ENAMETOOLONG }, > + }; > + const unsigned int WIN2ERRNO_TABLE_SIZE = sizeof(WIN2ERRNO_TABLE) / > sizeof(WIN2ERRNO_TABLE[0]); > + const errno_t DEFAULT_ERRNO_ERROR = -1; > + unsigned int i; > + > + for (i = 0; i< WIN2ERRNO_TABLE_SIZE; ++i) { > + if (WIN2ERRNO_TABLE[i].windows_code == lastError) { > + return WIN2ERRNO_TABLE[i].errno_code; > + } > + } > + return DEFAULT_ERRNO_ERROR; > +} > + > struct dirent { > char *d_name; > }; > @@ -493,9 +527,10 @@ DIR *opendir( char *path ) > > h = FindFirstFile( tmp,&data ); > > - if ( h == INVALID_HANDLE_VALUE ) > + if ( h == INVALID_HANDLE_VALUE ) { > + errno = win2errno(GetLastError()); > return NULL; > - > + } > d = ber_memalloc( sizeof(DIR) ); > if ( !d ) > return NULL; > @@ -518,10 +553,11 @@ struct dirent *readdir(DIR *dir) > } > return&dir->data; > } > -void closedir(DIR *dir) > +int closedir(DIR *dir) > { > FindClose(dir->dir); > ber_memfree(dir); > + return 0; > } > #endif > > > -- -- Howard Chu CTO, 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#6041) opendir() and closedir() bugs on Windows
by hyc＠symas.com 30 Mar '09

30 Mar '09

test.007(a)seznam.cz wrote: > Full_Name: Test Seven > Version: 2.4.15 > OS: Windows > URL: > Submission from: (NULL) (195.113.184.20) Thanks for the patch. The Windows code is known to work fine using gcc. Use of MSVC is discouraged since there have been so many incompatible CRT version changes in such a short span of time. (Actually, use of Microsoft-anything is generally discouraged; the same OpenLDAP code is at least 2-3x faster on the same hardware when using Linux instead of Windows.) > Two problems solved in the patch: > 1. closedir()'s retval is used, thus it must not be void. Windows uses implicit > declaration ``int closedir(int)'' because the function is not declared in > headers, thus the call in ldif.c compiles, but uses random value in EAX. > > 2. opendir() must set errno, because ldif_readdir() reads it. The patch converts > the most common Windows error codes to errno codes after FindFirstFile() WinAPI > call. > > The bugs prevent me from loading cn=config configuration. > > --- utils.c.orig 2009-01-22 01:00:58.000000000 +0100 > +++ utils.c 2009-03-27 14:28:33.500000000 +0100 > @@ -466,6 +466,40 @@ int mkstemp( char * template ) > #endif > > #ifdef _MSC_VER > +/* Equivalent of MS CRT's _dosmaperr(). > + * @param lastError[in] Result of GetLastError(). > + */ > +static errno_t win2errno(DWORD lastError) > +{ > + const struct { > + DWORD windows_code; > + errno_t errno_code; > + } WIN2ERRNO_TABLE[] = { > + { ERROR_SUCCESS, 0 }, > + { ERROR_FILE_NOT_FOUND, ENOENT }, > + { ERROR_PATH_NOT_FOUND, ENOENT }, > + { ERROR_TOO_MANY_OPEN_FILES, EMFILE }, > + { ERROR_ACCESS_DENIED, EACCES }, > + { ERROR_INVALID_HANDLE, EBADF }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_LOCK_VIOLATION, EACCES }, > + { ERROR_FILE_EXISTS, EEXIST }, > + { ERROR_INVALID_PARAMETER, EINVAL }, > + { ERROR_FILENAME_EXCED_RANGE, ENAMETOOLONG }, > + }; > + const unsigned int WIN2ERRNO_TABLE_SIZE = sizeof(WIN2ERRNO_TABLE) / > sizeof(WIN2ERRNO_TABLE[0]); > + const errno_t DEFAULT_ERRNO_ERROR = -1; > + unsigned int i; > + > + for (i = 0; i< WIN2ERRNO_TABLE_SIZE; ++i) { > + if (WIN2ERRNO_TABLE[i].windows_code == lastError) { > + return WIN2ERRNO_TABLE[i].errno_code; > + } > + } > + return DEFAULT_ERRNO_ERROR; > +} > + > struct dirent { > char *d_name; > }; > @@ -493,9 +527,10 @@ DIR *opendir( char *path ) > > h = FindFirstFile( tmp,&data ); > > - if ( h == INVALID_HANDLE_VALUE ) > + if ( h == INVALID_HANDLE_VALUE ) { > + errno = win2errno(GetLastError()); > return NULL; > - > + } > d = ber_memalloc( sizeof(DIR) ); > if ( !d ) > return NULL; > @@ -518,10 +553,11 @@ struct dirent *readdir(DIR *dir) > } > return&dir->data; > } > -void closedir(DIR *dir) > +int closedir(DIR *dir) > { > FindClose(dir->dir); > ber_memfree(dir); > + return 0; > } > #endif > > > -- -- Howard Chu CTO, 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#6041) opendir() and closedir() bugs on Windows
by hyc＠symas.com 30 Mar '09

30 Mar '09

test.007(a)seznam.cz wrote: > Full_Name: Test Seven > Version: 2.4.15 > OS: Windows > URL: > Submission from: (NULL) (195.113.184.20) Thanks for the patch. The Windows code is known to work fine using gcc. Use of MSVC is discouraged since there have been so many incompatible CRT version changes in such a short span of time. (Actually, use of Microsoft-anything is generally discouraged; the same OpenLDAP code is at least 2-3x faster on the same hardware when using Linux instead of Windows.) > Two problems solved in the patch: > 1. closedir()'s retval is used, thus it must not be void. Windows uses implicit > declaration ``int closedir(int)'' because the function is not declared in > headers, thus the call in ldif.c compiles, but uses random value in EAX. > > 2. opendir() must set errno, because ldif_readdir() reads it. The patch converts > the most common Windows error codes to errno codes after FindFirstFile() WinAPI > call. > > The bugs prevent me from loading cn=config configuration. > > --- utils.c.orig 2009-01-22 01:00:58.000000000 +0100 > +++ utils.c 2009-03-27 14:28:33.500000000 +0100 > @@ -466,6 +466,40 @@ int mkstemp( char * template ) > #endif > > #ifdef _MSC_VER > +/* Equivalent of MS CRT's _dosmaperr(). > + * @param lastError[in] Result of GetLastError(). > + */ > +static errno_t win2errno(DWORD lastError) > +{ > + const struct { > + DWORD windows_code; > + errno_t errno_code; > + } WIN2ERRNO_TABLE[] = { > + { ERROR_SUCCESS, 0 }, > + { ERROR_FILE_NOT_FOUND, ENOENT }, > + { ERROR_PATH_NOT_FOUND, ENOENT }, > + { ERROR_TOO_MANY_OPEN_FILES, EMFILE }, > + { ERROR_ACCESS_DENIED, EACCES }, > + { ERROR_INVALID_HANDLE, EBADF }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, > + { ERROR_LOCK_VIOLATION, EACCES }, > + { ERROR_FILE_EXISTS, EEXIST }, > + { ERROR_INVALID_PARAMETER, EINVAL }, > + { ERROR_FILENAME_EXCED_RANGE, ENAMETOOLONG }, > + }; > + const unsigned int WIN2ERRNO_TABLE_SIZE = sizeof(WIN2ERRNO_TABLE) / > sizeof(WIN2ERRNO_TABLE[0]); > + const errno_t DEFAULT_ERRNO_ERROR = -1; > + unsigned int i; > + > + for (i = 0; i< WIN2ERRNO_TABLE_SIZE; ++i) { > + if (WIN2ERRNO_TABLE[i].windows_code == lastError) { > + return WIN2ERRNO_TABLE[i].errno_code; > + } > + } > + return DEFAULT_ERRNO_ERROR; > +} > + > struct dirent { > char *d_name; > }; > @@ -493,9 +527,10 @@ DIR *opendir( char *path ) > > h = FindFirstFile( tmp,&data ); > > - if ( h == INVALID_HANDLE_VALUE ) > + if ( h == INVALID_HANDLE_VALUE ) { > + errno = win2errno(GetLastError()); > return NULL; > - > + } > d = ber_memalloc( sizeof(DIR) ); > if ( !d ) > return NULL; > @@ -518,10 +553,11 @@ struct dirent *readdir(DIR *dir) > } > return&dir->data; > } > -void closedir(DIR *dir) > +int closedir(DIR *dir) > { > FindClose(dir->dir); > ber_memfree(dir); > + return 0; > } > #endif > > > -- -- Howard Chu CTO, 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#6038) Write slapadd progress meter to stdout
by michael＠stroeder.com 27 Mar '09

27 Mar '09

rein(a)OpenLDAP.org wrote: > For programs that produce real output (slapcat, ldapsearch) this might > be true, but for those that don't (slapadd, slapd) I totally disagree. > Stderr is for errors, not chatty progress or debug messages. I always > hate it when the error messages gets burried in lots of noice. > > My preference is to have progress and debug messages on stdout, enabled > by options or at least possible to disable by options. Real output must > be possible to send to a file specified as argument, to separate it from > the progress and debug noice. Stderr is for error messages, and > possibly out-of-band messages. +1 Ciao, Michael.

1 0

(ITS#6041) opendir() and closedir() bugs on Windows
by test.007＠seznam.cz 27 Mar '09

27 Mar '09

Full_Name: Test Seven Version: 2.4.15 OS: Windows URL: Submission from: (NULL) (195.113.184.20) Two problems solved in the patch: 1. closedir()'s retval is used, thus it must not be void. Windows uses implicit declaration ``int closedir(int)'' because the function is not declared in headers, thus the call in ldif.c compiles, but uses random value in EAX. 2. opendir() must set errno, because ldif_readdir() reads it. The patch converts the most common Windows error codes to errno codes after FindFirstFile() WinAPI call. The bugs prevent me from loading cn=config configuration. --- utils.c.orig 2009-01-22 01:00:58.000000000 +0100 +++ utils.c 2009-03-27 14:28:33.500000000 +0100 @@ -466,6 +466,40 @@ int mkstemp( char * template ) #endif #ifdef _MSC_VER +/* Equivalent of MS CRT's _dosmaperr(). + * @param lastError[in] Result of GetLastError(). + */ +static errno_t win2errno(DWORD lastError) +{ + const struct { + DWORD windows_code; + errno_t errno_code; + } WIN2ERRNO_TABLE[] = { + { ERROR_SUCCESS, 0 }, + { ERROR_FILE_NOT_FOUND, ENOENT }, + { ERROR_PATH_NOT_FOUND, ENOENT }, + { ERROR_TOO_MANY_OPEN_FILES, EMFILE }, + { ERROR_ACCESS_DENIED, EACCES }, + { ERROR_INVALID_HANDLE, EBADF }, + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, + { ERROR_NOT_ENOUGH_MEMORY, ENOMEM }, + { ERROR_LOCK_VIOLATION, EACCES }, + { ERROR_FILE_EXISTS, EEXIST }, + { ERROR_INVALID_PARAMETER, EINVAL }, + { ERROR_FILENAME_EXCED_RANGE, ENAMETOOLONG }, + }; + const unsigned int WIN2ERRNO_TABLE_SIZE = sizeof(WIN2ERRNO_TABLE) / sizeof(WIN2ERRNO_TABLE[0]); + const errno_t DEFAULT_ERRNO_ERROR = -1; + unsigned int i; + + for (i = 0; i < WIN2ERRNO_TABLE_SIZE; ++i) { + if (WIN2ERRNO_TABLE[i].windows_code == lastError) { + return WIN2ERRNO_TABLE[i].errno_code; + } + } + return DEFAULT_ERRNO_ERROR; +} + struct dirent { char *d_name; }; @@ -493,9 +527,10 @@ DIR *opendir( char *path ) h = FindFirstFile( tmp, &data ); - if ( h == INVALID_HANDLE_VALUE ) + if ( h == INVALID_HANDLE_VALUE ) { + errno = win2errno(GetLastError()); return NULL; - + } d = ber_memalloc( sizeof(DIR) ); if ( !d ) return NULL; @@ -518,10 +553,11 @@ struct dirent *readdir(DIR *dir) } return &dir->data; } -void closedir(DIR *dir) +int closedir(DIR *dir) { FindClose(dir->dir); ber_memfree(dir); + return 0; } #endif

1 0

Re: (ITS#6033) Segmentation fault - somewhere below syncprov_matchops / test_filter
by Howard Chu 27 Mar '09

27 Mar '09

Please test with RE24 instead of 2.4.15 and followup with your results.

1 0

Re: (ITS#6040) SEGV - hdb_cache_lru_purge / bdb_cache_delete_internal / ldap_pvt_thread_mutex_lock
by Howard Chu 27 Mar '09

27 Mar '09

Please use RE24 instead of 2.4.15 and followup with your results.

1 0

Re: (ITS#6038) Write slapadd progress meter to stdout
by hyc＠symas.com 27 Mar '09

27 Mar '09

Rein Tollevik wrote: > hyc(a)symas.com wrote: >> Hallvard B Furuseth wrote: >>> hyc(a)symas.com writes: >>>> Just redirect stdin, and the meter will automatically turn itself off. >>> It was stderr, not stdin - which after all can be the ldif. >>> Stdout is better. >>> >> No, you misunderstand. >> >> Redirecting stderr (or stdout) merely sends the progress info to that >> destination. Piping input on stdin will turn it off, so there's no progress >> info to redirect. > > No, the code only enables the meter if stdout (was: stderr) is a tty, so > redirecting it anywhere disables it. The meter was enabled both by > "slapadd -l file" and "slapadd< file", but "cat file | slapadd" did > disable it. IMHO not exactly an obvious way.. > > An option sounds more and more like the solution, what about "-m" for > meter? Though I'm not sure whether I prefer that it should be enabled > by default and the option disable it, or the other way around.. We already decided it should be enabled by default. Setting any explicit debug flag already disables it. >> Writing progress to stderr is the correct thing to do, because it's transient >> info that isn't meant to be saved. Checking whether stdout is redirected or >> not is fine, but writing progress info to stdout is not. > > For programs that produce real output (slapcat, ldapsearch) this might > be true, but for those that don't (slapadd, slapd) I totally disagree. > Stderr is for errors, not chatty progress or debug messages. I always > hate it when the error messages gets burried in lots of noice. Stderr is for any diagnostic info. Progress reports are diagnostic, they are certainly not the main purpose of any of these programs. Stdout is for anything related to the primary purpose of these programs. For slapcat, that's obvious. For slapadd/slapindex, these programs' purpose is not to produce any textual output at all; stdout is unused. That doesn't mean it's correct to ignore these long-standard Unix I/O usage conventions and suddenly switch stderr output to stdout. And as already noted, if you're looking for debug messages, then the meter will be disabled. > My preference is to have progress and debug messages on stdout, enabled > by options or at least possible to disable by options. Quite simply, this is completely wrong. It also introduces an unnecessary inconsistency between slapcat and slapadd behavior. Progress belongs on stderr, period. -- -- Howard Chu CTO, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

(ITS#6040) SEGV - hdb_cache_lru_purge / bdb_cache_delete_internal / ldap_pvt_thread_mutex_lock
by oskar＠deckle.co.za 27 Mar '09

27 Mar '09

Full_Name: Oskar Pearson Version: 2.4.15 OS: Ubuntu hardy URL: ftp://ftp.openldap.org/incoming/slapd-configs-qualica-1.tar.gz Submission from: (NULL) (94.194.140.71) This is the same machine as ITS #6033. Details included here for easy reference: Openldap 2.4.15 (with no patches) was self-compiled with no patches (CFLAGS="-ggdb3 -O0" ./configure), linked against standard OS libraries (ubuntu hardy) for BDB etc. BDB is 4.6.21-6ubuntu1. Kernel is 2.6.24-19-server, libc is libc6-i686/2.7-10ubuntu3. I do not have a debug log of this crash as it occurred on a live server, which is too busy for that level of logging. I also do not have a replaceable example. I have core dumps and can print out whatever is necessary there - I appreciate your time and help. If I've missed some important info, please let me know and I'll rectify asap. System overview: We have a master server which replicates to > 100 machines for Samba purposes across ADSL lines with a VPN. These lines are quite busy, so replication writes may be slow across the network, which may have implications for locks. Config files are included at the url below. I've realised through looking at the code and config files that I should probably increase the dncachesize parameter, as the number of DNs in the db is > 2000. This would help reduce the possibility of the bug occurring, but I think the bug will still exist. gdb --core=/root/core.26366 servers/slapd/slapd ... (gdb) bt #0 0xb7dd3540 in pthread_mutex_lock () from /lib/tls/i686/cmov/libpthread.so.0 #1 0x0817beef in ldap_pvt_thread_mutex_lock (mutex=0x34) at thr_posix.c:296 #2 0x08144215 in bdb_cache_delete_internal (cache=0x82b6df4, e=0x84017d8, decr=0) at cache.c:1335 #3 0x081430e5 in hdb_cache_lru_purge (bdb=0x82b6db8) at cache.c:749 #4 0x081438ba in hdb_cache_find_id (op=0x86c01d0, tid=0x86d0a30, id=1792, eip=0x9f6f1e68, flag=2, lock=0x9f6f1e00) at cache.c:1016 #5 0x08115c29 in hdb_search (op=0x86c01d0, rs=0x9f6f3100) at search.c:705 #6 0x080f179a in overlay_op_walk (op=0x86c01d0, rs=0x9f6f3100, which=op_search, oi=0x82d2b88, on=0x0) at backover.c:669 #7 0x080f19a2 in over_op_func (op=0x86c01d0, rs=0x9f6f3100, which=op_search) at backover.c:721 #8 0x080f1a46 in over_op_search (op=0x86c01d0, rs=0x9f6f3100) at backover.c:743 #9 0x080724ed in fe_op_search (op=0x86c01d0, rs=0x9f6f3100) at search.c:366 #10 0x08071e60 in do_search (op=0x86c01d0, rs=0x9f6f3100) at search.c:217 #11 0x0806eb44 in connection_operation (ctx=0x9f6f31f0, arg_v=0x86c01d0) at connection.c:1133 #12 0x0806f01e in connection_read_thread (ctx=0x9f6f31f0, argv=0x46) at connection.c:1259 #13 0x0817ad1f in ldap_int_thread_pool_wrapper (xpool=0x829caf0) at tpool.c:663 #14 0xb7dd14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #15 0xb7ba0e5e in clone () from /lib/tls/i686/cmov/libc.so.6 (gdb) frame 2 #2 0x08144215 in bdb_cache_delete_internal (cache=0x82b6df4, e=0x84017d8, decr=0) at cache.c:1335 1335 bdb_cache_entryinfo_lock( e->bei_parent ); (gdb) p e->bei_parent $1 = (struct bdb_entry_info *) 0x0 (gdb) up #3 0x081430e5 in hdb_cache_lru_purge (bdb=0x82b6db8) at cache.c:749 749 bdb_cache_delete_internal( &bdb->bi_cache, elru, 0 ); (gdb) p *elru $2 = {bei_parent = 0x0, bei_id = 17, bei_lockpad = 0, bei_state = 0, bei_finders = 0, bei_nrdn = {bv_len = 24, bv_val = 0x0}, bei_rdn = {bv_len = 24, bv_val = 0x0}, bei_modrdns = 0, bei_ckids = 0, bei_dkids = 0, bei_e = 0x0, bei_kids = 0x0, bei_kids_mutex = {__data = {__lock = 1, __count = 0, __owner = 28050, __kind = 0, __nusers = 1, {__spins = 0, __list = {__next = 0x0}}}, __size = "\001\000\000\000\000\000\000\000\222m\000\000\000\000\000\000\001\000\000\000\000\000\000", __align = 1}, bei_lrunext = 0x83b5e70, bei_lruprev = 0x867b980} (gdb) p bdb->bi_cache.c_leaves $3 = 3737 (gdb) p eimax $4 = 2000 (gdb) p bdb->bi_cache $5 = {c_eifree = 0x8665b00, c_idtree = 0x9b322498, c_lruhead = 0x84017d8, c_lrutail = 0x867b980, c_dntree = {bei_parent = 0x0, bei_id = 0, bei_lockpad = 0, bei_state = 128, bei_finders = 0, bei_nrdn = {bv_len = 0, bv_val = 0x0}, bei_rdn = {bv_len = 0, bv_val = 0x0}, bei_modrdns = 0, bei_ckids = 1, bei_dkids = 2, bei_e = 0x8320704, bei_kids = 0x8320248, bei_kids_mutex = {__data = {__lock = 0, __count = 0, __owner = 0, __kind = 0, __nusers = 0, { __spins = 0, __list = {__next = 0x0}}}, __size = '\0' <repeats 23 times>, __align = 0}, bei_lrunext = 0x0, bei_lruprev = 0x0}, c_maxsize = 1000, c_cursize = 117171, c_minfree = 1, c_eimax = 2000, c_eiused = 3740, c_leaves = 3737, c_purging = 1, c_txn = 0x8320188, c_rwlock = {__data = {__lock = 0, __nr_readers = 0, __readers_wakeup = 8985, __writer_wakeup = 9500, __nr_readers_queued = 0, __nr_writers_queued = 0, __flags = 0 '\0', __shared = 0 '\0', __pad1 = 0 '\0', __pad2 = 0 '\0', __writer = 0}, __size = "\000\000\000\000\000\000\000\000\031#\000\000\034%", '\0' <repeats 17 times>, __align = 0}, c_lru_mutex = {__data = {__lock = 1, __count = 0, __owner = 28050, __kind = 0, __nusers = 1, {__spins = 0, __list = {__next = 0x0}}}, __size = "\001\000\000\000\000\000\000\000\222m\000\000\000\000\000\000\001\000\000\000\000\000\000", __align = 1}, c_count_mutex = {__data = { __lock = 0, __count = 0, __owner = 0, __kind = 0, __nusers = 0, {__spins = 0, __list = {__next = 0x0}}}, __size = '\0' <repeats 23 times>, __align = 0}, c_eifree_mutex = {__data = {__lock = 0, __count = 0, __owner = 0, __kind = 0, __nusers = 0, {__spins = 0, __list = {__next = 0x0}}}, __size = '\0' <repeats 23 times>, __align = 0}} If there's anything else you'd like me to print out, please let me know. thread apply all bt follows: Thread 10 (process 26366): #0 0xb7f18410 in __kernel_vsyscall () #1 0xb7dd2775 in pthread_join () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817be19 in ldap_pvt_thread_join (thread=2730888080, thread_return=0x0) at thr_posix.c:197 #3 0x0806bca3 in slapd_daemon () at daemon.c:2740 #4 0x0804e8c7 in main (argc=1, argv=0xbfc63124) at main.c:948 Thread 9 (process 26372): #0 0xb7f18410 in __kernel_vsyscall () #1 0xb7ba1676 in epoll_wait () from /lib/tls/i686/cmov/libc.so.6 #2 0x0806ae3a in slapd_daemon_task (ptr=0x0) at daemon.c:2366 #3 0xb7dd14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #4 0xb7ba0e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 8 (process 26374): #0 0xb7f18410 in __kernel_vsyscall () #1 0xb7dd5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817beae in ldap_pvt_thread_cond_wait (cond=0x829cb0c, mutex=0x829caf4) at thr_posix.c:277 #3 0x0817acac in ldap_int_thread_pool_wrapper (xpool=0x829caf0) at tpool.c:654 #4 0xb7dd14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #5 0xb7ba0e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 7 (process 27916): #0 0xb7f18410 in __kernel_vsyscall () #1 0xb7dd5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817beae in ldap_pvt_thread_cond_wait (cond=0x829cb0c, mutex=0x829caf4) at thr_posix.c:277 #3 0x0817acac in ldap_int_thread_pool_wrapper (xpool=0x829caf0) at tpool.c:654 #4 0xb7dd14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #5 0xb7ba0e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 6 (process 28043): #0 0xb7f18410 in __kernel_vsyscall () #1 0xb7dd5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817beae in ldap_pvt_thread_cond_wait (cond=0x829cb0c, mutex=0x829caf4) at thr_posix.c:277 #3 0x0817acac in ldap_int_thread_pool_wrapper (xpool=0x829caf0) at tpool.c:654 #4 0xb7dd14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #5 0xb7ba0e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 5 (process 28044): #0 0xb7f18410 in __kernel_vsyscall () #1 0xb7dd5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817beae in ldap_pvt_thread_cond_wait (cond=0x829cb0c, mutex=0x829caf4) at thr_posix.c:277 #3 0x0817acac in ldap_int_thread_pool_wrapper (xpool=0x829caf0) at tpool.c:654 #4 0xb7dd14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #5 0xb7ba0e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 4 (process 28045): #0 0xb7f18410 in __kernel_vsyscall () #1 0xb7dd5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817beae in ldap_pvt_thread_cond_wait (cond=0x829cb0c, mutex=0x829caf4) at thr_posix.c:277 #3 0x0817acac in ldap_int_thread_pool_wrapper (xpool=0x829caf0) at tpool.c:654 #4 0xb7dd14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #5 0xb7ba0e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 3 (process 28046): #0 0xb7f18410 in __kernel_vsyscall () #1 0xb7dd5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817beae in ldap_pvt_thread_cond_wait (cond=0x829cb0c, mutex=0x829caf4) at thr_posix.c:277 #3 0x0817acac in ldap_int_thread_pool_wrapper (xpool=0x829caf0) at tpool.c:654 #4 0xb7dd14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #5 0xb7ba0e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 2 (process 28048): #0 0xb7f18410 in __kernel_vsyscall () #1 0xb7dd5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817beae in ldap_pvt_thread_cond_wait (cond=0x829cb0c, mutex=0x829caf4) at thr_posix.c:277 #3 0x0817acac in ldap_int_thread_pool_wrapper (xpool=0x829caf0) at tpool.c:654 #4 0xb7dd14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #5 0xb7ba0e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 1 (process 28050): #0 0xb7dd3540 in pthread_mutex_lock () from /lib/tls/i686/cmov/libpthread.so.0 #1 0x0817beef in ldap_pvt_thread_mutex_lock (mutex=0x34) at thr_posix.c:296 #2 0x08144215 in bdb_cache_delete_internal (cache=0x82b6df4, e=0x84017d8, decr=0) at cache.c:1335 #3 0x081430e5 in hdb_cache_lru_purge (bdb=0x82b6db8) at cache.c:749 #4 0x081438ba in hdb_cache_find_id (op=0x86c01d0, tid=0x86d0a30, id=1792, eip=0x9f6f1e68, flag=2, lock=0x9f6f1e00) at cache.c:1016 #5 0x08115c29 in hdb_search (op=0x86c01d0, rs=0x9f6f3100) at search.c:705 #6 0x080f179a in overlay_op_walk (op=0x86c01d0, rs=0x9f6f3100, which=op_search, oi=0x82d2b88, on=0x0) at backover.c:669 #7 0x080f19a2 in over_op_func (op=0x86c01d0, rs=0x9f6f3100, which=op_search) at backover.c:721 #8 0x080f1a46 in over_op_search (op=0x86c01d0, rs=0x9f6f3100) at backover.c:743 #9 0x080724ed in fe_op_search (op=0x86c01d0, rs=0x9f6f3100) at search.c:366 #10 0x08071e60 in do_search (op=0x86c01d0, rs=0x9f6f3100) at search.c:217 #11 0x0806eb44 in connection_operation (ctx=0x9f6f31f0, arg_v=0x86c01d0) at connection.c:1133 #12 0x0806f01e in connection_read_thread (ctx=0x9f6f31f0, argv=0x46) at connection.c:1259 #13 0x0817ad1f in ldap_int_thread_pool_wrapper (xpool=0x829caf0) at tpool.c:663 #14 0xb7dd14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #15 0xb7ba0e5e in clone () from /lib/tls/i686/cmov/libc.so.6

1 0

Re: (ITS#6038) Write slapadd progress meter to stdout
by rein＠OpenLDAP.org 27 Mar '09

27 Mar '09

hyc(a)symas.com wrote: > Hallvard B Furuseth wrote: >> hyc(a)symas.com writes: >>> Just redirect stdin, and the meter will automatically turn itself off. >> It was stderr, not stdin - which after all can be the ldif. >> Stdout is better. >> > No, you misunderstand. > > Redirecting stderr (or stdout) merely sends the progress info to that > destination. Piping input on stdin will turn it off, so there's no progress > info to redirect. No, the code only enables the meter if stdout (was: stderr) is a tty, so redirecting it anywhere disables it. The meter was enabled both by "slapadd -l file" and "slapadd < file", but "cat file | slapadd" did disable it. IMHO not exactly an obvious way.. An option sounds more and more like the solution, what about "-m" for meter? Though I'm not sure whether I prefer that it should be enabled by default and the option disable it, or the other way around.. > Writing progress to stderr is the correct thing to do, because it's transient > info that isn't meant to be saved. Checking whether stdout is redirected or > not is fine, but writing progress info to stdout is not. For programs that produce real output (slapcat, ldapsearch) this might be true, but for those that don't (slapadd, slapd) I totally disagree. Stderr is for errors, not chatty progress or debug messages. I always hate it when the error messages gets burried in lots of noice. My preference is to have progress and debug messages on stdout, enabled by options or at least possible to disable by options. Real output must be possible to send to a file specified as argument, to separate it from the progress and debug noice. Stderr is for error messages, and possibly out-of-band messages. Rein

1 0

(ITS#6039) Windows sockets initialization and shutdown bugs in slapd
by test.007＠seznam.cz 27 Mar '09

27 Mar '09

Full_Name: Test Seven Version: 2.4.15 OS: Windows URL: Submission from: (NULL) (195.113.184.20) I'm afraid slapd won't run on Windows without the first half of the following patch. AppVerifier will choke on double tcp_close() without the second part. --- daemon.c.orig 2009-02-13 03:35:40.000000000 +0100 +++ daemon.c 2009-03-27 09:24:22.515625000 +0100 @@ -537,6 +537,7 @@ static struct slap_daemon { slap_daemon.sd_flags = (char *)(slapd_ws_sockets + dtblsize); \ slap_daemon.sd_rflags = slap_daemon.sd_flags + dtblsize; \ memset( slapd_ws_sockets, -1, dtblsize * sizeof(SOCKET) ); \ + memset(slap_daemon.sd_flags, 0, dtblsize); /* Don't forget to initialize this, used in slapd_add() called from slapd_daemon_task() */ \ slapd_ws_sockets[0] = wake_sds[0]; \ slapd_ws_sockets[1] = wake_sds[1]; \ wake_sds[0] = 0; \ @@ -1646,7 +1647,7 @@ slapd_daemon_destroy( void ) { connections_destroy(); #ifdef HAVE_WINSOCK - if ( wake_sds[1] != INVALID_SOCKET && wake_sds[1] != wake_sds[0] ) + if ( wake_sds[1] != INVALID_SOCKET && SLAP_FD2SOCK(wake_sds[1]) != SLAP_FD2SOCK(wake_sds[0]) ) #endif /* HAVE_WINSOCK */ tcp_close( SLAP_FD2SOCK(wake_sds[1]) ); #ifdef HAVE_WINSOCK

1 0

RE: (ITS#5919) URI syntaxe (ldap:///dc=my%2cdc=domaine)
by philippe.eychart＠informatique.gov.pf 27 Mar '09

27 Mar '09

"result waited" : sorry, I wanted to say "result such as hoped" ;-)

1 0

RE: (ITS#5919) URI syntaxe (ldap:///dc=my%2cdc=domaine)
by philippe.eychart＠informatique.gov.pf 27 Mar '09

27 Mar '09

Hi, So, no idea about how to save the result of a previous SRV request in the implementation of the weight notion according to the rfc 2782?... >From my part either :-( (perhaps, somewhere in the kernel?...) PE PS: About the patch, I use it for several weeks now and everything seems good : no bugg, functioning and result waited ;-)

1 0

Re: (ITS#6038) Write slapadd progress meter to stdout
by hyc＠symas.com 27 Mar '09

27 Mar '09

Hallvard B Furuseth wrote: > hyc(a)symas.com writes: >> Just redirect stdin, and the meter will automatically turn itself off. > > It was stderr, not stdin - which after all can be the ldif. > Stdout is better. > No, you misunderstand. Redirecting stderr (or stdout) merely sends the progress info to that destination. Piping input on stdin will turn it off, so there's no progress info to redirect. Writing progress to stderr is the correct thing to do, because it's transient info that isn't meant to be saved. Checking whether stdout is redirected or not is fine, but writing progress info to stdout is not. -- -- Howard Chu CTO, 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#6038) Write slapadd progress meter to stdout
by h.b.furuseth＠usit.uio.no 27 Mar '09

27 Mar '09

hyc(a)symas.com writes: > Just redirect stdin, and the meter will automatically turn itself off. It was stderr, not stdin - which after all can be the ldif. Stdout is better. -- Hallvard

1 0

Re: (ITS#6038) Write slapadd progress meter to stdout
by hyc＠symas.com 26 Mar '09

26 Mar '09

rein(a)OpenLDAP.org wrote: > Full_Name: Rein Tollevik > Version: CVS HEAD > OS: Irrelevant > URL: > Submission from: (NULL) (84.215.36.97) > Submitted by: rein > > > The slapadd progress meter is written to stderr, and the only way to get rid of > it is currently to redirect stderr. I don't want the meter in my small test > scripts, as it is not very useful on minimal input and clutters the test output. > It should be possible to disable the meter without loosing all error messages, > a fix that writes the meter to stdout instead is coming. > > Options to enable/disable the meter would be nice, but the obvious candidates -q > and -v are both occupied... Just redirect stdin, and the meter will automatically turn itself off. -- -- Howard Chu CTO, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

(ITS#6038) Write slapadd progress meter to stdout
by rein＠OpenLDAP.org 26 Mar '09

26 Mar '09

Full_Name: Rein Tollevik Version: CVS HEAD OS: Irrelevant URL: Submission from: (NULL) (84.215.36.97) Submitted by: rein The slapadd progress meter is written to stderr, and the only way to get rid of it is currently to redirect stderr. I don't want the meter in my small test scripts, as it is not very useful on minimal input and clutters the test output. It should be possible to disable the meter without loosing all error messages, a fix that writes the meter to stdout instead is coming. Options to enable/disable the meter would be nice, but the obvious candidates -q and -v are both occupied... Rein Tollevik Basefarm AS

1 0

(ITS#6037) errno in signal handlers
by h.b.furuseth＠usit.uio.no 26 Mar '09

26 Mar '09

Full_Name: Hallvard B Furuseth Version: HEAD, RE24 OS: URL: Submission from: (NULL) (129.240.6.233) Submitted by: hallvard Signal handlers slapd/daemon.c:slap_sig_shutdown()/slap_sig_wake() need to save/restore errno, since they call functions that can modify errno. And/or possibly they need sock_errno() - sock_errset() from ac/socket.h, since they use tcp_write()? I don't know if that's safe inside signal handlers, or if Windows itself saves/restores this code around signal handlers. slapd/main.c:wait4child() can loop forever after EINTR, since it checks for errno even when the waitpid returns 0. I don't know if there is a good reason it does that. Some of the commits have my name on them but I think I copied it from elsewhere. Anyway, I'm doing the minimal changes now: save/restore errno, and set errno=0 in each loop iteration instead of before the loop. Anyone with better ideas can take it from there.

1 0

Re: (ITS#6035) slapd requires restart after modifying olcAuthzRegexp
by ando＠sys-net.it 26 Mar '09

26 Mar '09

quanah(a)zimbra.com wrote: > --On Thursday, March 26, 2009 4:14 PM +0000 hyc(a)symas.com wrote: > >> This is a known limitation in authz regexp support. There are no plans >> to change this any time soon. > > Where's this limitation documented? What other parameters in the config > backend have the same flaw? We've certainly fixed this for a number of > other things. Indeed, it has been finally, although rather inelegantly, fixed in slapo-rwm(5), AFAIR. 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 Fax: +39 0382 476497 Email: ando(a)sys-net.it -----------------------------------

1 0

Re: (ITS#6035) slapd requires restart after modifying olcAuthzRegexp
by quanah＠zimbra.com 26 Mar '09

26 Mar '09

--On Thursday, March 26, 2009 4:14 PM +0000 hyc(a)symas.com wrote: > This is a known limitation in authz regexp support. There are no plans > to change this any time soon. Where's this limitation documented? What other parameters in the config backend have the same flaw? We've certainly fixed this for a number of other things. I don't even see authz-regexp/olcAuthzRegexp mentioned in chapters 5 or 6 in the admin guide, and the man pages don't note this limitation. --Quanah -- Quanah Gibson-Mount Principal Software Engineer Zimbra, Inc -------------------- Zimbra :: the leader in open source messaging and collaboration

1 0

Re: (ITS#6035) slapd requires restart after modifying olcAuthzRegexp
by hyc＠symas.com 26 Mar '09

26 Mar '09

This is a known limitation in authz regexp support. There are no plans to change this any time soon. On Thu, Mar 26, 2009 at 03:33:37AM +0000, dgbaley27(a)verizon.net wrote: > Full_Name: Matthew Monaco > Version: 2.4.11 > OS: GNU/Linux 2.6.27-7 (Ubuntu 8.10 Server JeOS) > URL: > Submission from: (NULL) (96.242.209.249) > > > After modifying existing olcAuthzRegexp and/or adding/removing additional > olcAuthzRegexp from cn=config, I needed to restart slapd for the changes to take > effect. > > I'm not sure if it matters but I was using ldapvi to do the modification. > > I can however, confirm that the changes immediately appeared in various other > ldap browsers (such as Apache Directory Studio).

1 0

(ITS#6036) Crash: UUIDNormalize doesn't clear output on error
by test.007＠seznam.cz 26 Mar '09

26 Mar '09

Full_Name: Test Seven Version: 2.4.15 OS: Windows URL: Submission from: (NULL) (195.39.55.20) --- schema_init.orig 2009-01-22 01:01:03.000000000 +0100 +++ schema_init.c 2009-03-26 15:12:15.578125000 +0100 @@ -2814,6 +2814,7 @@ UUIDNormalize( } else { slap_sl_free( normalized->bv_val, ctx ); + BER_BVZERO(normalized); return LDAP_INVALID_SYNTAX; } I encountered a crash in str2entry2() final cleanup. It turned out that it called ordered_value_normalize(), which called UUIDNormalize(), which left the value nonzero but already free-d.

1 0

(ITS#6035) slapd requires restart after modifying olcAuthzRegexp
by dgbaley27＠verizon.net 26 Mar '09

26 Mar '09

Full_Name: Matthew Monaco Version: 2.4.11 OS: GNU/Linux 2.6.27-7 (Ubuntu 8.10 Server JeOS) URL: Submission from: (NULL) (96.242.209.249) After modifying existing olcAuthzRegexp and/or adding/removing additional olcAuthzRegexp from cn=config, I needed to restart slapd for the changes to take effect. I'm not sure if it matters but I was using ldapvi to do the modification. I can however, confirm that the changes immediately appeared in various other ldap browsers (such as Apache Directory Studio).

1 0

Re: (ITS#6034) Assert failure - connection_state_closing / send_ldap_ber / slap_send_search_entry / syncprov_sendresp
by hyc＠symas.com 25 Mar '09

25 Mar '09

oskar(a)deckle.co.za wrote: > Full_Name: Oskar Pearson > Version: 2.4.15 > OS: Ubuntu hardy > URL: ftp://ftp.openldap.org/incoming/slapd-configs-qualica-1.tar.gz > Submission from: (NULL) (94.194.140.71) > > > This is the same machine as ITS #6033. Details included here for easy > reference: > > Openldap 2.4.15 (with no patches) was self-compiled with no patches > (CFLAGS="-ggdb3 -O0" ./configure), linked against standard OS libraries (ubuntu > hardy) for BDB etc. BDB is 4.6.21-6ubuntu1. Kernel is 2.6.24-19-server, libc is > libc6-i686/2.7-10ubuntu3. > > I do not have a debug log of this crash as it occurred on a live server, which > is too busy for that level of logging. I also do not have a replaceable > example. > I have core dumps and can print out whatever is necessary there - I appreciate > your time and help. This bug was fixed in CVS already and the fix will be out in 2.4.16 soon. The fix is already in RE24... > gdb --core=../core.29266 --directory=servers/slapd/ > --directory=servers/slapd/overlays servers/slapd/slapd > (gdb) bt > #0 0xb7ef8410 in __kernel_vsyscall () > #1 0xb7ad5085 in raise () from /lib/tls/i686/cmov/libc.so.6 > #2 0xb7ad6a01 in abort () from /lib/tls/i686/cmov/libc.so.6 > #3 0xb7ace10e in __assert_fail () from /lib/tls/i686/cmov/libc.so.6 > #4 0x0806d6c6 in connection_state_closing (c=0xb79c9644) at connection.c:686 > #5 0x08081bdd in send_ldap_ber (conn=0xb79c9644, ber=0xa143ab8c) at > result.c:147 > #6 0x08085b20 in slap_send_search_entry (op=0xa143ae38, rs=0xa143acdc) at > result.c:1246 > #7 0x0815dbcb in syncprov_sendresp (op=0xa143ae38, opc=0xa143adb8, > so=0x9ab66dc8, e=0xa143ade8, mode=1) at syncprov.c:825 > #8 0x0815df12 in syncprov_qplay (op=0xa143ae38, rtask=0x9b303110) at > syncprov.c:896 > #9 0x0815e1a7 in syncprov_qtask (ctx=0xa143b1f0, arg=0x9b303110) at > syncprov.c:959 > #10 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 > #11 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 > #12 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 -- -- Howard Chu CTO, 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#6033) Segmentation fault - somewhere below syncprov_matchops / test_filter
by oskar＠deckle.co.za 25 Mar '09

25 Mar '09

Hi Further to this - please note that we know the dbconfig values in the config files are tiny - we use a DB_CONFIG file to override those, created before the ldap database is initialised. On master: set_cachesize 0 268435456 1 set_lg_regionmax 262144 set_lg_bsize 2097152 set_lk_max_objects 5000 set_lk_max_locks 5000 set_lk_max_lockers 5000 On clients: set_cachesize 0 16777216 0 set_lk_max_objects 1500 set_lk_max_locks 1500 set_lk_max_lockers 1500 Oskar

1 0

Re: (ITS#6034) Assert failure - connection_state_closing / send_ldap_ber / slap_send_search_entry / syncprov_sendresp
by oskar＠deckle.co.za 25 Mar '09

25 Mar '09

Hi Further to this - please note that we know the dbconfig values in the config files are tiny - we use a DB_CONFIG file to override those, created before the ldap database is initialised. On master: set_cachesize 0 268435456 1 set_lg_regionmax 262144 set_lg_bsize 2097152 set_lk_max_objects 5000 set_lk_max_locks 5000 set_lk_max_lockers 5000 On clients: set_cachesize 0 16777216 0 set_lk_max_objects 1500 set_lk_max_locks 1500 set_lk_max_lockers 1500 Oskar

1 0

(ITS#6034) Assert failure - connection_state_closing / send_ldap_ber / slap_send_search_entry / syncprov_sendresp
by oskar＠deckle.co.za 25 Mar '09

25 Mar '09

Full_Name: Oskar Pearson Version: 2.4.15 OS: Ubuntu hardy URL: ftp://ftp.openldap.org/incoming/slapd-configs-qualica-1.tar.gz Submission from: (NULL) (94.194.140.71) This is the same machine as ITS #6033. Details included here for easy reference: Openldap 2.4.15 (with no patches) was self-compiled with no patches (CFLAGS="-ggdb3 -O0" ./configure), linked against standard OS libraries (ubuntu hardy) for BDB etc. BDB is 4.6.21-6ubuntu1. Kernel is 2.6.24-19-server, libc is libc6-i686/2.7-10ubuntu3. I do not have a debug log of this crash as it occurred on a live server, which is too busy for that level of logging. I also do not have a replaceable example. I have core dumps and can print out whatever is necessary there - I appreciate your time and help. If I've missed some important info, please let me know and I'll rectify asap. System overview: We have a master server which replicates to > 100 machines for Samba purposes across ADSL lines with a VPN. These lines are quite busy, so replication writes may be slow across the network, which may have implications for locks. Config files are included at the url below. gdb --core=../core.29266 --directory=servers/slapd/ --directory=servers/slapd/overlays servers/slapd/slapd (gdb) bt #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7ad5085 in raise () from /lib/tls/i686/cmov/libc.so.6 #2 0xb7ad6a01 in abort () from /lib/tls/i686/cmov/libc.so.6 #3 0xb7ace10e in __assert_fail () from /lib/tls/i686/cmov/libc.so.6 #4 0x0806d6c6 in connection_state_closing (c=0xb79c9644) at connection.c:686 #5 0x08081bdd in send_ldap_ber (conn=0xb79c9644, ber=0xa143ab8c) at result.c:147 #6 0x08085b20 in slap_send_search_entry (op=0xa143ae38, rs=0xa143acdc) at result.c:1246 #7 0x0815dbcb in syncprov_sendresp (op=0xa143ae38, opc=0xa143adb8, so=0x9ab66dc8, e=0xa143ade8, mode=1) at syncprov.c:825 #8 0x0815df12 in syncprov_qplay (op=0xa143ae38, rtask=0x9b303110) at syncprov.c:896 #9 0x0815e1a7 in syncprov_qtask (ctx=0xa143b1f0, arg=0x9b303110) at syncprov.c:959 #10 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #11 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #12 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 (gdb) frame 4 #4 0x0806d6c6 in connection_state_closing (c=0xb79c9644) at connection.c:686 686 assert( c->c_struct_state == SLAP_C_USED ); (gdb) p c->c_struct_state $1 = 1 # maps to SLAP_C_UNUSED (gdb) p *c $4 = {c_struct_state = 1, c_conn_state = 0, c_conn_idx = 107, c_sd = -1, c_close_reason = 0x81b647b "?", c_mutex = {__data = {__lock = 0, __count = 0, __owner = 0, __kind = 0, __nusers = 0, {__spins = 0, __list = {__next = 0x0}}}, __size = '\0' <repeats 23 times>, __align = 0}, c_sb = 0x838c1b8, c_starttime = 0, c_activitytime = 0, c_connid = 4294967295, c_peer_domain = {bv_len = 0, bv_val = 0x0}, c_peer_name = {bv_len = 0, bv_val = 0x0}, c_listener = 0x0, c_sasl_bind_mech = {bv_len = 0, bv_val = 0x0}, c_sasl_dn = {bv_len = 0, bv_val = 0x0}, c_sasl_authz_dn = {bv_len = 0, bv_val = 0x0}, c_authz_backend = 0x0, c_authz_cookie = 0x0, c_authz = {sai_method = 128, sai_mech = {bv_len = 0, bv_val = 0x0}, sai_dn = {bv_len = 0, bv_val = 0x0}, sai_ndn = {bv_len = 0, bv_val = 0x0}, sai_ssf = 0, sai_transport_ssf = 0, sai_tls_ssf = 0, sai_sasl_ssf = 0}, c_protocol = 0, c_ops = { stqh_first = 0x0, stqh_last = 0xb79c96e4}, c_pending_ops = {stqh_first = 0x0, stqh_last = 0xb79c96ec}, c_write1_mutex = {__data = {__lock = 1, __count = 0, __owner = 29297, __kind = 0, __nusers = 1, {__spins = 0, __list = {__next = 0x0}}}, __size = "\001\000\000\000\000\000\000\000qr\000\000\000\000\000\000\001\000\000\000\000\000\000", __align = 1}, c_write1_cv = {__data = {__lock = 0, __futex = 0, __total_seq = 0, __wakeup_seq = 0, __woken_seq = 0, __mutex = 0x0, __nwaiters = 0, __broadcast_seq = 0}, __size = '\0' <repeats 47 times>, __align = 0}, c_write2_mutex = {__data = {__lock = 0, __count = 0, __owner = 0, __kind = 0, __nusers = 0, { __spins = 0, __list = {__next = 0x0}}}, __size = '\0' <repeats 23 times>, __align = 0}, c_write2_cv = {__data = {__lock = 0, __futex = 0, __total_seq = 0, __wakeup_seq = 0, __woken_seq = 0, __mutex = 0x0, __nwaiters = 0, __broadcast_seq = 0}, __size = '\0' <repeats 47 times>, __align = 0}, c_currentber = 0x0, c_writers = 0, c_sasl_bind_in_progress = 0 '\0', c_writewaiter = 0 '\0', c_is_tls = 0 '\0', c_needs_tls_accept = 0 '\0', c_sasl_layers = 0 '\0', c_sasl_done = 0 '\0', c_sasl_authctx = 0x0, c_sasl_sockctx = 0x0, c_sasl_extra = 0x0, c_sasl_bindop = 0x0, c_pagedresults_state = {ps_be = 0x0, ps_size = 0, ps_count = 0, ps_cookie = 0, ps_cookieval = {bv_len = 0, bv_val = 0x0}}, c_n_ops_received = 2, c_n_ops_executing = 0, c_n_ops_pending = 0, c_n_ops_completed = 2, c_n_get = 3, c_n_read = 2, c_n_write = 0, c_extensions = 0x0, c_clientfunc = 0, c_clientarg = 0x0, c_send_ldap_result = 0x8082e44 <slap_send_ldap_result>, c_send_search_entry = 0x8083b12 <slap_send_search_entry>, c_send_search_reference = 0x8085e4b <slap_send_search_reference>, c_send_ldap_extended = 0x8083676 <slap_send_ldap_extended>, c_send_ldap_intermediate = 0x80838fe <slap_send_ldap_intermediate>} frame 6 is the first location I have "op" access, so I then went (gdb) frame 6 (gdb) p *op $10 = {o_hdr = 0xa143af10, o_tag = 99, o_time = 1236686741, o_tincr = 0, o_bd = 0xa143b0b8, o_req_dn = {bv_len = 18, bv_val = 0x9dcd15ee "dc=HIDDEN-gw,dc=com"}, o_req_ndn = {bv_len = 18, bv_val = 0x9dcd1601 "dc=HIDDEN-gw,dc=com"}, o_request = {oq_add = { rs_modlist = 0x2, rs_e = 0x0}, oq_bind = {rb_method = 2, rb_cred = {bv_len = 0, bv_val = 0xffffffff <Address 0xffffffff out of bounds>}, rb_edn = { bv_len = 4294967295, bv_val = 0x0}, rb_ssf = 0, rb_mech = {bv_len = 2647463064, bv_val = 0x9dc1bbe8 "\207"}}, oq_compare = {rs_ava = 0x2}, oq_modify = {rs_mods = {rs_modlist = 0x2, rs_no_opattrs = 0 '\0'}, rs_increment = -1}, oq_modrdn = {rs_mods = {rs_modlist = 0x2, rs_no_opattrs = 0 '\0'}, rs_deleteoldrdn = -1, rs_newrdn = {bv_len = 4294967295, bv_val = 0x0}, rs_nnewrdn = {bv_len = 0, bv_val = 0x9dcd1498 "\t"}, rs_newSup = 0x9dc1bbe8, rs_nnewSup = 0xf}, oq_search = {rs_scope = 2, rs_deref = 0, rs_slimit = -1, rs_tlimit = -1, rs_limit = 0x0, rs_attrsonly = 0, rs_attrs = 0x9dcd1498, rs_filter = 0x9dc1bbe8, rs_filterstr = {bv_len = 15, bv_val = 0x9dcd1614 "(objectClass=*)"}}, oq_abandon = {rs_msgid = 2}, oq_cancel = {rs_msgid = 2}, oq_extended = {rs_reqoid = {bv_len = 2, bv_val = 0x0}, rs_flags = -1, rs_reqdata = 0xffffffff}, oq_pwdexop = {rs_extended = {rs_reqoid = {bv_len = 2, bv_val = 0x0}, rs_flags = -1, rs_reqdata = 0xffffffff}, rs_old = {bv_len = 0, bv_val = 0x0}, rs_new = {bv_len = 2647463064, bv_val = 0x9dc1bbe8 "\207"}, rs_mods = 0xf, rs_modtail = 0x9dcd1614}}, o_abandon = 0, o_cancel = 0, o_groups = 0x0, o_do_not_cache = 1 '\001', o_is_auth_check = 0 '\0', o_dont_replicate = 0 '\0', o_acl_priv = ACL_NONE, o_nocaching = 0 '\0', o_delete_glue_parent = 0 '\0', o_no_schema_check = 0 '\0', o_no_subordinate_glue = 0 '\0', o_ctrlflag = '\0' <repeats 31 times>, o_controls = 0xa143b038, o_authz = {sai_method = 128, sai_mech = {bv_len = 0, bv_val = 0x0}, sai_dn = {bv_len = 27, bv_val = 0x9dcd15d2 "cn=admin,dc=HIDDEN-gw,dc=com"}, sai_ndn = {bv_len = 27, bv_val = 0x9dcd15d2 "cn=admin,dc=HIDDEN-gw,dc=com"}, sai_ssf = 0, sai_transport_ssf = 0, sai_tls_ssf = 0, sai_sasl_ssf = 0}, o_ber = 0x0, o_res_ber = 0x0, o_callback = 0x0, o_ctrls = 0x0, o_csn = {bv_len = 0, bv_val = 0x0}, o_private = 0x0, o_extra = {slh_first = 0x0}, o_next = { stqe_next = 0x0}} Full backtrace: Thread 10 (process 29266): #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7db2775 in pthread_join () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817ba95 in ldap_pvt_thread_join (thread=2730757008, thread_return=0x0) at thr_posix.c:197 #3 0x0806ba57 in slapd_daemon () at daemon.c:2665 #4 0x0804e8c7 in main (argc=1, argv=0xbfe1d2e4) at main.c:948 Thread 9 (process 29268): #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7b81676 in epoll_wait () from /lib/tls/i686/cmov/libc.so.6 #2 0x0806abee in slapd_daemon_task (ptr=0x0) at daemon.c:2291 #3 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #4 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 8 (process 29269): #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7db5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817bb2a in ldap_pvt_thread_cond_wait (cond=0xb79ccf88, mutex=0xb79ccf70) at thr_posix.c:277 #3 0x08081eac in send_ldap_ber (conn=0xb79cce78, ber=0xa203db8c) at result.c:217 #4 0x08085b20 in slap_send_search_entry (op=0xa203de38, rs=0xa203dcdc) at result.c:1246 #5 0x0815dbcb in syncprov_sendresp (op=0xa203de38, opc=0xa203ddb8, so=0x9dc65f10, e=0xa203dde8, mode=1) at syncprov.c:825 #6 0x0815df12 in syncprov_qplay (op=0xa203de38, rtask=0x9b30c640) at syncprov.c:896 #7 0x0815e1a7 in syncprov_qtask (ctx=0xa203e1f0, arg=0x9b30c640) at syncprov.c:959 #8 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #9 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #10 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 7 (process 29295): #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7db5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817bb2a in ldap_pvt_thread_cond_wait (cond=0xb79ce090, mutex=0xb79ce078) at thr_posix.c:277 #3 0x08081eac in send_ldap_ber (conn=0xb79cdf80, ber=0xa1c3cb8c) at result.c:217 #4 0x08085b20 in slap_send_search_entry (op=0xa1c3ce38, rs=0xa1c3ccdc) at result.c:1246 #5 0x0815dbcb in syncprov_sendresp (op=0xa1c3ce38, opc=0xa1c3cdb8, so=0x9ab8ecd8, e=0xa1c3cde8, mode=1) at syncprov.c:825 #6 0x0815df12 in syncprov_qplay (op=0xa1c3ce38, rtask=0x9af477a8) at syncprov.c:896 #7 0x0815e1a7 in syncprov_qtask (ctx=0xa1c3d1f0, arg=0x9af477a8) at syncprov.c:959 #8 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #9 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #10 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 6 (process 29296): #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7db5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817bb2a in ldap_pvt_thread_cond_wait (cond=0xb79cf500, mutex=0xb79cf4e8) at thr_posix.c:277 #3 0x08081eac in send_ldap_ber (conn=0xb79cf3f0, ber=0xa183bb8c) at result.c:217 #4 0x08085b20 in slap_send_search_entry (op=0xa183be38, rs=0xa183bcdc) at result.c:1246 #5 0x0815dbcb in syncprov_sendresp (op=0xa183be38, opc=0xa183bdb8, so=0xa0ae45c8, e=0xa183bde8, mode=1) at syncprov.c:825 #6 0x0815df12 in syncprov_qplay (op=0xa183be38, rtask=0x9ab5e400) at syncprov.c:896 #7 0x0815e1a7 in syncprov_qtask (ctx=0xa183c1f0, arg=0x9ab5e400) at syncprov.c:959 #8 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #9 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #10 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 5 (process 29298): #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7db5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817bb2a in ldap_pvt_thread_cond_wait (cond=0xb79c8d1c, mutex=0xb79c8d04) at thr_posix.c:277 #3 0x08081eac in send_ldap_ber (conn=0xb79c8c0c, ber=0xa1039b8c) at result.c:217 #4 0x08085b20 in slap_send_search_entry (op=0xa1039e38, rs=0xa1039cdc) at result.c:1246 #5 0x0815dbcb in syncprov_sendresp (op=0xa1039e38, opc=0xa1039db8, so=0x9dc0bed0, e=0xa1039de8, mode=2) at syncprov.c:825 #6 0x0815df12 in syncprov_qplay (op=0xa1039e38, rtask=0x83781c8) at syncprov.c:896 #7 0x0815e1a7 in syncprov_qtask (ctx=0xa103a1f0, arg=0x83781c8) at syncprov.c:959 #8 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #9 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #10 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 4 (process 29300): #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7db5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817bb2a in ldap_pvt_thread_cond_wait (cond=0xb79cefe4, mutex=0xb79cefcc) at thr_posix.c:277 #3 0x08081eac in send_ldap_ber (conn=0xb79ceed4, ber=0x9dbfeb8c) at result.c:217 #4 0x08085b20 in slap_send_search_entry (op=0x9dbfee38, rs=0x9dbfecdc) at result.c:1246 #5 0x0815dbcb in syncprov_sendresp (op=0x9dbfee38, opc=0x9dbfedb8, so=0x9ab63788, e=0x9dbfede8, mode=1) at syncprov.c:825 #6 0x0815df12 in syncprov_qplay (op=0x9dbfee38, rtask=0xa0a03b60) at syncprov.c:896 #7 0x0815e1a7 in syncprov_qtask (ctx=0x9dbff1f0, arg=0xa0a03b60) at syncprov.c:959 #8 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #9 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #10 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 3 (process 29301): #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7db5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817bb2a in ldap_pvt_thread_cond_wait (cond=0xb79bfaa4, mutex=0xb79bfa8c) at thr_posix.c:277 #3 0x08081eac in send_ldap_ber (conn=0xb79bf994, ber=0x9cefbb8c) at result.c:217 #4 0x08085b20 in slap_send_search_entry (op=0x9cefbe38, rs=0x9cefbcdc) at result.c:1246 #5 0x0815dbcb in syncprov_sendresp (op=0x9cefbe38, opc=0x9cefbdb8, so=0x9af1df00, e=0x9cefbde8, mode=1) at syncprov.c:825 #6 0x0815df12 in syncprov_qplay (op=0x9cefbe38, rtask=0xa0a33d90) at syncprov.c:896 #7 0x0815e1a7 in syncprov_qtask (ctx=0x9cefc1f0, arg=0xa0a33d90) at syncprov.c:959 #8 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #9 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #10 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 2 (process 29302): #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7db5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817bb2a in ldap_pvt_thread_cond_wait (cond=0xb79cb0e0, mutex=0xb79cb0c8) at thr_posix.c:277 #3 0x08081eac in send_ldap_ber (conn=0xb79cafd0, ber=0x9cafab8c) at result.c:217 #4 0x08085b20 in slap_send_search_entry (op=0x9cafae38, rs=0x9cafacdc) at result.c:1246 #5 0x0815dbcb in syncprov_sendresp (op=0x9cafae38, opc=0x9cafadb8, so=0x9af21c10, e=0x9cafade8, mode=1) at syncprov.c:825 #6 0x0815df12 in syncprov_qplay (op=0x9cafae38, rtask=0xa0a31988) at syncprov.c:896 #7 0x0815e1a7 in syncprov_qtask (ctx=0x9cafb1f0, arg=0xa0a31988) at syncprov.c:959 #8 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #9 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #10 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 1 (process 29297): #0 0xb7ef8410 in __kernel_vsyscall () #1 0xb7ad5085 in raise () from /lib/tls/i686/cmov/libc.so.6 #2 0xb7ad6a01 in abort () from /lib/tls/i686/cmov/libc.so.6 #3 0xb7ace10e in __assert_fail () from /lib/tls/i686/cmov/libc.so.6 #4 0x0806d6c6 in connection_state_closing (c=0xb79c9644) at connection.c:686 #5 0x08081bdd in send_ldap_ber (conn=0xb79c9644, ber=0xa143ab8c) at result.c:147 #6 0x08085b20 in slap_send_search_entry (op=0xa143ae38, rs=0xa143acdc) at result.c:1246 #7 0x0815dbcb in syncprov_sendresp (op=0xa143ae38, opc=0xa143adb8, so=0x9ab66dc8, e=0xa143ade8, mode=1) at syncprov.c:825 #8 0x0815df12 in syncprov_qplay (op=0xa143ae38, rtask=0x9b303110) at syncprov.c:896 #9 0x0815e1a7 in syncprov_qtask (ctx=0xa143b1f0, arg=0x9b303110) at syncprov.c:959 #10 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #11 0xb7db14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #12 0xb7b80e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thanks for your time! Oskar

1 0

(ITS#6033) Segmentation fault - somewhere below syncprov_matchops / test_filter
by oskar＠deckle.co.za 25 Mar '09

25 Mar '09

Full_Name: Oskar Pearson Version: 2.4.15 OS: Ubuntu hardy URL: ftp://ftp.openldap.org/incoming/slapd-configs-qualica-1.tar.gz Submission from: (NULL) (94.194.140.71) Openldap 2.4.15 (with no patches) was self-compiled with no patches (CFLAGS="-ggdb3 -O0" ./configure), linked against standard OS libraries (ubuntu hardy) for BDB etc. BDB is 4.6.21-6ubuntu1. Kernel is 2.6.24-19-server, libc is libc6-i686/2.7-10ubuntu3. I do not have a debug log of this crash as it occurred on a live server, which is too busy for that level of logging. I also do not have a replaceable example. I have core dumps and can print out whatever is necessary there - I appreciate your time and help. If I've missed some important info, please let me know and I'll rectify asap. System overview: We have a master server which replicates to > 100 machines for Samba purposes across ADSL lines with a VPN. These lines are quite busy, so replication writes may be slow across the network, which may have implications for locks. Config files are included at the url below. The specific crash was in thread 1. Other threads are included in a full backtrace below. gdb --core=../core.24122 --directory=servers/slapd/ --directory=servers/slapd/overlays servers/slapd/slapd #0 0x080c0e82 in is_ad_subtype (sub=0x0, super=0x8294e08) at ad.c:502 502 for ( a = sub->ad_type; a; a=a->sat_sup ) { Thread 1 (process 24208): #0 0x080c0e82 in is_ad_subtype (sub=0x0, super=0x8294e08) at ad.c:502 #1 0x0807a6da in attrs_find (a=0xa2d39fa4, desc=0x8294e08) at attr.c:645 #2 0x080964dd in test_presence_filter (op=0x9e6f1b08, e=0x83277f4, desc=0x8294e08) at filterentry.c:848 #3 0x080951ce in test_filter (op=0x9e6f1b08, e=0x83277f4, f=0x9ae022a8) at filterentry.c:108 #4 0x0815efdf in syncprov_matchops (op=0x83455c8, opc=0x9cfed214, saveit=1) at syncprov.c:1251 #5 0x08160fd9 in syncprov_op_mod (op=0x83455c8, rs=0x9e6f3100) at syncprov.c:1905 #6 0x080f1396 in overlay_op_walk (op=0x83455c8, rs=0x9e6f3100, which=op_modify, oi=0x82d19c8, on=0x82d1ac8) at backover.c:659 #7 0x080f161e in over_op_func (op=0x83455c8, rs=0x9e6f3100, which=op_modify) at backover.c:721 #8 0x080f1706 in over_op_modify (op=0x83455c8, rs=0x9e6f3100) at backover.c:755 #9 0x0808c802 in fe_op_modify (op=0x83455c8, rs=0x9e6f3100) at modify.c:301 #10 0x0808c115 in do_modify (op=0x83455c8, rs=0x9e6f3100) at modify.c:175 #11 0x0806e7c2 in connection_operation (ctx=0x9e6f31f0, arg_v=0x83455c8) at connection.c:1097 #12 0x0806ec9c in connection_read_thread (ctx=0x9e6f31f0, argv=0x39) at connection.c:1223 #13 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #14 0xb7da14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #15 0xb7b70e5e in clone () from /lib/tls/i686/cmov/libc.so.6 (gdb) p sub $15 = (AttributeDescription *) 0x0 (gdb) up #1 0x0807a6da in attrs_find (a=0xa2d39fa4, desc=0x8294e08) at attr.c:645 645 if ( is_ad_subtype( a->a_desc, desc ) ) { (gdb) p *a $24 = {a_desc = 0x0, a_vals = 0x0, a_nvals = 0x0, a_numvals = 0, a_flags = 0, a_next = 0xa2d39d1c} (gdb) p *desc $32 = {ad_next = 0x0, ad_type = 0x8294d30, ad_cname = {bv_len = 11, bv_val = 0x8294ca8 "objectClass"}, ad_tags = {bv_len = 0, bv_val = 0x0}, ad_flags = 0} (gdb) p *op $33 = {o_hdr = 0x9e6f18e8, o_tag = 99, o_time = 1236840350, o_tincr = 0, o_bd = 0x82b5af8, o_req_dn = {bv_len = 18, bv_val = 0x9ae0360e "dc=HIDDEN-gw,dc=com"}, o_req_ndn = {bv_len = 18, bv_val = 0x9ae03621 "dc=HIDDEN-gw,dc=com"}, o_request = {oq_add = { rs_modlist = 0x2, rs_e = 0x0}, oq_bind = {rb_method = 2, rb_cred = {bv_len = 0, bv_val = 0xffffffff <Address 0xffffffff out of bounds>}, rb_edn = { bv_len = 4294967295, bv_val = 0x0}, rb_ssf = 0, rb_mech = {bv_len = 2598384824, bv_val = 0x9ae022a8 "\207"}}, oq_compare = {rs_ava = 0x2}, oq_modify = {rs_mods = {rs_modlist = 0x2, rs_no_opattrs = 0 '\0'}, rs_increment = -1}, oq_modrdn = {rs_mods = {rs_modlist = 0x2, rs_no_opattrs = 0 '\0'}, rs_deleteoldrdn = -1, rs_newrdn = {bv_len = 4294967295, bv_val = 0x0}, rs_nnewrdn = {bv_len = 0, bv_val = 0x9ae034b8 "\t"}, rs_newSup = 0x9ae022a8, rs_nnewSup = 0xf}, oq_search = {rs_scope = 2, rs_deref = 0, rs_slimit = -1, rs_tlimit = -1, rs_limit = 0x0, rs_attrsonly = 0, rs_attrs = 0x9ae034b8, rs_filter = 0x9ae022a8, rs_filterstr = {bv_len = 15, bv_val = 0x9ae03634 "(objectClass=*)"}}, oq_abandon = {rs_msgid = 2}, oq_cancel = {rs_msgid = 2}, oq_extended = {rs_reqoid = {bv_len = 2, bv_val = 0x0}, rs_flags = -1, rs_reqdata = 0xffffffff}, oq_pwdexop = {rs_extended = {rs_reqoid = {bv_len = 2, bv_val = 0x0}, rs_flags = -1, rs_reqdata = 0xffffffff}, rs_old = {bv_len = 0, bv_val = 0x0}, rs_new = {bv_len = 2598384824, bv_val = 0x9ae022a8 "\207"}, rs_mods = 0xf, rs_modtail = 0x9ae03634}}, o_abandon = 0, o_cancel = 0, o_groups = 0x0, o_do_not_cache = 1 '\001', o_is_auth_check = 0 '\0', o_dont_replicate = 0 '\0', o_acl_priv = ACL_NONE, o_nocaching = 0 '\0', o_delete_glue_parent = 0 '\0', o_no_schema_check = 0 '\0', o_no_subordinate_glue = 0 '\0', o_ctrlflag = '\0' <repeats 31 times>, o_controls = 0x0, o_authz = {sai_method = 128, sai_mech = {bv_len = 0, bv_val = 0x0}, sai_dn = {bv_len = 27, bv_val = 0x9ae035f2 "cn=admin,dc=HIDDEN-gw,dc=com"}, sai_ndn = {bv_len = 27, bv_val = 0x9ae035f2 "cn=admin,dc=HIDDEN-gw,dc=com"}, sai_ssf = 0, sai_transport_ssf = 0, sai_tls_ssf = 0, sai_sasl_ssf = 0}, o_ber = 0x0, o_res_ber = 0x0, o_callback = 0x0, o_ctrls = 0x0, o_csn = {bv_len = 0, bv_val = 0x0}, o_private = 0x0, o_extra = {slh_first = 0x9cfed24c}, o_next = { stqe_next = 0x0}} (gdb) p *e $34 = {e_id = 0, e_name = {bv_len = 39, bv_val = 0x0}, e_nname = {bv_len = 39, bv_val = 0x0}, e_attrs = 0x0, e_ocflags = 0, e_bv = {bv_len = 0, bv_val = 0x0}, e_private = 0x83277a4} (gdb) p *desc $35 = {ad_next = 0x0, ad_type = 0x8294d30, ad_cname = {bv_len = 11, bv_val = 0x8294ca8 "objectClass"}, ad_tags = {bv_len = 0, bv_val = 0x0}, ad_flags = 0} --- full backtrace follows: thread apply all bt Thread 10 (process 24122): #0 0xb7ee8410 in __kernel_vsyscall () #1 0xb7da2775 in pthread_join () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817ba95 in ldap_pvt_thread_join (thread=2730691472, thread_return=0x0) at thr_posix.c:197 #3 0x0806ba57 in slapd_daemon () at daemon.c:2665 #4 0x0804e8c7 in main (argc=1, argv=0xbffcccc4) at main.c:948 Thread 9 (process 24127): #0 0xb7ee8410 in __kernel_vsyscall () #1 0xb7b71676 in epoll_wait () from /lib/tls/i686/cmov/libc.so.6 #2 0x0806abee in slapd_daemon_task (ptr=0x0) at daemon.c:2291 #3 0xb7da14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #4 0xb7b70e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 8 (process 24128): #0 0xb7ee8410 in __kernel_vsyscall () #1 0xb7da5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817bb2a in ldap_pvt_thread_cond_wait (cond=0xb79af8f0, mutex=0xb79af8d8) at thr_posix.c:277 #3 0x08081eac in send_ldap_ber (conn=0xb79af7e0, ber=0xa202db8c) at result.c:217 #4 0x08085b20 in slap_send_search_entry (op=0xa202de38, rs=0xa202dcdc) at result.c:1246 #5 0x0815dbcb in syncprov_sendresp (op=0xa202de38, opc=0xa202ddb8, so=0xa1bb3490, e=0xa202dde8, mode=2) at syncprov.c:825 #6 0x0815df12 in syncprov_qplay (op=0xa202de38, rtask=0x9ae030b8) at syncprov.c:896 #7 0x0815e1a7 in syncprov_qtask (ctx=0xa202e1f0, arg=0x9ae030b8) at syncprov.c:959 #8 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #9 0xb7da14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #10 0xb7b70e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 7 (process 24169): #0 0xb7ee8410 in __kernel_vsyscall () #1 0xb7da5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817bb2a in ldap_pvt_thread_cond_wait (cond=0xb79b6958, mutex=0xb79b6940) at thr_posix.c:277 #3 0x08081eac in send_ldap_ber (conn=0xb79b6848, ber=0xa1afeb8c) at result.c:217 #4 0x08085b20 in slap_send_search_entry (op=0xa1afee38, rs=0xa1afecdc) at result.c:1246 #5 0x0815dbcb in syncprov_sendresp (op=0xa1afee38, opc=0xa1afedb8, so=0x8a0deb0, e=0xa1afede8, mode=2) at syncprov.c:825 #6 0x0815df12 in syncprov_qplay (op=0xa1afee38, rtask=0x8a12e20) at syncprov.c:896 #7 0x0815e1a7 in syncprov_qtask (ctx=0xa1aff1f0, arg=0x8a12e20) at syncprov.c:959 #8 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #9 0xb7da14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #10 0xb7b70e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 3 (process 24209): #0 0xb7ee8410 in __kernel_vsyscall () #1 0xb7da5aa5 in pthread_cond_wait@@GLIBC_2.3.2 () from /lib/tls/i686/cmov/libpthread.so.0 #2 0x0817bb2a in ldap_pvt_thread_cond_wait (cond=0xb79b28a0, mutex=0xb79b2888) at thr_posix.c:277 #3 0x08081eac in send_ldap_ber (conn=0xb79b2790, ber=0x9d9efb8c) at result.c:217 #4 0x08085b20 in slap_send_search_entry (op=0x9d9efe38, rs=0x9d9efcdc) at result.c:1246 #5 0x0815dbcb in syncprov_sendresp (op=0x9d9efe38, opc=0x9d9efdb8, so=0x8a01da0, e=0x9d9efde8, mode=2) at syncprov.c:825 #6 0x0815df12 in syncprov_qplay (op=0x9d9efe38, rtask=0x9ad957e0) at syncprov.c:896 #7 0x0815e1a7 in syncprov_qtask (ctx=0x9d9f01f0, arg=0x9ad957e0) at syncprov.c:959 #8 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #9 0xb7da14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #10 0xb7b70e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 2 (process 24210): #0 0xb7b0d9bc in memcpy () from /lib/tls/i686/cmov/libc.so.6 #1 0xb7e7e6cd in __db_retcopy () from /usr/lib/libdb-4.6.so #2 0xb7e7e80b in __db_ret () from /usr/lib/libdb-4.6.so #3 0xb7e6126b in __dbc_get () from /usr/lib/libdb-4.6.so #4 0xb7e6a04b in __dbc_get_pp () from /usr/lib/libdb-4.6.so #5 0x0814fd28 in hdb_id2entry (be=0x9d5edefc, tid=0xa1b11180, id=2355, e=0x9d52d958) at id2entry.c:151 #6 0x081431e9 in hdb_cache_find_id (op=0x9d5edc24, tid=0xa1b11180, id=2355, eip=0x9d5edaf8, flag=2, lock=0x9d5eda90) at cache.c:923 #7 0x081158a5 in hdb_search (op=0x9d5edc24, rs=0x9d5edb78) at search.c:705 #8 0x08160364 in syncprov_playlog (op=0xa1b0a300, rs=0x9d5ef100, sl=0x82d1c50, srs=0x9c6eb264, ctxcsn=0x9c6eb31c, numcsns=1, sids=0x9c6eb364) at syncprov.c:1592 #9 0x08162893 in syncprov_op_search (op=0xa1b0a300, rs=0x9d5ef100) at syncprov.c:2348 #10 0x080f1396 in overlay_op_walk (op=0xa1b0a300, rs=0x9d5ef100, which=op_search, oi=0x82d19c8, on=0x82d1ac8) at backover.c:659 #11 0x080f161e in over_op_func (op=0xa1b0a300, rs=0x9d5ef100, which=op_search) at backover.c:721 #12 0x080f16c2 in over_op_search (op=0xa1b0a300, rs=0x9d5ef100) at backover.c:743 #13 0x08072169 in fe_op_search (op=0xa1b0a300, rs=0x9d5ef100) at search.c:366 #14 0x08071adc in do_search (op=0xa1b0a300, rs=0x9d5ef100) at search.c:217 #15 0x0806e7c2 in connection_operation (ctx=0x9d5ef1f0, arg_v=0xa1b0a300) at connection.c:1097 #16 0x0806ec9c in connection_read_thread (ctx=0x9d5ef1f0, argv=0x1d) at connection.c:1223 #17 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #18 0xb7da14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #19 0xb7b70e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thread 1 (process 24208): #0 0x080c0e82 in is_ad_subtype (sub=0x0, super=0x8294e08) at ad.c:502 #1 0x0807a6da in attrs_find (a=0xa2d39fa4, desc=0x8294e08) at attr.c:645 #2 0x080964dd in test_presence_filter (op=0x9e6f1b08, e=0x83277f4, desc=0x8294e08) at filterentry.c:848 #3 0x080951ce in test_filter (op=0x9e6f1b08, e=0x83277f4, f=0x9ae022a8) at filterentry.c:108 #4 0x0815efdf in syncprov_matchops (op=0x83455c8, opc=0x9cfed214, saveit=1) at syncprov.c:1251 #5 0x08160fd9 in syncprov_op_mod (op=0x83455c8, rs=0x9e6f3100) at syncprov.c:1905 #6 0x080f1396 in overlay_op_walk (op=0x83455c8, rs=0x9e6f3100, which=op_modify, oi=0x82d19c8, on=0x82d1ac8) at backover.c:659 #7 0x080f161e in over_op_func (op=0x83455c8, rs=0x9e6f3100, which=op_modify) at backover.c:721 #8 0x080f1706 in over_op_modify (op=0x83455c8, rs=0x9e6f3100) at backover.c:755 #9 0x0808c802 in fe_op_modify (op=0x83455c8, rs=0x9e6f3100) at modify.c:301 #10 0x0808c115 in do_modify (op=0x83455c8, rs=0x9e6f3100) at modify.c:175 #11 0x0806e7c2 in connection_operation (ctx=0x9e6f31f0, arg_v=0x83455c8) at connection.c:1097 #12 0x0806ec9c in connection_read_thread (ctx=0x9e6f31f0, argv=0x39) at connection.c:1223 #13 0x0817a99b in ldap_int_thread_pool_wrapper (xpool=0x829baf0) at tpool.c:663 #14 0xb7da14fb in start_thread () from /lib/tls/i686/cmov/libpthread.so.0 #15 0xb7b70e5e in clone () from /lib/tls/i686/cmov/libc.so.6 Thanks for your time, Oskar

1 0

Re: (ITS#6032) lap_add: Invalid DN syntax (34) additional info: invalid DN
by quanah＠zimbra.com 25 Mar '09

25 Mar '09

--On Wednesday, March 25, 2009 11:52 AM +0000 sudeshchandimal(a)gmail.com wrote: > Full_Name: sudesh chandimal > Version: openldap-2.4.15 > OS: fedora > URL: ftp://ftp.openldap.org/incoming/ > Submission from: (NULL) (124.43.51.7) > > > lap_add: Invalid DN syntax (34) > additional info: invalid DN > > this is the error i'm getting after runing this ldif file.. > ># Base object for Request Balance > dn: dview=ECS1_request_bal, dc=my-domain, dc=com > dview: ECS1_request_bal > objectClass: top > objectClass: dview > userPassword: balance Send usage questions to openldap-software(a)openldap.org, not the ITS system. The ITS system is for reporting bugs in the software. --Quanah -- Quanah Gibson-Mount Principal Software Engineer Zimbra, Inc -------------------- Zimbra :: the leader in open source messaging and collaboration

1 0

(ITS#6032) lap_add: Invalid DN syntax (34) additional info: invalid DN
by sudeshchandimal＠gmail.com 25 Mar '09

25 Mar '09

Full_Name: sudesh chandimal Version: openldap-2.4.15 OS: fedora URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (124.43.51.7) lap_add: Invalid DN syntax (34) additional info: invalid DN this is the error i'm getting after runing this ldif file.. # Base object for Request Balance dn: dview=ECS1_request_bal, dc=my-domain, dc=com dview: ECS1_request_bal objectClass: top objectClass: dview userPassword: balance ldif file.............. ldapadd -x -D "cn=Manager, dc=my-domain, dc=com" -w secret -f add_request_bal_entry.ldif please please help me.. quickly....!

1 0

(ITS#6031) An account locked on consumer is unlocked only when password is changed twice in a row on provider
by stlist＠gmail.com 24 Mar '09

24 Mar '09

Full_Name: Samuel Tran Version: 2.3.43 OS: CentOS 5.x URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (216.73.248.203) If I lock an account on a consumer 'pwdMaxFailure' consecutive failed bind attempts, two password changes on the provider is required to unlock the account on the consumer. The first password change updates 'userPassword', 'pwdChangedTime' and removes 'pwdFailureTime'. The second updates 'userPassword', 'pwdChangedTime' and removes 'pwdAccountLockedTime'. The replication mode is delta-syncrepl. Here is the configuration file on the provider: #------------------------------------------------- # Accesslog DB definition (slapo-accesslog) #------------------------------------------------- database hdb suffix "cn=accesslog" rootdn "cn=root,cn=accesslog" rootpw {SSHA}xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx directory /var/lib/ldap/accesslog index default eq index entryUUID,entryCSN,objectClass,reqEnd,reqResult,reqStart limits dn.exact="cn=syncrepl,ou=Accounts,ou=Apps,dc=example,dc=com" time.soft=unlimited time.hard=unlimited size.soft=unlimited size.hard=unlimited overlay syncprov syncprov-nopresent TRUE syncprov-reloadhint TRUE #------------------------------------------------- # Primary example.com database definition #------------------------------------------------- database hdb suffix "dc=example,dc=com" rootdn "cn=Manager,dc=example,dc=com" rootpw {SSHA}xxxxxxxxxxxxxxxxxxxxxxxxxxx directory /var/lib/ldap/example.com [snip] index objectClass,uidNumber,gidNumber,memberUid,employeeNumber eq,pres index employeeType,accountActive,ftpActive,mailActive,vacationActive,ou,mailRoutingAddress eq index cn,mail,surname,givenname eq,pres,subinitial index displayName,gecos,telephoneNumber sub,subany index uid,aliasUid eq,sub,subany index entryUUID,entryCSN eq limits dn.exact="cn=syncrepl,ou=Accounts,ou=Apps,dc=example,dc=com" time.soft=unlimited time.hard=unlimited size.soft=unlimited size.hard=unlimited overlay syncprov syncprov-checkpoint 100 30 syncprov-sessionlog 100 overlay accesslog logdb cn=accesslog logops writes logsuccess TRUE logpurge 28+00:00 01+00:00 overlay ppolicy ppolicy_use_lockout Here is the configuration file on the consumer: #------------------------------------------------- # Primary example.com database definition #------------------------------------------------- database hdb suffix "dc=example,dc=com" rootdn "cn=Manager,dc=example,dc=com" rootpw {SSHA}xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx directory /var/lib/ldap/example.com [snip] index objectClass,uidNumber,gidNumber,memberUid,employeeNumber eq,pres index employeeType,accountActive,ftpActive,mailActive,vacationActive,ou,mailRoutingAddress,mailAlternateAddress,mailAliasActive,allowedService eq index cn,mail,surname,givenname eq,pres,subinitial index displayName,gecos,telephoneNumber sub,subany index uid,aliasUid eq,sub,subany index entryUUID eq ############################################################# # Syncrepl - Consumer configuration ############################################################# syncrepl rid=002 provider=ldaps://info-ldap-001.example.com:636 bindmethod=simple binddn="cn=syncrepl,ou=Accounts,ou=Apps,dc=example,dc=com" credentials=xxxxxxxx type=refreshAndPersist retry="5 +" searchbase="dc=example,dc=com" logbase="cn=accesslog" logfilter="(&(objectClass=auditWriteObject)(reqResult=0))" schemachecking=on syncdata=accesslog overlay ppolicy ppolicy_use_lockout The problem is similar to the one reported in ITS #5398 for OL 2.4.8. I saw Howard's reply stating that he was not able to reproduce the problem in the current OL 2.4.x code. I was wondering if someone was able to reproduce the problem using OL 2.3.43. Thanks.

1 0

Re: (ITS#6030) slapd segfaults when loading already present module
by luca.scamoni＠sys-net.it 23 Mar '09

23 Mar '09

Tested latest commit and it's fixed now thanks Ing. Luca Scamoni Responsabile Ricerca e Sviluppo SysNet s.r.l. via Dossi, 8 - 27100 Pavia - ITALIA http://www.sys-net.it ----------------------------------- Office: +39 0382 573859 (137) Mobile: +39 347 1014425 Fax: +39 0382 476497 Email: luca.scamoni(a)sys-net.it -----------------------------------

1 0

Re: (ITS#6030) slapd segfaults when loading already present module
by hyc＠symas.com 23 Mar '09

23 Mar '09

Luca Scamoni wrote: > Howard Chu ha scritto: >> HEAD has been changed to silently ignore these cases, please test. >> >> (I guess we could log something at LDAP_DEBUG_CONFIG level but that >> seems unnecessary at the moment.) >> > thanks, > now it doesn't segfaults anymore but something like: > modulepath /usr/local/openldap/sbin > moduleload syncprov.la > moduleload back_hdb.la > moduleload syncprov.la > > becomes: > dn: cn=module{0},cn=config > objectClass: olcModuleList > cn: module{0} > olcModulePath: /usr/local/openldap/sbin > olcModuleLoad: {0}syncprov.la > olcModuleLoad: {1}back_hdb.la > olcModuleLoad: {2}syncprov.la > > any chance this can cause problems? It's a bit redundant. Perhaps we should fail it instead. > about logging I vote for it > > > Ing. Luca Scamoni > Responsabile Ricerca e Sviluppo > > SysNet s.r.l. > via Dossi, 8 - 27100 Pavia - ITALIA > http://www.sys-net.it > ----------------------------------- > Office: +39 0382 573859 (137) > Mobile: +39 347 1014425 > Fax: +39 0382 476497 > Email: luca.scamoni(a)sys-net.it > ----------------------------------- > > -- -- Howard Chu CTO, 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#6030) slapd segfaults when loading already present module
by luca.scamoni＠sys-net.it 23 Mar '09

23 Mar '09

Howard Chu ha scritto: > HEAD has been changed to silently ignore these cases, please test. > > (I guess we could log something at LDAP_DEBUG_CONFIG level but that > seems unnecessary at the moment.) > thanks, now it doesn't segfaults anymore but something like: modulepath /usr/local/openldap/sbin moduleload syncprov.la moduleload back_hdb.la moduleload syncprov.la becomes: dn: cn=module{0},cn=config objectClass: olcModuleList cn: module{0} olcModulePath: /usr/local/openldap/sbin olcModuleLoad: {0}syncprov.la olcModuleLoad: {1}back_hdb.la olcModuleLoad: {2}syncprov.la any chance this can cause problems? about logging I vote for it Ing. Luca Scamoni Responsabile Ricerca e Sviluppo SysNet s.r.l. via Dossi, 8 - 27100 Pavia - ITALIA http://www.sys-net.it ----------------------------------- Office: +39 0382 573859 (137) Mobile: +39 347 1014425 Fax: +39 0382 476497 Email: luca.scamoni(a)sys-net.it -----------------------------------

1 0

Re: (ITS#6030) slapd segfaults when loading already present module
by hyc＠symas.com 23 Mar '09

23 Mar '09

HEAD has been changed to silently ignore these cases, please test. (I guess we could log something at LDAP_DEBUG_CONFIG level but that seems unnecessary at the moment.) luca(a)OpenLDAP.org wrote: > Full_Name: Luca Scamoni > Version: RE24/HEAD > OS: any > URL: ftp://ftp.openldap.org/incoming/ > Submission from: (NULL) (82.63.140.131) > > > When loading a module through cn=config or slapd.conf moduleload directive, if > the module has already been loaded, slapd segfaults. Since it correctly finds > out the module is already registered shouldn't it refuse the modification when > the operation is attempted through cn=config or simply refuse to start when done > through slapd.conf? > > Last log lines (-d -1): > line 20 (moduleload syncprov.la) > loaded module syncprov.la > Control 1.3.6.1.4.1.4203.1.9.1.1 already registered. > syncprov_init: Failed to register control -9 > Segmentation fault > > backtrace: > #0 0x0000000b in ?? () > No symbol table info available. > #1 0x080dcb94 in module_load (file_name=0x89746bb "syncprov.la", argc=0, > argv=0x0) > at module.c:218 > module = (module_loaded_t *) 0x8976718 > error = 0x0 > rc = -9 > initialize = (MODULE_INIT_FN) 0x5f6936<init_module> > #2 0x0805bd1c in config_generic (c=0x8967970) at bconfig.c:1940 > i = 11784196 > __PRETTY_FUNCTION__ = "config_generic" > #3 0x0806a586 in config_set_vals (Conf=0x8168160, c=0x8967970) at config.c:314 > rc = 0 > arg_type = -2013265890 > ptr = (void *) 0x0 > #4 0x0806aa68 in config_add_vals (Conf=0x8168160, c=0x8967970) at config.c:383 > rc = 0 > arg_type = -2013265890 > #5 0x0806ba42 in read_config_file (fname=0x8932308 > "/usr/local/openldap/etc/slapd.conf", > depth=0, cf=0x0, cft=0x8167bc0) at config.c:736 > fp = (FILE *) 0x89692f8 > ct = (ConfigTable *) 0x8168160 > c = (ConfigArgs *) 0x8967970 > rc = 0 > s = {st_dev = 64768, __pad1 = 0, st_ino = 172196293, st_mode = 33152, > st_nlink = 1, > st_uid = 0, st_gid = 0, st_rdev = 0, __pad2 = 0, st_size = 1087, st_blksize = > 4096, > st_blocks = 8, st_atim = {tv_sec = 1237802811, tv_nsec = 0}, st_mtim = {tv_sec > = 1237802556, > tv_nsec = 0}, st_ctim = {tv_sec = 1237802556, tv_nsec = 0}, __unused4 = 0, > __unused5 = 0} > #6 0x08061626 in read_config (fname=0x8932308 > "/usr/local/openldap/etc/slapd.conf", dir=0x0) > at bconfig.c:3613 > be = (BackendDB *) 0x8967800 > cfb = (CfBackInfo *) 0x816c340 > cfdir = 0x11f0bc "" > cfname = 0x8932308 "/usr/local/openldap/etc/slapd.conf" > rc = 0 > __PRETTY_FUNCTION__ = "read_config" > #7 0x08057a5d in main (argc=5, argv=0xbfc1ce24) at main.c:754 > i = -1 > no_detach = 0 > rc = 0 > urls = 0x89323f0 "ldap://:9000" > username = 0x0 > groupname = 0x0 > sandbox = 0x0 > syslogUser = 160 > g_argc = 5 > g_argv = (char **) 0xbfc1ce24 > configfile = 0x8932308 "/usr/local/openldap/etc/slapd.conf" > configdir = 0x0 > serverName = 0xbfc1eb66 "slapd" > serverMode = 1 > scp = (struct sync_cookie *) 0x0 > scp_entry = (struct sync_cookie *) 0x0 > debug_unknowns = (char **) 0x0 > syslog_unknowns = (char **) 0x0 > serverNamePrefix = 0x811b537 "" > l = 1347407256 > slapd_pid_file_unlink = 0 > slapd_args_file_unlink = 0 > firstopt = 0 > __PRETTY_FUNCTION__ = "main" > > > -- -- Howard Chu CTO, Symas Corp. http://www.symas.com Director, Highland Sun http://highlandsun.com/hyc/ Chief Architect, OpenLDAP http://www.openldap.org/project/

1 0

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

openldap-bugs