[389-devel] Please review: [389 Project] #48226: In MMR, double free coould occur under some special condition

2015-09-15 Thread Noriko Hosoi

https://fedorahosted.org/389/ticket/48226

https://fedorahosted.org/389/attachment/ticket/48226/0001-Ticket-48226-In-MMR-double-free-coould-occur-under-s.2.patch
git patch file (master) -- additinal fix for the memory leak (regression 
by the previous patch)


Thanks to Mark Reynolds for the CI test valgrind improvement. This leak 
was found by his work.

--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel

[389-devel] please review: Ticket 48273 - lib389 - improve valgrind functions

2015-09-15 Thread Mark Reynolds

https://fedorahosted.org/389/attachment/ticket/48273/0001-Ticket-48273-Improve-valgrind-functions.patch

lib389 update
https://fedorahosted.org/389/attachment/ticket/48273/0001-Ticket-48273-Improve-valgrind-functions.patch

389-ds update to test scripts
https://fedorahosted.org/389/attachment/ticket/48273/0001-Ticket-48273-Update-lib389-tests-for-new-valgrind-fu.patch
--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel

[389-devel] Please review: Ticket #48277 - Assorted build fixes and make rpms target

2015-09-15 Thread Simo Sorce
https://fedorahosted.org/389/ticket/48277

There are 5 patches attached to it.

Please keep me in CC if you want feedback as I am not subscribed to the
list.

HTH,
Simo.

-- 
Simo Sorce * Red Hat, Inc * New York

--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel

[389-devel] Please review ticket 48266: Fractional replication evaluates several times the same CSN

2015-09-15 Thread thierry bordaz

Thank you so much Rich and Noriko for your feedbacks.
I was late to take them into account and I am sorry for that delay.
Here is the last patch and the associated test case.


https://fedorahosted.org/389/ticket/48266

https://fedorahosted.org/389/attachment/ticket/48266/0001-Ticket-48266-2-Fractional-replication-evaluates-severa.patch

https://fedorahosted.org/389/attachment/ticket/48266/0001-Ticket-48266-test-case.patch
--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel

[389-devel] please review: Ticket 47957 - Add replication test suite for a wait async feature

2015-09-15 Thread Simon Pichugin
Hi team,

please, review my patch that contains new test suite for the
nsDS5ReplicaWaitForAsyncResults feature.

https://fedorahosted.org/389/ticket/47957

https://fedorahosted.org/389/attachment/ticket/47957/0001-Ticket-47957-Add-replication-test-suite-for-a-wait-a.patch


pgpzJTmzMDwn_.pgp
Description: PGP signature
--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel

Re: [389-devel] please review (revised): Ticket #47553 - Automated the verification procedure

2015-09-15 Thread Simon Pichugin
Hi team,

please, review my slightly revised patch.

Python 2 > Python 3 fix.

https://fedorahosted.org/389/ticket/47553

https://fedorahosted.org/389/attachment/ticket/47553/0001-Ticket-47553-Automated-the-verification-procedure.2.patch

On Wed, Sep 02, 2015 at 05:50:50PM +0200, Simon Pichugin wrote:
> https://fedorahosted.org/389/ticket/47553
> 
> https://fedorahosted.org/389/attachment/ticket/47553/0001-Ticket-47553-Automated-the-verification-procedure.patch



> --
> 389-devel mailing list
> 389-devel@lists.fedoraproject.org
> https://admin.fedoraproject.org/mailman/listinfo/389-devel



pgp2kqm2quyJ9.pgp
Description: PGP signature
--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel

Re: [389-devel] please review (revised): Ticket #47761 - Added a few testcases to the basic testsuite

2015-09-15 Thread Simon Pichugin
Hi team,

please, review my revised patch.

https://fedorahosted.org/389/ticket/47761

https://fedorahosted.org/389/attachment/ticket/47761/0001-Ticket-47761-Added-a-few-testcases-to-the-basic-test.2.patch

On Mon, Sep 14, 2015 at 02:28:29PM +0200, Simon Pichugin wrote:
> Hi team,
> 
> please, review my patch. :)
> 
> On Mon, Aug 31, 2015 at 11:32:22AM +0200, Simon Pichugin wrote:
> > https://fedorahosted.org/389/ticket/47761
> > 
> > https://fedorahosted.org/389/attachment/ticket/47761/0001-Ticket-47761-Added-a-few-testcases-to-the-basic-test.patch
> 
> 
> 
> > --
> > 389-devel mailing list
> > 389-devel@lists.fedoraproject.org
> > https://admin.fedoraproject.org/mailman/listinfo/389-devel
> 



> --
> 389-devel mailing list
> 389-devel@lists.fedoraproject.org
> https://admin.fedoraproject.org/mailman/listinfo/389-devel



pgpq45S84ZC3w.pgp
Description: PGP signature
--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel

[389-devel] resent: please review: Ticket 48204 - Add support for python3 to lib389 and DS

2015-09-15 Thread Mark Reynolds


https://fedorahosted.org/389/ticket/48204

lib389 patch
https://fedorahosted.org/389/attachment/ticket/48204/0001-Ticket-48204-update-lib389-for-python3.patch

389-ds-base patch
https://fedorahosted.org/389/attachment/ticket/48204/0001-Ticket-48204-update-lib389-test-scripts-for-python-3.patch




--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel

Re: [389-devel] Please review (memory leak): [389 Project] #48252: db2index creates index entry from deleted records

2015-09-15 Thread Ludwig Krispenz
Thanks for this, it not only fixes a memory leak but also a regression. 
with the previous version of the fix after the processing of the first 
tombstone nstombstone_vals was set an duse in deciding what to index :-)



On 08/28/2015 02:19 AM, Noriko Hosoi wrote:

https://fedorahosted.org/389/ticket/48252

https://fedorahosted.org/389/attachment/ticket/48252/0001-Ticket-48252-db2index-creates-index-entry-from-delet.2.patch 



Description: Commit ca4d38dff3bd820af2bf60c9bcc82fd64aac2556 fixing
ticket #48252 introduced memory leaks.
. nstombstone_vals needs to be allocated just once and freed at 
the end.
. when skipping to index an entry for being a tombstone, the entry 
needs

  to be freed.

--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel


--
389-devel mailing list
389-devel@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/389-devel