openldap-bugs January 2015

openldap-bugs@openldap.org

22 participants
114 discussions

Re: (ITS#8018) a lot of warnings building with -Wall
by hyc＠symas.com 06 Jan '15

06 Jan '15

leo(a)yuriev.ru wrote: > Full_Name: Leonid Yuriev > Version: git-master > OS: RHEL7 > URL: ftp://ftp.openldap.org/incoming/ > Submission from: (NULL) (31.130.36.33) > > > It is about 5K warnings reported by gcc/clang if building OpenLDAP with -Wall. > > CFLAGS="-Wall" ./configure ... > make depend && make > > The vast majority of warnings about harmless things: unnecessary arguments to > printf through Debug macro, Ignore those. Use -Wno-format-extra-args > unused variables and functions, etc. > But such amount of warnings makes impossible to use -Wall to find a potential > problems. > > I have prepared the patchset which fixes most of these warnings. > It will be submited shortly. > > Leonid. > > -- -- 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#8018) a lot of warnings building with -Wall
by pierangelo.masarati＠polimi.it 06 Jan '15

06 Jan '15

On 06/01/2015 15:47, leo(a)yuriev.ru wrote: > Full_Name: Leonid Yuriev > Version: git-master > OS: RHEL7 > URL: ftp://ftp.openldap.org/incoming/ > Submission from: (NULL) (31.130.36.33) > > > It is about 5K warnings reported by gcc/clang if building OpenLDAP with -Wall. > > CFLAGS="-Wall" ./configure ... > make depend && make > > The vast majority of warnings about harmless things: unnecessary arguments to > printf through Debug macro, unused variables and functions, etc. > But such amount of warnings makes impossible to use -Wall to find a potential > problems. > > I have prepared the patchset which fixes most of these warnings. > It will be submited shortly. My 2c: - normal users do not need to build with -Wall; only developers might need to - in addition to -Wall, if you add -Wno-[specific harmless warnings you want to disable], then only relevant ones do surface. p. -- Pierangelo Masarati Associate Professor Dipartimento di Scienze e Tecnologie Aerospaziali Politecnico di Milano

1 0

(ITS#8018) a lot of warnings building with -Wall
by leo＠yuriev.ru 06 Jan '15

06 Jan '15

Full_Name: Leonid Yuriev Version: git-master OS: RHEL7 URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (31.130.36.33) It is about 5K warnings reported by gcc/clang if building OpenLDAP with -Wall. CFLAGS="-Wall" ./configure ... make depend && make The vast majority of warnings about harmless things: unnecessary arguments to printf through Debug macro, unused variables and functions, etc. But such amount of warnings makes impossible to use -Wall to find a potential problems. I have prepared the patchset which fixes most of these warnings. It will be submited shortly. Leonid.

1 0

Re: (ITS#8017) Enhancement: Limit core dump size
by quanah＠zimbra.com 05 Jan '15

05 Jan '15

--On Tuesday, January 06, 2015 1:31 AM +0000 gnoe(a)symas.com wrote: > Full_Name: Greg Noe > Version: NA > OS: NA > URL: ftp://ftp.openldap.org/incoming/ > Submission from: (NULL) (63.142.209.94) > > > Request: If possible, add an option to limit the size of a core dump > when slapd crashes. > > Reason for request: In cases where very large caches are used and slapd > crashes, the size of the core dump files can exceed the available disk > space. Core dump size is controlled by the kernel already, via sysctl. Why does one need an option to slapd for this? -- Quanah Gibson-Mount Platform Architect Zimbra, Inc. -------------------- Zimbra :: the leader in open source messaging and collaboration

1 0

(ITS#8017) Enhancement: Limit core dump size
by gnoe＠symas.com 05 Jan '15

05 Jan '15

Full_Name: Greg Noe Version: NA OS: NA URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (63.142.209.94) Request: If possible, add an option to limit the size of a core dump when slapd crashes. Reason for request: In cases where very large caches are used and slapd crashes, the size of the core dump files can exceed the available disk space.

1 0

Re: (ITS#8016) silence on wrong envflags for lmdb-backend
by hyc＠symas.com 05 Jan '15

05 Jan '15

leo(a)yuriev.ru wrote: > Full_Name: Leonid Yuriev > Version: 2.4-HEAD > OS: RHEL7 > URL: ftp://ftp.openldap.org/incoming/ > Submission from: (NULL) (31.130.36.33) > > > No any error reported by lmdb-backend when wrong options was specified in > envflags. > > For example slapd.conf: > envflags This is a rebus codestyle and the technical debt > > Patch will be available shortly. fixed in master -- -- 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#8016) silence on wrong envflags for lmdb-backend
by hyc＠symas.com 05 Jan '15

05 Jan '15

leo(a)yuriev.ru wrote: > This is a multi-part message in MIME format. > --------------080401010006020303000400 > Content-Type: text/plain; charset=windows-1251; format=flowed > Content-Transfer-Encoding: 7bit > > Please review and merge. > > Leonid. > > --- > > The attached files is derived from OpenLDAP Software. All of the > modifications > to OpenLDAP Software represented in the following patch(es) were > developed by > Peter-Service LLC, Moscow, Russia. Peter-Service LLC has not assigned > rights > and/or interest in this work to any party. I, Leonid Yuriev am > authorized by > Peter-Service LLC, my employer, to release this work under the following > terms. > > Peter-Service LLC hereby places the following modifications to OpenLDAP > Software > (and only these modifications) into the public domain. Hence, these > modifications may be freely used and/or redistributed for any purpose > with or without attribution and/or other notice. > > > > --------------080401010006020303000400 > Content-Type: text/x-patch; > name="its8016.patch" > Content-Transfer-Encoding: 7bit > Content-Disposition: attachment; > filename="its8016.patch" > > commit 36ecfd44559d3b89a7dd5bcbfb6a22d8aeb8142b > Author: Leo Yuriev <leo(a)yuriev.ru> > Date: 2015-01-04 09:05:36 +0300 > > fix: report about wrong envflags in lmdb-backend instead of silently ignore them. > > diff --git a/servers/slapd/back-mdb/config.c b/servers/slapd/back-mdb/config.c > index bb9be73..68009b6 100644 > --- a/servers/slapd/back-mdb/config.c > +++ b/servers/slapd/back-mdb/config.c > @@ -620,8 +620,10 @@ mdb_cf_gen( ConfigArgs *c ) > } > mdb->mi_dbenv_flags |= mdb_envflags[j].mask; > } else { > - /* unknown keyword */ > - rc = 1; > + fprintf( stderr, "%s: " > + "unknown keyword \"%s\" in \"envflags\".\n", > + c->log, c->argv[i] ); > + return 1; > } > } > } > > --------------080401010006020303000400-- Use of fprintf(stderr) for config errors is deprecated. New code should only be using c->cr_msg. -- -- 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#8016) silence on wrong envflags for lmdb-backend
by leo＠yuriev.ru 04 Jan '15

04 Jan '15

This is a multi-part message in MIME format. --------------080401010006020303000400 Content-Type: text/plain; charset=windows-1251; format=flowed Content-Transfer-Encoding: 7bit Please review and merge. Leonid. --- The attached files is derived from OpenLDAP Software. All of the modifications to OpenLDAP Software represented in the following patch(es) were developed by Peter-Service LLC, Moscow, Russia. Peter-Service LLC has not assigned rights and/or interest in this work to any party. I, Leonid Yuriev am authorized by Peter-Service LLC, my employer, to release this work under the following terms. Peter-Service LLC hereby places the following modifications to OpenLDAP Software (and only these modifications) into the public domain. Hence, these modifications may be freely used and/or redistributed for any purpose with or without attribution and/or other notice. --------------080401010006020303000400 Content-Type: text/x-patch; name="its8016.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="its8016.patch" commit 36ecfd44559d3b89a7dd5bcbfb6a22d8aeb8142b Author: Leo Yuriev <leo(a)yuriev.ru> Date: 2015-01-04 09:05:36 +0300 fix: report about wrong envflags in lmdb-backend instead of silently ignore them. diff --git a/servers/slapd/back-mdb/config.c b/servers/slapd/back-mdb/config.c index bb9be73..68009b6 100644 --- a/servers/slapd/back-mdb/config.c +++ b/servers/slapd/back-mdb/config.c @@ -620,8 +620,10 @@ mdb_cf_gen( ConfigArgs *c ) } mdb->mi_dbenv_flags |= mdb_envflags[j].mask; } else { - /* unknown keyword */ - rc = 1; + fprintf( stderr, "%s: " + "unknown keyword \"%s\" in \"envflags\".\n", + c->log, c->argv[i] ); + return 1; } } } --------------080401010006020303000400--

1 0

(ITS#8016) silence on wrong envflags for lmdb-backend
by leo＠yuriev.ru 04 Jan '15

04 Jan '15

Full_Name: Leonid Yuriev Version: 2.4-HEAD OS: RHEL7 URL: ftp://ftp.openldap.org/incoming/ Submission from: (NULL) (31.130.36.33) No any error reported by lmdb-backend when wrong options was specified in envflags. For example slapd.conf: envflags This is a rebus codestyle and the technical debt Patch will be available shortly.

1 0

Re: (ITS#8011) few mistakes in lmdb-backend
by hyc＠symas.com 03 Jan '15

03 Jan '15

Леонид Юрьев wrote: > Updated. > Please pickup. > diff --git a/servers/slapd/back-mdb/dn2id.c b/servers/slapd/back-mdb/dn2id.c > index 41c4758..0890bce 100644 > --- a/servers/slapd/back-mdb/dn2id.c > +++ b/servers/slapd/back-mdb/dn2id.c > @@ -746,7 +746,8 @@ mdb_idscopes( > /* remember our chain of parents */ > id2.mid = id; > id2.mval = data; > - mdb_id2l_insert( isc->sctmp, &id2 ); > + rc = mdb_id2l_insert( isc->sctmp, &id2 ); > + assert(rc == 0); > } > ptr = data.mv_data; > ptr += data.mv_size - sizeof(ID); Unnecessary, the total number of parents is constrained by the max length of a DN which is 8192 bytes. A maximum depth DN (max number of shortest possible RDNs) will always fit in sctmp. I will commit the remainder of the patch. -- -- 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

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

openldap-bugs January 2015