#50858 Ticket 50857 - Memory leak in ACI using IP subject
Closed 3 years ago by spichugi. Opened 4 years ago by tbordaz.
tbordaz/389-ds-base ticket_50857  into  master

file modified
+1
@@ -436,6 +436,7 @@ 

          return;

  

      LASIpTreeDealloc(((LASIpContext_t *)*las_cookie)->treetop);

+     LASIpTreeDealloc(((LASIpContext_t *)*las_cookie)->treetop_ipv6);

      PERM_FREE(*las_cookie);

      *las_cookie = NULL;

      return;

Bug Description:
When a ACI is evaluated (LASIpEval) a context (cookie) is allocated.
At the end of the connection, the context is freed
via a callback (LASIpFlush).
The context contains two LASIpTree_t tree (ipv4 and ipv6)
In free callback, only ipv4 tree is freed

Fix Description:
Free ipv6 tree in LASIpTree

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

Reviewed by: ?

Platforms tested: F31

Flag Day: no

Doc impact: no

ACK! I think I introduced this leak when I added IPv6 support to ACI's (like 7 years ago).

rebased onto 473d616

4 years ago

Pull-Request has been merged by tbordaz

4 years ago

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/3912

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

3 years ago
Metadata