From leo@yuriev.ru Thu Mar 19 12:11:05 2015 From: leo@yuriev.ru To: openldap-bugs@openldap.org Subject: Re: (ITS#8062) Assertion 'IS_LEAF(mp)' failed in mdb_cursor_next() Date: Thu, 19 Mar 2015 12:11:03 +0000 Message-ID: MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="===============6635110775717709431==" --===============6635110775717709431== Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: quoted-printable --001a113602ac48c9b00511a317b0 Content-Type: text/plain; charset=3DUTF-8 Content-Transfer-Encoding: quoted-printable Hm, in some case valgrind was reported as I noticed, sure 101%. Leonid 18.03.2015 23:12 =3DD0=3DBF=3DD0=3DBE=3DD0=3DBB=3DD1=3D8C=3DD0=3DB7=3DD0=3DBE= =3DD0=3DB2=3DD0=3DB0=3DD1=3D82=3DD0=3D =3DB5=3DD0=3DBB=3DD1=3D8C "Howard Chu" =3DD0=3DBD=3DD0=3DB0= =3DD0=3DBF=3DD0=3DB8=3DD1=3D81=3D =3DD0=3DB0=3DD0=3DBB: > leo(a)yuriev.ru wrote: > >> As far as I remember this was fixed in ReOpenLDAP at January. >> Valgrind was reporting 'using uninitialized'. >> > > Not happening here. mdb_cursor_first() always initializes mc_ki. > >> >> Leonid. >> >> @@ -7301,6 +7551,7 @@ mdb_cursor_init(MDB_cursor *mc, MDB_txn *txn, >> MDB_dbi dbi, MDB_xcursor *mx) >> mc->mc_top =3D3D 0; >> mc->mc_pg[0] =3D3D 0; >> mc->mc_flags =3D3D 0; >> + mc->mc_ki[0] =3D3D 0; >> if (txn->mt_dbs[dbi].md_flags & MDB_DUPSORT) { >> mdb_tassert(txn, mx !=3D3D NULL); >> mc->mc_xcursor =3D3D mx; >> >> >> >> >> >> > > -- > -- Howard Chu > CTO, Symas Corp. http://www.symas.com > Director, Highland Sun http://highlandsun.com/hyc/ > Chief Architect, OpenLDAP http://www.openldap.org/project/ > --001a113602ac48c9b00511a317b0 Content-Type: text/html; charset=3DUTF-8 Content-Transfer-Encoding: quoted-printable

Hm, in some case valgrind was reported as I noticed, sure 10= =3D 1%.

Leonid

18.03.2015 23:12 =3DD0=3DBF=3DD0=3DBE=3DD0=3DBB= =3DD1=3D8C=3DD0=3DB7=3D =3DD0=3DBE=3DD0=3DB2=3DD0=3DB0=3DD1=3D82=3DD0=3DB5=3DD0=3DBB=3DD1=3D8C "= Howard Chu" <hyc(a)symas.com> =3DD0=3DBD=3DD0=3DB0= =3DD0=3DBF=3DD0=3DB8=3D =3DD1=3D81=3DD0=3DB0=3DD0=3DBB:
= =3D leo(a)yuriev.ru = wrote:=3D
As far as I remember this was fixed in ReOpenLDAP at January.
Valgrind was reporting 'using uninitialized'.

Not happening here. mdb_cursor_first() always initializes mc_ki.

Leonid.

@@ -7301,6 +7551,7 @@ mdb_cursor_init(MDB_cursor *mc, MDB_txn *txn,
MDB_dbi dbi, MDB_xcursor *mx)
=3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 mc->mc_top =3D3D 0;=
=3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 mc->mc_pg[0] =3D3D = 0;
=3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 mc->mc_flags =3D3D = 0;
+=3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0mc->mc_ki[0] =3D3D 0;
=3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 if (txn->mt_dbs[dbi= ].md_flags & M=3D DB_DUPSORT) {
=3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 = =3DC2=3DA0 =3DC2=3DA0 mdb_tassert(=3D txn, mx !=3D3D NULL);
=3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 = =3DC2=3DA0 =3DC2=3DA0 mc->mc_xc=3D ursor =3D3D mx;







--
=3DC2=3DA0 -- Howard Chu
=3DC2=3DA0 CTO, Symas Corp.=3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0 =3DC2= =3DA0 =3DC2=3DA0http://www.symas.com
=3DC2=3DA0 Director, Highland Sun=3DC2=3DA0 =3DC2=3DA0 =3DC2=3DA0http://highlandsun.com/hyc/
=3DC2=3DA0 Chief Architect, OpenLDAP=3DC2=3DA0 http://www.openldap.org/project/
--001a113602ac48c9b00511a317b0-- --===============6635110775717709431==--