opentelemetry-cpp icon indicating copy to clipboard operation
opentelemetry-cpp copied to clipboard

Fix memory leak in TlsRandomNumberGenerator() constructor #2660

Open hongweipeng opened this issue 1 year ago • 1 comments

Fixes #2660

Changes

Please provide a brief description of the changes here.

For significant contributions please make sure you have completed the following items:

  • [ ] CHANGELOG.md updated for non-trivial changes
  • [x] Unit tests have been added
  • [ ] Changes in public API reviewed

hongweipeng avatar May 07 '24 07:05 hongweipeng

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: hongweipeng / name: Weipeng Hong (6b44492fd0b2907430b9f9958287a1f874baf0e2)
  • :white_check_mark: login: marcalff / name: Marc Alff (1f404a3d9f320fcc7b25d7c686f6acccef13ae43)

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 87.60%. Comparing base (497eaf4) to head (1f404a3). Report is 108 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2661      +/-   ##
==========================================
+ Coverage   87.12%   87.60%   +0.49%     
==========================================
  Files         200      190      -10     
  Lines        6109     5870     -239     
==========================================
- Hits         5322     5142     -180     
+ Misses        787      728      -59     
Files Coverage Δ
sdk/src/common/random.cc 96.00% <100.00%> (+0.17%) :arrow_up:

... and 121 files with indirect coverage changes

codecov[bot] avatar Jul 19 '24 10:07 codecov[bot]