-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
memory leak running test #1839
Comments
@weinberggithub do you mean that this problem starts happening since the commit 3b6597b? |
no,I mean my test based on this commit |
@weinberggithub could you try with f10720e? |
httplib.txt |
I'll try to reproduce the problem on my machine. How did you make your |
on macos(M2, apple chips), simply add two line to CMakeLists.txt and enable HTTPLIB-TEST. |
@weinberggithub sorry for the late reply. I just tried your suggestion, but it reports a lot of errors which are not apparently caused by cpp-htpplib. For instance, the following command doesn't run any unit test, but it still reports many errors...
It seems like gtest anyway causes lots of memory leaks, and I am not sure that the leak report for cpp-httplib is really true. In the past, I did test with apache bench and wrk by sending many requests to a server for long time, and made sure that there was no symptom indicating a possibility of memory leak. I did it again now, and I don't see any problem. So I'll close it for now, since I don't see any clear evidence. Could you identify exact lines in httplib.h which are absolutely causing memory leaks with a simple example instead of the unit test, and when you find a problem with the simple problem, please create a new issue. Thanks for your understanding. |
I run the unit test with asan,
ASAN_OPTIONS=detect_leaks=1 ./httplib-test
cpp-httplib: commit of 3b6597b on master branch
os: macos ventura 13.5
compiler: clang-1500.1.0.2.5
below is one of the stack
The text was updated successfully, but these errors were encountered: