-
Notifications
You must be signed in to change notification settings - Fork 292
Memory leak #7
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
Comments
Hi @LinuxJedi, Good observation. I am investigating. |
Hi @LinuxJedi, I have made a good improvement on the memory management in general. There isn't a linear growth anymore. I am not closing this issue yet because I want to test with the full OWASP CRS before consider it done. Tomorrow I will continue with the tests. |
Closing this issue after test with OWASP CRS. No leak was identified. |
@zimmerle did your changes for memory management optimization get committed? I've been playing around with this some more and noticing non-trivial leaks that seem to correspond with what @LinuxJedi found. I can provide some more debug info if you need. |
Hi @p0pr0ck5, Yes, those modifications were committed. @LinuxJedi also reported other memory issues here: owasp-modsecurity/ModSecurity#1078 some of those may lead to memory leak. It will be very helpful if you can attach the massif output of your findings on owasp-modsecurity/ModSecurity#1078 |
Silly me, I should have realized that the commit went to libmodsec, not this connector :p I'll do more debugging and ensure a clean build, and toss up some debug info tomorrow. |
…rity from fix/by_address_sanitizer_notes to develop * commit 'ad071ca5f518d98af21d5fc730bbb87f01a0bc3a': Minor fixes for notes from AddressSanitizer
I haven't fully diagnosed the leak yet but I've managed to get some readouts in the attached massif output in the hope it will aid you in finding the route cause.
The leak appears to happen when using this:
As part of the following:
Several KB appear to be leaked on every request which significantly grows over time.
massif.out.11899.zip
The text was updated successfully, but these errors were encountered: