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

alt-svc torture test fails #4021

Closed
MarcelRaad opened this issue Jun 13, 2019 · 0 comments
Closed

alt-svc torture test fails #4021

MarcelRaad opened this issue Jun 13, 2019 · 0 comments

Comments

@MarcelRaad
Copy link
Member

I did this

CC=gcc-8 ./configure --enable-debug --disable-shared --disable-threaded-resolver --enable-werror --enable-alt-svc --with-libssh2
make
cd tests
make
./runtests.pl -t -n -v 1654

I expected the following

Successful test. Instead, I got:

** MEMORY FAILURE 1 at 10:33:12
Leak detected: memory still allocated: 56 bytes
At 7fffeef464d8, there's 56 bytes.
 allocated by altsvc.c:245
LIMIT altsvc.c:188 malloc reached memlimit
 Failed on function number 1 in test.
 invoke with "-t1" to repeat this single case.

curl/libcurl version

8b987cc

curl 7.65.2-DEV (x86_64-pc-linux-gnu) libcurl/7.65.2-DEV OpenSSL/1.1.0g zlib/1.2.11 brotli/1.0.4 libidn2/2.0.4 libpsl/0.19.1 (+libidn2/2.0.4) libssh2/1.8.0 nghttp2/1.30.0 librtmp/2.3
Release-Date: [unreleased]
Protocols: dict file ftp ftps gopher http https imap imaps ldap ldaps pop3 pop3s rtmp rtsp scp sftp smb smbs smtp smtps telnet tftp
Features: alt-svc brotli Debug HTTP2 HTTPS-proxy IDN IPv6 Largefile libz NTLM NTLM_WB PSL SSL TLS-SRP TrackMemory UnixSockets

operating system

Ubuntu 18.04

bagder added a commit that referenced this issue Jun 13, 2019
... to make it handle torture tests properly.

Reported-by: Marcel Raad
Fixes #4021
@bagder bagder closed this as completed in ff38760 Jun 13, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Sep 11, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging a pull request may close this issue.

1 participant