#50833 Ticket 50709 - (cont) Several memory leaks reported by Valgrind for 389-ds 1.3.9.1-10
Closed by spichugi. Opened by tbordaz.
tbordaz/389-ds-base ticket_50709_cont  into  master

Download 50833.patch

Description of the problem:

Original fix was incorrect as it set again in pblock (SLAPI_CONN_CLIENTNETADDR_ACLIP)
    the same structure. As old structure is freed during the slapi_pblock_set,
pblock refers to a freed structure.
    Later an other threads using the same aclpb contain will use it after free

Description of the fix:
Only sets in pblock a newly allocated structure

https://pagure.io/389-ds-base/issue/50709

Reviewed by: ?

If this now passes the paged results test (with ASAN) it gets my ack!

rebased onto 058f4da2bc40d2ea4ebf4890b3d42847ac059122

Pull-Request has been merged by tbordaz

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3887

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi

Metadata