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

Asan: Try to fix st_memory_leak for asan check #3264

Merged

Conversation

chen-guanghua
Copy link
Contributor

Asan: fix st_memory_leak for asan check

@chen-guanghua chen-guanghua force-pushed the feature/try_to_fix_st_epoll_memory_leak branch from 7914759 to af129e2 Compare November 21, 2022 13:32
@winlinvip winlinvip changed the title asan: try to fix st_memory_leak for asan check Asan: try to fix st_memory_leak for asan check Nov 21, 2022
@winlinvip winlinvip changed the title Asan: try to fix st_memory_leak for asan check Asan: Try to fix st_memory_leak for asan check Nov 21, 2022
@winlinvip
Copy link
Member

winlinvip commented Nov 21, 2022

See ossrs/state-threads@843e74b

@winlinvip
Copy link
Member

See 6fa17aa

@winlinvip winlinvip force-pushed the feature/try_to_fix_st_epoll_memory_leak branch from af129e2 to 4c3a86c Compare November 21, 2022 14:32
@winlinvip winlinvip force-pushed the feature/try_to_fix_st_epoll_memory_leak branch from 5680b2c to 90f63b9 Compare November 21, 2022 15:38
@winlinvip winlinvip merged commit 6b130d4 into ossrs:develop Nov 21, 2022
johzzy pushed a commit to johzzy/srs that referenced this pull request Jun 26, 2023
* asan: try to fix st_memory_leak for asan check

* asan: srs_st_unit should be call in hybrid server stop

* Rename st_uninit to st_destroy. v5.0.91

Co-authored-by: winlin <winlin@vip.126.com>
@winlinvip winlinvip added the EnglishNative This issue is conveyed exclusively in English. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
EnglishNative This issue is conveyed exclusively in English.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants