#49910 Ticket 49029 - Internal logging thread data needs to allocate int pointers
Closed by spichugi. Opened by mreynolds.
mreynolds/389-ds-base ticket49029  into  master

Download 49910.patch

Bug Description:

The original version of the fix incorrectly used stack pointers
to update the thread data. These pointers would go out of scope
and could cause a crash when updated.

Fix Description:

Allocate the integer pointers at initalization time for each
worker thread including main() thread. Also cleaned up other
areas of this code/feature.

Passed ASAN tests.

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

Reviewed by: ?

I've tested this PR with and without ASAN. The ipa-server-install command was successful, no crashes were observed.

rebased onto 9f9f4a9d82a1342facf34956d1355fa76fc97313

Pull-Request has been merged by mreynolds

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

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