- May 23, 2015
-
-
Dan Fandrich authored
-
- May 22, 2015
-
-
Daniel Stenberg authored
By (void) prefixing it and adding a comment. Did some minor related cleanups. Coverity CID 1299423.
-
Daniel Stenberg authored
Coverity CID 1299424 identified dead code because of checks that could never equal true (if the mechanism's name was NULL). Simplified the function by removing a level of pointers and removing the loop and array that weren't used.
-
Daniel Stenberg authored
Replace use of assert with code that properly catches bad input at run-time even in non-debug builds. This flaw was sort of detected by Coverity CID 1299425 which claimed the "case RTSPREQ_NONE" was dead code.
-
Daniel Stenberg authored
A failed calloc() would lead to NULL pointer use. Coverity CID 1299427.
-
Daniel Stenberg authored
non-HTTP proxy implies not using CURLOPT_HTTPPROXYTUNNEL Bug: http://curl.haxx.se/mail/lib-2015-05/0056.html Reported-by: Sean Boudreau
-
Daniel Stenberg authored
Coverity CID 1299428: Dereference after null check (FORWARD_NULL)
-
Daniel Stenberg authored
Coverity CID 1299426 warned about possible NULL dereference otherwise, but that would only ever happen if we get invalid HTTP/2 data with frames for stream 0. Avoid this risk by returning early when stream 0 is used.
-
Daniel Stenberg authored
Follow-up fix from b0143a2a Detected by coverity. CID 1299429
-
Tatsuhiro Tsujikawa authored
This commit just add implicitly opened stream 1 to streams hash.
-
Jay Satiro authored
Prior to this change the description for SEC_E_ILLEGAL_MESSAGE was OS and language specific, and invariably translated to something not very helpful like: "The message received was unexpected or badly formatted." Bug: https://github.com/bagder/curl/issues/267 Reported-by: Michael Osipov
-
Jay Satiro authored
Refer to b0143a2a for more information on the read-callback change.
-
- May 21, 2015
-
-
Daniel Stenberg authored
-
Dan Fandrich authored
This way, the build directory can be located on an entirely different filesystem from the source code (e.g. a tmpfs).
-
- May 20, 2015
-
-
Daniel Stenberg authored
With many easy handles using the same connection for multiplexing, it is important we store and keep the transfer-oriented stuff in the SessionHandle so that callbacks and callback data work fine even when many easy handles share the same physical connection.
-
Daniel Stenberg authored
It makes them easier to match output from the nghttpd test server.
-
Tatsuhiro Tsujikawa authored
Previously, when we send all given buffer in data_source_callback, we return NGHTTP2_ERR_DEFERRED, and nghttp2 library removes this stream temporarily for writing. This itself is good. If this is the sole stream in the session, nghttp2_session_want_write() returns zero, which means that libcurl does not check writeability of the underlying socket. This leads to very slow upload, because it seems curl only upload 16k something per 1 second. To fix this, if we still have data to send, call nghttp2_session_resume_data after nghttp2_session_send. This makes nghttp2_session_want_write() returns nonzero (if connection window still opens), and as a result, socket writeability is checked, and upload speed becomes normal.
-
Dmitry Eremin-Solenikov authored
Stop curl from failing when non-fatal alert is received during handshake. This e.g. fixes lots of problems when working with https sites through proxies.
-
Daniel Stenberg authored
Reported-by: bsammon Bug: https://github.com/bagder/curl/issues/282
-
Jay Satiro authored
.. also correct some variable naming in curl_easy_escape.3 Bug: https://github.com/bagder/curl/issues/281 Reported-by: <bsammon@users.noreply.github.com>
-
- May 19, 2015
-
-
Brian Prodoehl authored
BoringSSL removed support for direct callers of SSL_CTX_callback_ctrl and SSL_CTX_ctrl, so move to a way that should work on BoringSSL and OpenSSL. re #275
-
Jay Satiro authored
-
Daniel Stenberg authored
-
Kamil Dudka authored
Error: CLANG_WARNING: lib/http.c:173:16: warning: Value stored to 'http' during its initialization is never read Error: COMPILER_WARNING: lib/http.c: scope_hint: In function ‘http_disconnect’ lib/http.c:173:16: warning: unused variable ‘http’ [-Wunused-variable]
-
Jay Satiro authored
.. also make __func__ replacement in multi. Prior to this change debug builds would fail to build if the compiler was building pre-c99 and didn't support __func__.
-
Viktor Szakats authored
-
Daniel Stenberg authored
-
Jay Satiro authored
-
- May 18, 2015
-
-
Daniel Stenberg authored
Use a curl_off_t for upload left
-
Daniel Stenberg authored
-
Daniel Stenberg authored
as after 4883f701, the *_clean() function only flushes the hash.
-
Daniel Stenberg authored
As it breaks configure, curl-config and test 1023 if not.
-
Anthony Avina authored
.. and added unit1602 for hash.c
-
Daniel Stenberg authored
-
Daniel Stenberg authored
-
Tatsuhiro Tsujikawa authored
We could get stream ID not in the hash in on_stream_close. For example, if we decided to reject stream (e.g., PUSH_PROMISE), then we don't create stream and store it in hash with its stream ID.
-
Tatsuhiro Tsujikawa authored
This commit requires nghttp2 v1.0.0 to compile, and migrate to v1.0.0, and utilize recent version of nghttp2 to simplify the code, First we use nghttp2_option_set_no_recv_client_magic function to detect nghttp2 v1.0.0. That function only exists since v1.0.0. Since nghttp2 v0.7.5, nghttp2 ensures header field ordering, and validates received header field. If it found error, RST_STREAM with PROTOCOL_ERROR is issued. Since we require v1.0.0, we can utilize this feature to simplify libcurl code. This commit does this. Migration from 0.7 series are done based on nghttp2 migration document. For libcurl, we removed the code sending first 24 bytes client magic. It is now done by nghttp2 library. on_invalid_frame_recv callback signature changed, and is updated accordingly.
-
Daniel Stenberg authored
-
Daniel Stenberg authored
... to "compartmentalize" a bit and make it easier to change behavior when multiplexing is used instead of good old pipelining.
-
Daniel Stenberg authored
-