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.

Sign up for GitHub

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

Jump to bottom

3.0 release has a memory leak, it has been identified, please merge. #2606

Closed
zhouxiaojun2008 opened this issue Sep 13, 2021 · 3 comments
Closed

3.0 release has a memory leak, it has been identified, please merge. #2606

zhouxiaojun2008 opened this issue Sep 13, 2021 · 3 comments
Assignees
@winlinvip
Labels
Bug It might be a bug. PullRequest Has PR or solution in issue. TransByAI Translated by AI/GPT.
Milestone
3.0

Comments

@zhouxiaojun2008
Copy link
Contributor

zhouxiaojun2008 commented Sep 13, 2021 •
edited by winlinvip
Loading

Description
Using srs3.0 r5 version, we found a problem of slow memory growth. The cluster consists of 2 edges and 2 origin servers. There are some sources that are missing or closed, and the edge nodes frequently receive pull requests for these sources. At this time, the edge will continuously perform edge-pull operations between the 2 origin servers. After running for a long time, we noticed that the memory keeps increasing, although the growth is relatively slow (the speed of growth depends on the number of invalid pull requests).
We analyzed the subsequent updates of version 3.0 r5 and found no modifications addressing this issue. Then we compared it with version 4.0 and found that it has been changed in the 4.0 version, for more details refer to [reference].
b9355c1

Additionally, after encountering the issue, we tried using the gmc tool with srs3.0 r5 version, but it was not successful. We received the following error message. Do you have any suggestions?
image

TRANS_BY_GPT3
@winlinvip winlinvip added Bug It might be a bug. PullRequest Has PR or solution in issue. labels Sep 15, 2021
@winlinvip winlinvip assigned winlinvip and unassigned chen-guanghua Sep 15, 2021
@winlinvip
Copy link
Member

winlinvip commented Sep 15, 2021 •
edited
Loading

👍 I will find time to merge it into 3.0.

TRANS_BY_GPT3

@winlinvip winlinvip added this to the 3.0 milestone Sep 15, 2021
winlinvip added a commit that referenced this issue Oct 7, 2021
@winlinvip
Fix #2606: Memory leak for RTMP client, pick from 4.0. v3.0.170
e801174
@winlinvip
Copy link
Member

Fixed by e801174

@digitalm8 digitalm8 mentioned this issue Oct 11, 2021
Closed
@duiniuluantanqin duiniuluantanqin mentioned this issue Oct 20, 2021
Closed
@winlinvip winlinvip changed the title 3.0release 有内存泄漏，已查明，望合并 3.0 release has a memory leak, it has been identified, please merge. Jul 29, 2023
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 29, 2023
@winlinvip
Copy link
Member

Fixed by SmartPtr: Support shared ptr for live source. v6.0.129

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@winlinvip winlinvip

Labels
Bug It might be a bug. PullRequest Has PR or solution in issue. TransByAI Translated by AI/GPT.
Projects
None yet
Milestone
3.0
Development

No branches or pull requests
3 participants
@winlinvip @chen-guanghua @zhouxiaojun2008