389-ds-base

#50985 Issue 50984 - Memory leaks in disk monitoring
Closed by spichugi. Opened by spichugi.
spichugi/389-ds-base i50984  into  master

Download 50985.patch

Description: Memory leaks are reported by the disk monitoring test suite.
The direct leak is related to char **dirs array which is not freed at all.
Free the array when we clean up or go to shutdown.
Fix disk_monitoring_test.py::test_below_half_of_the_threshold_not_starting_after_shutdown.
It should accept different exception whe the instance is not started.

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

Reviewed by: ?

Ack looks good to me

rebased onto 8ecada01035e8b90538d72b42510e742870b0671

Pull-Request has been merged by spichugi

minor comment. slapi_ch_array_free frees the dirs elements but does not reset dirs pointer.
you may add 'dirs = NULL;' after each slapi_ch_array_free(dirs).

the same

the same

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

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
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.