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

Valgrind warnings when accessing https://mail.microsoft.com/ews/exchange.asmx #734

Closed
nikic opened this issue Mar 28, 2016 · 4 comments
Closed

Comments

@nikic
Copy link

nikic commented Mar 28, 2016

I did this

valgrind curl -v https://mail.microsoft.com/ews/exchange.asmx

Output: https://gist.github.com/nikic/b7af10da8f73923a46a0

Are these valgrind warnings expected or do they indicate an issue in libcurl? I assume the first one is one of the "usual" libssl warnings, but there are quite a few following in Curl_http_readwrite_headers which look like they could be valid.

This is coming originally from a PHP bug report: https://bugs.php.net/bug.php?id=71861

curl/libcurl version

curl 7.35.0 (x86_64-pc-linux-gnu) libcurl/7.35.0 OpenSSL/1.0.1f zlib/1.2.8 libidn/1.28 librtmp/2.3
Protocols: dict file ftp ftps gopher http https imap imaps ldap ldaps pop3 pop3s rtmp rtsp smtp smtps telnet tftp 
Features: AsynchDNS GSS-Negotiate IDN IPv6 Largefile NTLM NTLM_WB SSL libz TLS-SRP 

operating system

Ubuntu 14.04

Linux saturn 3.13.0-79-generic #123-Ubuntu SMP Fri Feb 19 14:27:58 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
@nikic
Copy link
Author

nikic commented Mar 28, 2016

Same happens with a 7.48.1-dev build: https://gist.github.com/nikic/d685d44e5ca671ffd049

@bagder
Copy link
Member

bagder commented Mar 28, 2016

This is a known issue with openssl

@bagder
Copy link
Member

bagder commented Mar 28, 2016

I believe the ones below are caused by the first, as the uninitialized values get used further down the line. You can test this theory by building a debug version of openssl that doesn't cause these warnings and see.

@nikic
Copy link
Author

nikic commented Mar 28, 2016

Thanks for the quick response. I wasn't able to test a -DPURIFY build because libtool hates me, but running valgrind with --track-origins=yes shows that all warnings are caused by

==32281==  Uninitialised value was created by a stack allocation
==32281==    at 0x5FC9E77: aesni_cbc_encrypt (in /lib/x86_64-linux-gnu/libcrypto.so.1.0.0)

so clearly not an issue with curl.

@nikic nikic closed this as completed Mar 28, 2016
@lock lock bot locked as resolved and limited conversation to collaborators May 7, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

No branches or pull requests

2 participants