Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mon 33334 cbd memory leak jc 2 #1076

Merged
merged 7 commits into from
Jan 29, 2024

Conversation

jean-christophe81
Copy link
Contributor

Description

Fixes # (issue)

memory leak in tls module

Type of change

  • Patch fixing an issue (non-breaking change)
  • New functionality (non-breaking change)
  • Breaking change (patch or feature) that might cause side effects breaking part of the Software
  • Updating documentation (missing information, typo...)

Target serie

  • 22.04.x
  • 22.10.x
  • 23.04.x
  • 23.10.x
  • 24.04.x (master)

How this pull request can be tested ?

Please describe the procedure to verify that the goal of the PR is matched. Provide clear instructions so that it can be correctly tested.

Any relevant details of the configuration to perform the test should be added.

Checklist

  • I have followed the coding style guidelines provided by Centreon
  • I have commented my code, especially new classes, functions or any legacy code modified. (docblock)
  • I have commented my code, especially hard-to-understand areas of the PR.
  • I have made corresponding changes to the documentation.
  • I have rebased my development branch on the base branch (master, maintenance).

@jean-christophe81 jean-christophe81 marked this pull request as ready for review January 23, 2024 16:11
@jean-christophe81 jean-christophe81 requested a review from a team as a code owner January 23, 2024 16:11
@jean-christophe81 jean-christophe81 force-pushed the MON-33334_cbd_memory-leak_jc_2 branch from fa60b36 to e38dc6f Compare January 23, 2024 16:25
@jean-christophe81 jean-christophe81 force-pushed the MON-33334_cbd_memory-leak_jc_2 branch from e38dc6f to 04b83a9 Compare January 24, 2024 05:30
@jean-christophe81 jean-christophe81 requested a review from a team as a code owner January 24, 2024 05:30
@jean-christophe81 jean-christophe81 force-pushed the MON-33334_cbd_memory-leak_jc_2 branch from 04b83a9 to 53aaca4 Compare January 24, 2024 06:43
@sc979 sc979 removed their request for review January 24, 2024 10:10
@sc979
Copy link
Contributor

sc979 commented Jan 24, 2024

There's nothing to review in pipeline scope. That's why, I removed my "request for review"

@jean-christophe81 jean-christophe81 merged commit 9ac7cc3 into develop Jan 29, 2024
14 checks passed
@jean-christophe81 jean-christophe81 deleted the MON-33334_cbd_memory-leak_jc_2 branch January 29, 2024 13:13
sfarouq-ext pushed a commit that referenced this pull request Apr 1, 2024
* add malloc_trace library

* move some parts of malloc-trace code

* flush only orphan malloc and free

* MON-33334 malloc_trace realloc name fix, tls memory-leak fix

* function passed to gnutls must never throw

* generate package

* no generate package

REFS:MON-33334
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants