amazon-kinesis-video-streams-webrtc-sdk-c icon indicating copy to clipboard operation
amazon-kinesis-video-streams-webrtc-sdk-c copied to clipboard

Thread cancel on the media sender thread leads to memory leaks from w…

Open jdelapla opened this issue 2 years ago • 2 comments

…riteFrame() not freeing all its heap usage

Issue #, if available: #1450 Description of changes:

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

jdelapla avatar Apr 15 '22 18:04 jdelapla

Codecov Report

Merging #1452 (59c2d32) into develop (866aee4) will decrease coverage by 0.05%. The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           develop    #1452      +/-   ##
===========================================
- Coverage    51.50%   51.44%   -0.06%     
===========================================
  Files           46       46              
  Lines        12794    12795       +1     
===========================================
- Hits          6589     6582       -7     
- Misses        6205     6213       +8     
Impacted Files Coverage Δ
samples/Common.c 0.00% <0.00%> (ø)
samples/kvsWebRTCClientMaster.c 0.00% <0.00%> (ø)
src/source/Ice/IceAgent.c 85.81% <0.00%> (-0.46%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 866aee4...59c2d32. Read the comment docs.

codecov-commenter avatar Apr 15 '22 19:04 codecov-commenter

This is a very old issue. We encourage you to check if this is still an issue in the latest release and if you find that this is still a problem, please feel free to open a new one.

github-actions[bot] avatar Oct 13 '22 00:10 github-actions[bot]

Closing in favor of https://github.com/awslabs/amazon-kinesis-video-streams-webrtc-sdk-c/pull/1641

niyatim23 avatar Feb 12 '23 22:02 niyatim23