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

memory leak found by valgrind #106

Closed
chipitsine opened this issue May 24, 2019 · 4 comments
Closed

memory leak found by valgrind #106

chipitsine opened this issue May 24, 2019 · 4 comments
Labels
severity: minor This issue is of MINOR severity. status: fixed This issue is a now-fixed bug. type: bug This issue describes a bug.

Comments

@chipitsine
Copy link
Member 

==540== 24 bytes in 1 blocks are definitely lost in loss record 6 of 341
==540==    at 0x483880B: malloc (vg_replace_malloc.c:309)
==540==    by 0x490AB8A: ??? (in /usr/lib64/libssl.so.1.1.1b)
==540==    by 0x48B9106: __pthread_once_slow (in /usr/lib64/libpthread-2.28.so)
==540==    by 0x4B59D9C: CRYPTO_THREAD_run_once (in /usr/lib64/libcrypto.so.1.1.1b)
==540==    by 0x490CEDA: SSL_COMP_get_compression_methods (in /usr/lib64/libssl.so.1.1.1b)
==540==    by 0x426330: __ssl_sock_init (ssl_sock.c:9721)
==540==    by 0x5596EC: __libc_csu_init (in /builds/chipitsine/haproxy-sanitizer/haproxy)
==540==    by 0x4EAB39D: (below main) (in /usr/lib64/libc-2.28.so)

patch: https://www.mail-archive.com/haproxy@formilux.org/msg33928.html
@chipitsine chipitsine added status: needs-triage This issue needs to be triaged. type: bug This issue describes a bug. labels May 24, 2019
@chipitsine
Copy link
Member Author

it might be backported to earlier branches

@chipitsine
Copy link
Member Author

it's broken on LibreSSL

https://travis-ci.com/chipitsine/haproxy-1/jobs/202992909

will have a look tomorrow

@chipitsine
Copy link
Member Author

corrective patch was sent to list

https://www.mail-archive.com/haproxy@formilux.org/msg33940.html

@chipitsine
Copy link
Member Author

fixed in master

@TimWolla TimWolla added status: fixed This issue is a now-fixed bug. severity: minor This issue is of MINOR severity. and removed status: needs-triage This issue needs to be triaged. labels May 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
severity: minor This issue is of MINOR severity. status: fixed This issue is a now-fixed bug. type: bug This issue describes a bug.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@TimWolla @chipitsine