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

Jump to bottom

Torture tests 2034 and 2037 are failing now that we have stunnel installed on CI #5108

Closed
mback2k opened this issue Mar 15, 2020 · 1 comment
Closed

Torture tests 2034 and 2037 are failing now that we have stunnel installed on CI #5108

mback2k opened this issue Mar 15, 2020 · 1 comment
Assignees
@bagder
Labels
memory-leak tests

Comments

@mback2k
Copy link
Member

mback2k commented Mar 15, 2020

I did this

I added the stunnel tool to our CI builds via ac56683.

The following memory leak occurs:

** MEMORY FAILURE
Leak detected: memory still allocated: 28 bytes
At 5652c038ae68, there's 28 bytes.
 allocated by vtls/vtls.c:119
LIMIT vtls/vtls.c:125 strdup reached memlimit

I expected the following

I expected all existing tests to pass at least on Debian/Ubuntu based builds.

curl/libcurl version

master branch

operating system

Ubuntu:

cc @bagder
@bagder bagder self-assigned this Mar 15, 2020
bagder added a commit that referenced this issue Mar 15, 2020
@bagder
vtls: free ssl_config leftovers on out-of-memory
b7595a4 
Torture testing 2034 and 2037 found this.

Reported-by: Marc Hörsken
Fixes #5108
@bagder bagder mentioned this issue Mar 15, 2020
Closed
@bagder
Copy link
Member

bagder commented Mar 15, 2020

I don't think this is a regression, as in this memory leak seems to have landed here a very long time ago...

@bagder bagder closed this as completed in 2eba5e4 Mar 16, 2020
@lock lock bot locked as resolved and limited conversation to collaborators Jun 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@bagder bagder

Labels
memory-leak tests
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

vtls: free ssl_config leftovers on out-of-memory curl/curl
2 participants
@bagder @mback2k