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

Support valgrind for ST #2

Closed
winlinvip opened this issue Sep 3, 2016 · 4 comments
Closed

Support valgrind for ST #2

winlinvip opened this issue Sep 3, 2016 · 4 comments
Labels
feature

Comments

@winlinvip
Copy link
Member

Valgrind is very useful toolkit for memory leak detect and memory corrupt check, but not support ST for the setjmp/longjmp. Maybe we can patch ST to support valgrind.
@winlinvip
Copy link
Member Author

toffaletti already support this feature, we can merge this fork.

winlinvip added a commit that referenced this issue Sep 4, 2016
@winlinvip
for #2, support valgrind, use macro to enable it
4cca7a0
winlinvip referenced this issue in toffaletti/state-threads Sep 4, 2016
@toffaletti
add valgrind support to state-threads
7f57fc9
@winlinvip
Copy link
Member Author

winlinvip commented Sep 4, 2016 •
edited
Loading

We just need to tell valgrind the stack we allocate, by macro VALGRIND_STACK_REGISTER and VALGRIND_STACK_DEREGISTER

The usage: https://github.com/ossrs/state-threads/tree/srs#usage

@winlinvip
Copy link
Member Author

winlinvip commented Sep 4, 2016 •
edited
Loading

For valgrind usage: https://github.com/ossrs/state-threads/tree/srs#valgrind

winlinvip referenced this issue in ossrs/srs Sep 5, 2016
@winlinvip
fix memory leak at handshake. 2.0.213
6ec60b0
winlinvip referenced this issue in ossrs/srs Sep 5, 2016
@winlinvip
fix memory leak at source. 2.0.214
c7b97aa
winlinvip referenced this issue in ossrs/srs Sep 5, 2016
@winlinvip
fix memory leak of source.
590e951
@winlinvip
Copy link
Member Author

winlinvip commented Feb 28, 2017 •
edited
Loading

SRS3 already use state-threads-1.9.1.tar.gz, which is the latest ST, patched to support Valgrind.

@winlinvip winlinvip pinned this issue Oct 17, 2021
@winlinvip winlinvip unpinned this issue Sep 5, 2022
@enochi enochi mentioned this issue Jan 30, 2023
Closed
@DCZYewen DCZYewen mentioned this issue Nov 11, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@winlinvip