[Date Prev][Date Next] [Chronological] [Thread] [Top]

Re: SLAPI (ITS#2073)



Steve,

I've uploaded a patch to

http://www.sys-net.it/~ando/Download/Pierangelo-Masarati-20021104-Steve-Omrani.ns_slapi_.diff

which includes the porting of your submission
to current HEAD code and some fixes.  Among these
you should carefully review "slapi_ops.c" because
there is a number of pointers that are used
uninitialized (significantly the pointer to the
Backend structure, "be"). Other fixes are related
to slapi API functions, like the (apparently)
inconsistent use of slapi_strlen()/slapi_ch_stlen()
(none of which is defined).  I also get very scary
warnings from libtool because you statically link
some libtmpslapd.a to libslapi (if I get it correctly)
in an unportable manner.

Note that I didn't check the functionality of SLAPI
itself (this will require a little more); I only
verified that I can compile OpenLDAP safely and
that the usual tests run OK.

In case you think my changes are appropriate, send
me a note (or a patch with further fixes) and I'll
commit everything.

Of course, any other comment or review is welcome
before I actually commit the patch.

Pierangelo.

-- 
Pierangelo Masarati
mailto:pierangelo.masarati@sys-net.it