October 2010 - 389-devel - Fedora mailing-lists

Please Review: (647932) multiple memberOf configuration adding memberOf where there is no member
by Nathan Kinder 29 Oct '10

29 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=647932 https://bugzilla.redhat.com/attachment.cgi?id=456564&action=edit

1 0

Please Review: (189985) Improve attribute uniqueness error message
by Nathan Kinder 28 Oct '10

28 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=189985 https://bugzilla.redhat.com/attachment.cgi?id=456328&action=edit

1 0

Please Review: (619633) attr-unique-plugin does not check for requiredObjectClass on the 'other' object
by Nathan Kinder 27 Oct '10

27 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=619633 https://bugzilla.redhat.com/attachment.cgi?id=455649&action=edit

1 0

Please Review: (619623) attr-unique-plugin ignores requiredObjectClass on modrdn operations
by Nathan Kinder 27 Oct '10

27 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=619623 https://bugzilla.redhat.com/attachment.cgi?id=456094&action=edit

1 0

Please review: [Bug 470576] Migration could do addition checks before commiting actions
by Endi Sukma Dewata 22 Oct '10

22 Oct '10

Hi, Please review the patches for the following bug: https://bugzilla.redhat.com/show_bug.cgi?id=470576 https://bugzilla.redhat.com/attachment.cgi?id=455211&action=edit https://bugzilla.redhat.com/attachment.cgi?id=455211&action=diff https://bugzilla.redhat.com/attachment.cgi?id=455212&action=edit https://bugzilla.redhat.com/attachment.cgi?id=455212&action=diff Thanks! -- Endi S. Dewata

1 0

Please review: [Bug 491733] dbtest crashes
by Noriko Hosoi 22 Oct '10

22 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=491733 https://bugzilla.redhat.com/attachment.cgi?id=455193&action=diff https://bugzilla.redhat.com/attachment.cgi?id=455193&action=edit Description: Getting rid of the dbtest.c file and its command line option in main.c. The tool is obsolete (new idl format is not supported). Files: Makefile.am ldap/servers/slapd/back-ldbm/dbtest.c ldap/servers/slapd/back-ldbm/init.c ldap/servers/slapd/back-ldbm/proto-back-ldbm.h ldap/servers/slapd/main.c

1 0

Please review: FDS-to-389 migration does not remove Fedora entries
by Endi Sukma Dewata 21 Oct '10

21 Oct '10

Hi, Please review the patch for this bug: https://bugzilla.redhat.com/show_bug.cgi?id=644929 https://bugzilla.redhat.com/attachment.cgi?id=454920&action=edit https://bugzilla.redhat.com/attachment.cgi?id=454920&action=diff Thanks! -- Endi S. Dewata

1 0

Please review: [Bug 629681] Retro Changelog trimming does not behave as expected
by Noriko Hosoi 20 Oct '10

20 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=645061 https://bugzilla.redhat.com/attachment.cgi?id=454711&action=diff https://bugzilla.redhat.com/attachment.cgi?id=454711&action=edit Description: As reporter Oliver Thalmann (oliver.thalmann(a)chuv.ch) pointed out, retrocl_init_trimming was repeating the retrocl_ housekeeping event every 5000 minutes instead of 5 minutes. This patch fixes it to 5 minutes. Files: ldap/servers/plugins/retrocl/retrocl.h ldap/servers/plugins/retrocl/retrocl_trim.c Thanks to Oliver Thalmann for his debugging. As he figured out, retrocl_init_trimming was passing the interval in microseconds not in milliseconds. /* * slapi_eq_repeat: cause an event to happen repeatedly. * * Arguments: * fn: the function to call * arg: an argument to pass to the called function * when: the time that the function should first be called * interval: the amount of time (in milliseconds) between * successive calls to the function * Returns: * slapi_eq_context - a handle to an opaque object which * the caller can use to refer to this particular scheduled */ Slapi_Eq_Context slapi_eq_repeat(slapi_eq_fn_t fn, void *arg, time_t when, unsigned long interval)

1 0

Please Review: (305131) Allow empty modify operation
by Nathan Kinder 19 Oct '10

19 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=305131 https://bugzilla.redhat.com/attachment.cgi?id=454373&action=edit

1 0

Please review : [Bug 592397] Upgrade tool dn2rdn: it does not clean up the entrydn in id2entry
by Noriko Hosoi 18 Oct '10

18 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=592397 https://bugzilla.redhat.com/attachment.cgi?id=454233&action=diff https://bugzilla.redhat.com/attachment.cgi?id=454233&action=edit Description: If entries created by the 389 v1.2.5 or older, the primary db (id2entry.db4) contains "entrydn:<normalized dn>". Upgrading from the old version to v1.2.6 keeps the entrydn attribute type and its value even though v1.2.6 is not supposed to store the entrydn in the database. 1) This patch drops the entrydn attribute and value in upgrading the db. 2) If an ldif file contains entrydn attribute type and value, import (ldif2db[.pl]) ignores it. 3) A leak was found in the export (db2ldif[.pl]) which is fixed. 4) When nsslapd-subtree-rename-switch configuration attribute has the value "on", entrydn is not used nor created. But the server accepted reindexing entrydn request and generated an entrydn index file. This patch rejects it. 5) Entry and dn cache clear calls (cache_clear) are added to dblayer_instance_close in "#if defined(_USE_VALGRIND)", which is not defined. To enable the code, the server needs to be rebuilt with defining the macro. This is purely for debugging. Files: ldap/servers/slapd/back-ldbm/dblayer.c ldap/servers/slapd/back-ldbm/id2entry.c ldap/servers/slapd/back-ldbm/import-threads.c ldap/servers/slapd/back-ldbm/ldbm_entryrdn.c ldap/servers/slapd/back-ldbm/ldif2ldbm.c ldap/servers/slapd/entry.c ldap/servers/slapd/slapi-plugin.h

1 0

Please Review: (643937) Initialize replication version flags
by Nathan Kinder 18 Oct '10

18 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=643937 https://bugzilla.redhat.com/attachment.cgi?id=454163&action=edit

1 0

jss/ldapjdk source
by Michele Baldessari 18 Oct '10

18 Oct '10

Hi all, some time ago the sources for jss could be found here: http://ftp.mozilla.org/pub/mozilla.org/security/jss/releases/ Now they seem to have disappeared. Anyone know if they're in some branch over at hg.mozilla.org? Similar question for ldapjdk. It used to be available through the following command: cvs -d :pserver:anonymous@cvs-mirror.mozilla.org:/cvsroot co -rLDAPJavaSDK_418 DirectorySDKSourceJava This still works, but I wonder if that is still to be considered the "official" upstream. Thanks, -- Michele Baldessari <michele(a)acksyn.org> C2A5 9DA3 9961 4FFB E01B D0BC DDD4 DCCB 7515 5C6D

3 2

Please review (take 2): [Bug 244229] targetattr not verified against schema when setting an aci
by Noriko Hosoi 15 Oct '10

15 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=244229 https://bugzilla.redhat.com/attachment.cgi?id=453769&action=diff https://bugzilla.redhat.com/attachment.cgi?id=453769&action=edit Following the comments by Rich, revised the proposal so that: 1) the error message is sent to the client, as well. 2) removed the useless virtual attribute type name checking. Thanks, --noriko

1 0

Please review: [Bug 244229] targetattr not verified against schema when setting an aci
by Noriko Hosoi 14 Oct '10

14 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=244229 https://bugzilla.redhat.com/attachment.cgi?id=453598&action=diff https://bugzilla.redhat.com/attachment.cgi?id=453598&action=edi Description: 1. When acl contains targetattr keyword: (targetattr [!]= "attribute_1 || attribute_2 ...|| attribute_n"), where attribute_n does not contain '*', the current ACL plugin accepts any attribute_n value even if it is not defined in the schema. This patch rejects the aci if it contains attribute_n not defined in schema with this error message: NSACLPlugin - targetattr "attribute_n" does not exist in schema. Please add attributeTypes "attribute_n" to schema if necessary. 2. To implement 1, slapi APIs slapi_attr_syntax_exists and slapi_vattr_type_exists are added. 3. An attributeTypes "connection" is added to 01core389.ldif which is referred in an aci of cn=monitor. Files: ldap/schema/01core389.ldif ldap/servers/plugins/acl/aclparse.c ldap/servers/slapd/attrsyntax.c ldap/servers/slapd/slapi-plugin.h ldap/servers/slapd/vattr.c

1 0

Please Review: (555955) Add CoS merge support
by Nathan Kinder 14 Oct '10

14 Oct '10

https://fedorahosted.org/reviewboard/r/91/

2 1

Please review: fix compiler warnings: remove extra format argument; use %lu for size_t printf format
by Rich Megginson 14 Oct '10

14 Oct '10

Trying out reviewboard https://fedorahosted.org/reviewboard/r/90/

1 0

Please review: [Bug 602456] Allow to add any cn=config attributes; allow to delete some cn=config attributes
by Noriko Hosoi 13 Oct '10

13 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=602456 https://bugzilla.redhat.com/attachment.cgi?id=453248&action=diff https://bugzilla.redhat.com/attachment.cgi?id=453248&action=edit Description: 1. Originally, configuration attributes are designed not to allow adding or deleting, but to allow just replacing. Due to a defect in checking the add operation, adding (LDAP_MOD_ADD) is not rejected. Instead of fixing the add checking to disallow adding, this patch logs the operation in the error log. 2. On the other hand, deleting configuration attributes is rejected by LDAP_UNWILLING_TO_PERFORM. We have a request that some attributes need to allow to delete. This patch introduces a config attribute nsslapd-allowed-to-delete-attrs, which value is configuration attributes separated by a space ' '. If an attribute is in the list, the attribute is allowed to delete. The delete operation is also logged in the error log. Files: ldap/servers/slapd/configdse.c ldap/servers/slapd/libglobs.c ldap/servers/slapd/proto-slap.h ldap/servers/slapd/slap.h Thanks, --noriko

1 1

Please review: [Bug 573889] Migration does not remove deprecated schema
by Endi Sukma Dewata 11 Oct '10

11 Oct '10

Hi, Please review the patches for the following bug. Thanks! https://bugzilla.redhat.com/show_bug.cgi?id=573889 https://bugzilla.redhat.com/attachment.cgi?id=452852&action=edit https://bugzilla.redhat.com/attachment.cgi?id=452852&action=diff https://bugzilla.redhat.com/attachment.cgi?id=452853&action=edit https://bugzilla.redhat.com/attachment.cgi?id=452853&action=diff -- Endi S. Dewata

1 0

Please review: [Bug 637852] sasl_io_start_packet: failed - read only 3 bytes of sasl packet length on connection 4
by Noriko Hosoi 11 Oct '10

11 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=637852 https://bugzilla.redhat.com/attachment.cgi?id=452772&action=diff https://bugzilla.redhat.com/attachment.cgi?id=452772&action=edit Description: A SASL packet is made from the 4 byte length and the length size of payload. When the first 4 bytes were not successfully received by one PR_Recv call, sasl_io_start_packet in sasl_io.c considered an error occurred and set PR_IO_ERROR, which terminates the SASL IO session. To give clients a chance to send the rest of the length in the next packet, this patch sets PR_WOULD_BLOCK_ERROR to the nspr error code and EWOULDBLOCK/EAGAIN to errno and once the succeeding packet comes in, it appends it to the previous incomplete length data and continues the SASL IO. File: ldap/servers/slapd/sasl_io.c

1 0

Please Review: (544321) remove-ds.pl should not throw fatal error is selinux port label is not found
by Nathan Kinder 11 Oct '10

11 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=544321 https://bugzilla.redhat.com/attachment.cgi?id=452770&action=edit

1 0

Please Review: (cov#16300) Unused variable in account policy plugin
by Nathan Kinder 08 Oct '10

08 Oct '10

2 1

Please Review: (631993) Proxy Accounts should log the actual account used
by Nathan Kinder 08 Oct '10

08 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=631993 https://bugzilla.redhat.com/attachment.cgi?id=452403&action=edit

1 0

Please review: [Bug 640854] changelog db: _cl5WriteOperation: failed to write entry; db error - 22 Invalid argument
by Noriko Hosoi 07 Oct '10

07 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=640854 https://bugzilla.redhat.com/attachment.cgi?id=452044&action=diff https://bugzilla.redhat.com/attachment.cgi?id=452044&action=edi Description: DBENV open flags is used to determine the DB_OPEN mode whether to set DB_AUTO_COMMIT or not. The info was eliminated in the change made for "Bug 633168 - Share backend dbEnv with the replication changelog". This patch picks up the backend dbenv openflags and uses it for the changelog DB_OPEN. Files: ldap/servers/plugins/replication/cl5_api.c ldap/servers/slapd/back-ldbm/dblayer.c ldap/servers/slapd/slapi-plugin.h

1 0

Please review: [Bug 640027] Naming attribute with a special char sequence parsing bug
by Noriko Hosoi 05 Oct '10

05 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=640027 https://bugzilla.redhat.com/attachment.cgi?id=451775&action=diff https://bugzilla.redhat.com/attachment.cgi?id=451775&action=edit Description: When DN is made from RDNs containing escaped plus "\+", the dn normalizer considers the value could be nested multi- valued RDNs. (e.g., cn=C\=Z\+A\=X\+B\=Y\,o\=O,o=OO) In that case, multi-valued RDNs are sorted by the normalizer. (==> cn=A\=X\+B\=Y\+C\=Z\,o\=O,o=OO) The sample DN provided by Andrey Ivanov contains "\+", but that is not a separator for the multi-valued RDNs: cn=mytest\+\=-123'\;456,dc=example,dc=com The dn normalizer should have checked the possibility, as well. The check is added in this patch. Also, sorting was not triggered if multi-valued RDNs are located at the end of the value. (e.g., cn=C\=X\,B\=Y\+A\=Z,o=OO) The bug was fixed, as well. File: ldap/servers/slapd/dn.c

1 0

Please Review: (625335) default "Enable self write for common attributes" aci has permission to invalid attribute
by Nathan Kinder 05 Oct '10

05 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=625335 https://bugzilla.redhat.com/attachment.cgi?id=451735&action=edit

1 0

Please review: [Bug 639289] Adding a new CN entry with UpperCase UTF-8 Character
by Noriko Hosoi 04 Oct '10

04 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=639289 https://bugzilla.redhat.com/attachment.cgi?id=451562&action=diff https://bugzilla.redhat.com/attachment.cgi?id=451562&action=edit Description: There was a bug in the utf8 uppe2Lower table: Character İ (LATIN CAPITAL LETTER I WITH DOT ABOVE) did not map to the corresponding LATIN SMALL LETTER DOTLESS I (2 bytes) but to ascii 'i' (1 byte). The shortened DN tailed with a garbage character and the entry was treated as an orphan entry which does not belong to any suffix. This patch fixes the mapping table mismatch as well as adds a code to dn_ignore_case_to_end to force to NULL terminate the converted string. Files: ldap/servers/slapd/dn.c ldap/servers/slapd/utf8compare.c Thanks, --noriko

1 0

Followup on crash
by Gary Morris 04 Oct '10

04 Oct '10

Hi guys.. so far since the fix things have been mostly stable. However, sometimes in the evening things when things are very quiet, i'm getting a crash, and i come in the next morning and ds is down. This is just an heads up. Unless there is a known issue and something I should try, I'm going to try to keep a session of gdb running on it so I can try to capture what goes on when it happens and send it your way.

2 1

Please review: Bug 628096 - spurious error message from /sbin/service when doing a stop on no instances
by Endi Sukma Dewata 04 Oct '10

04 Oct '10

Hi, Please review the patch for the following bug. Thanks! https://bugzilla.redhat.com/show_bug.cgi?id=628096 https://bugzilla.redhat.com/attachment.cgi?id=451446&action=edit https://bugzilla.redhat.com/attachment.cgi?id=451446&action=diff -- Endi S. Dewata

1 0

Please review: openldap ber_init will assert if the bv->bv_val is NULL
by Rich Megginson 01 Oct '10

01 Oct '10

2 1

Please Review: (522055) scope check for managed attribute fails
by Nathan Kinder 01 Oct '10

01 Oct '10

https://bugzilla.redhat.com/show_bug.cgi?id=522055 https://bugzilla.redhat.com/attachment.cgi?id=451123&action=edit

1 0

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

389-devel October 2010