]> granicus.if.org Git - curl/log
curl
9 years agocurl_setup: Add macros for FOPEN_READTEXT, FOPEN_WRITETEXT
Jay Satiro [Mon, 1 Jun 2015 07:20:18 +0000 (03:20 -0400)]
curl_setup: Add macros for FOPEN_READTEXT, FOPEN_WRITETEXT

- Change fopen calls to use FOPEN_READTEXT instead of "r" or "rt"
- Change fopen calls to use FOPEN_WRITETEXT instead of "w" or "wt"

This change is to explicitly specify when we need to read/write text.
Unfortunately 't' is not part of POSIX fopen so we can't specify it
directly. Instead we now have FOPEN_READTEXT, FOPEN_WRITETEXT.

Prior to this change we had an issue on Windows if an application that
uses libcurl overrides the default file mode to binary. The default file
mode in Windows is normally text mode (translation mode) and that's what
libcurl expects.

Bug: https://github.com/bagder/curl/pull/258#issuecomment-107093055
Reported-by: Orgad Shaneh
9 years agohttp2-upload.c: use PIPEWAIT for playing HTTP/2 better
Daniel Stenberg [Mon, 1 Jun 2015 06:46:18 +0000 (08:46 +0200)]
http2-upload.c: use PIPEWAIT for playing HTTP/2 better

9 years agohttp2-download: check for CURLPIPE_MULTIPLEX properly
Daniel Stenberg [Mon, 1 Jun 2015 06:45:30 +0000 (08:45 +0200)]
http2-download: check for CURLPIPE_MULTIPLEX properly

Bug: http://curl.haxx.se/mail/lib-2015-06/0001.html
Reported-by: Rafayel Mkrtchyan
9 years agoHTTP-NTLM: fail auth on connection close instead of looping
Isaac Boukris [Sun, 31 May 2015 21:21:15 +0000 (23:21 +0200)]
HTTP-NTLM: fail auth on connection close instead of looping

Bug: https://github.com/bagder/curl/issues/256

9 years ago5.6 Refuse "downgrade" redirects
Daniel Stenberg [Sat, 30 May 2015 22:39:19 +0000 (00:39 +0200)]
5.6 Refuse "downgrade" redirects

9 years agoREADME.pingpong: removed
Daniel Stenberg [Sat, 30 May 2015 22:20:22 +0000 (00:20 +0200)]
README.pingpong: removed

9 years agoROADMAP: remove HTTP/2 multiplexing - its here now
Daniel Stenberg [Sat, 30 May 2015 10:09:22 +0000 (12:09 +0200)]
ROADMAP: remove HTTP/2 multiplexing - its here now

9 years agoHTTP2.md: formatted properly
Daniel Stenberg [Sat, 30 May 2015 10:07:39 +0000 (12:07 +0200)]
HTTP2.md: formatted properly

9 years agoHTTP2: moved docs into docs/ and make it markdown
Daniel Stenberg [Sat, 30 May 2015 09:55:33 +0000 (11:55 +0200)]
HTTP2: moved docs into docs/ and make it markdown

9 years agoREADME.http2: refreshed and added multiplexing info
Daniel Stenberg [Sat, 30 May 2015 09:53:24 +0000 (11:53 +0200)]
README.http2: refreshed and added multiplexing info

9 years agodist: add the http2 examples
Daniel Stenberg [Thu, 28 May 2015 14:04:13 +0000 (16:04 +0200)]
dist: add the http2 examples

9 years agohttp2 examples: clean up some comments
Daniel Stenberg [Thu, 28 May 2015 14:04:00 +0000 (16:04 +0200)]
http2 examples: clean up some comments

9 years agoexamples: added two programs doing multiplexed HTTP/2
Daniel Stenberg [Thu, 28 May 2015 13:58:34 +0000 (15:58 +0200)]
examples: added two programs doing multiplexed HTTP/2

9 years agoscripts: moved contributors.sh and contrithanks.sh into subdir
Daniel Stenberg [Wed, 27 May 2015 14:10:08 +0000 (16:10 +0200)]
scripts: moved contributors.sh and contrithanks.sh into subdir

9 years agoRELEASE-NOTES: synced with c005790ff1c0a
Daniel Stenberg [Wed, 27 May 2015 12:48:59 +0000 (14:48 +0200)]
RELEASE-NOTES: synced with c005790ff1c0a

9 years agoopenssl: typo in comment
Daniel Melani [Wed, 27 May 2015 09:27:03 +0000 (11:27 +0200)]
openssl: typo in comment

9 years agoopenssl: Use TLS_client_method for OpenSSL 1.1.0+
Jay Satiro [Wed, 27 May 2015 05:29:16 +0000 (01:29 -0400)]
openssl: Use TLS_client_method for OpenSSL 1.1.0+

SSLv23_client_method is deprecated starting in OpenSSL 1.1.0. The
equivalent is TLS_client_method.

https://github.com/openssl/openssl/commit/13c9bb3#diff-708d3ae0f2c2973b272b811315381557

9 years agoFAQ: How do I port libcurl to my OS?
Daniel Stenberg [Tue, 26 May 2015 17:13:09 +0000 (19:13 +0200)]
FAQ: How do I port libcurl to my OS?

9 years agoCURLOPT_COOKIELIST.3: Explain Set-Cookie without a domain
Jay Satiro [Mon, 25 May 2015 21:27:53 +0000 (17:27 -0400)]
CURLOPT_COOKIELIST.3: Explain Set-Cookie without a domain

Document that if Set-Cookie is used without a domain then the cookie is
sent for any domain and will not be modified.

Bug: http://curl.haxx.se/mail/lib-2015-05/0137.html
Reported-by: Alexander Dyagilev
9 years agohttp2: Copy data passed in Curl_http2_switched into HTTP/2 connection buffer
Tatsuhiro Tsujikawa [Mon, 25 May 2015 15:10:05 +0000 (00:10 +0900)]
http2: Copy data passed in Curl_http2_switched into HTTP/2 connection buffer

Previously, after seeing upgrade to HTTP/2, we feed data followed by
upgrade response headers directly to nghttp2_session_mem_recv() in
Curl_http2_switched().  But it turns out that passed buffer, mem, is
part of stream->mem, and callbacks called by
nghttp2_session_mem_recv() will write stream specific data into
stream->mem, overwriting input data.  This will corrupt input, and
most likely frame length error is detected by nghttp2 library.  The
fix is first copy the passed data to HTTP/2 connection buffer,
httpc->inbuf, and call nghttp2_session_mem_recv().

9 years agoCURLOPT_COOKIE.3: Explain that the cookies won't be modified
Jay Satiro [Sun, 24 May 2015 23:01:13 +0000 (19:01 -0400)]
CURLOPT_COOKIE.3: Explain that the cookies won't be modified

The CURLOPT_COOKIE doc says it "sets the cookie header explicitly in the
outgoing request(s)." However there seems to be some user confusion
about cookie modification. Document that the cookies set by this option
are not modified by the cookie engine.

Bug: http://curl.haxx.se/mail/lib-2015-05/0115.html
Reported-by: Alexander Dyagilev
9 years agoCURLOPT_COOKIELIST.3: Add example
Jay Satiro [Sun, 24 May 2015 19:39:28 +0000 (15:39 -0400)]
CURLOPT_COOKIELIST.3: Add example

9 years agotestcurl.pl: use rel2abs to make the source directory absolute
Dan Fandrich [Sun, 24 May 2015 10:02:30 +0000 (12:02 +0200)]
testcurl.pl: use rel2abs to make the source directory absolute

This function makes a platform-specific absolute path which uses
backslashes on Windows. This form works when passing it on the
command-line, as well as if the source is on another drive.

9 years agoconncache: fixed memory leak on OOM (torture tests)
Dan Fandrich [Sun, 24 May 2015 09:19:07 +0000 (11:19 +0200)]
conncache: fixed memory leak on OOM (torture tests)

9 years agoperl: remove subdir, not touched in 9 years
Daniel Stenberg [Sat, 23 May 2015 22:54:55 +0000 (00:54 +0200)]
perl: remove subdir, not touched in 9 years

9 years agolog2changes.pl: moved to scripts/
Daniel Stenberg [Sat, 23 May 2015 22:09:23 +0000 (00:09 +0200)]
log2changes.pl: moved to scripts/

9 years agoscripts: add zsh.pl for generating zsh completion
Alessandro Ghedini [Thu, 7 May 2015 14:07:24 +0000 (16:07 +0200)]
scripts: add zsh.pl for generating zsh completion

9 years agotest1510: another flaky test
Dan Fandrich [Sat, 23 May 2015 14:27:49 +0000 (16:27 +0200)]
test1510: another flaky test

9 years agosecurity: fix "Unchecked return value" from sscanf()
Daniel Stenberg [Fri, 22 May 2015 14:52:03 +0000 (16:52 +0200)]
security: fix "Unchecked return value" from sscanf()

By (void) prefixing it and adding a comment. Did some minor related
cleanups.

Coverity CID 1299423.

9 years agosecurity: simplify choose_mech
Daniel Stenberg [Fri, 22 May 2015 14:43:58 +0000 (16:43 +0200)]
security: simplify choose_mech

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.

9 years agoRTSP: catch attempted unsupported requests better
Daniel Stenberg [Fri, 22 May 2015 14:32:42 +0000 (16:32 +0200)]
RTSP: catch attempted unsupported requests better

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.

9 years agoshare_init: fix OOM crash
Daniel Stenberg [Fri, 22 May 2015 14:26:14 +0000 (16:26 +0200)]
share_init: fix OOM crash

A failed calloc() would lead to NULL pointer use.

Coverity CID 1299427.

9 years agoparse_proxy: switch off tunneling if non-HTTP proxy
Daniel Stenberg [Fri, 22 May 2015 07:06:44 +0000 (09:06 +0200)]
parse_proxy: switch off tunneling if non-HTTP proxy

non-HTTP proxy implies not using CURLOPT_HTTPPROXYTUNNEL

Bug: http://curl.haxx.se/mail/lib-2015-05/0056.html
Reported-by: Sean Boudreau
9 years agocurl: fix potential NULL dereference
Daniel Stenberg [Fri, 22 May 2015 14:18:36 +0000 (16:18 +0200)]
curl: fix potential NULL dereference

Coverity CID 1299428: Dereference after null check (FORWARD_NULL)

9 years agohttp2: on_frame_recv: return early on stream 0
Daniel Stenberg [Fri, 22 May 2015 13:17:16 +0000 (15:17 +0200)]
http2: on_frame_recv: return early on stream 0

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.

9 years agohttp: removed self assignment
Daniel Stenberg [Fri, 22 May 2015 07:57:16 +0000 (09:57 +0200)]
http: removed self assignment

Follow-up fix from b0143a2a33f0

Detected by coverity. CID 1299429

9 years agohttp2: Make HTTP Upgrade work
Tatsuhiro Tsujikawa [Thu, 21 May 2015 16:21:59 +0000 (01:21 +0900)]
http2: Make HTTP Upgrade work

This commit just add implicitly opened stream 1 to streams hash.

9 years agostrerror: Change SEC_E_ILLEGAL_MESSAGE description
Jay Satiro [Fri, 22 May 2015 06:30:38 +0000 (02:30 -0400)]
strerror: Change SEC_E_ILLEGAL_MESSAGE description

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
9 years agotelnet: Fix read-callback change for Windows builds
Jay Satiro [Fri, 22 May 2015 03:26:32 +0000 (23:26 -0400)]
telnet: Fix read-callback change for Windows builds

Refer to b0143a2 for more information on the read-callback change.

9 years agoCURLOPT_HTTPPROXYTUNNEL.3: only works with a HTTP proxy!
Daniel Stenberg [Thu, 21 May 2015 12:17:17 +0000 (14:17 +0200)]
CURLOPT_HTTPPROXYTUNNEL.3: only works with a HTTP proxy!

9 years agotestcurl.pl: allow source to be in an arbitrary directory
Dan Fandrich [Thu, 21 May 2015 07:10:31 +0000 (09:10 +0200)]
testcurl.pl: allow source to be in an arbitrary directory

This way, the build directory can be located on an entirely different
filesystem from the source code (e.g. a tmpfs).

9 years agoread_callback: move to SessionHandle from connectdata
Daniel Stenberg [Wed, 20 May 2015 12:33:04 +0000 (14:33 +0200)]
read_callback: move to SessionHandle from connectdata

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.

9 years agohttp2: show stream IDs in decimal
Daniel Stenberg [Wed, 20 May 2015 12:27:08 +0000 (14:27 +0200)]
http2: show stream IDs in decimal

It makes them easier to match output from the nghttpd test server.

9 years agohttp2: Faster http2 upload
Tatsuhiro Tsujikawa [Wed, 20 May 2015 14:11:43 +0000 (23:11 +0900)]
http2: Faster http2 upload

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.

9 years agogtls: don't fail on non-fatal alerts during handshake
Dmitry Eremin-Solenikov [Wed, 20 May 2015 19:50:55 +0000 (22:50 +0300)]
gtls: don't fail on non-fatal alerts during handshake

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.

9 years agocurl_easy_unescape.3: update RFC reference
Daniel Stenberg [Wed, 20 May 2015 06:21:27 +0000 (08:21 +0200)]
curl_easy_unescape.3: update RFC reference

Reported-by: bsammon
Bug: https://github.com/bagder/curl/issues/282

9 years agoCURLOPT_POSTFIELDS.3: Mention curl_easy_escape
Jay Satiro [Wed, 20 May 2015 05:48:31 +0000 (01:48 -0400)]
CURLOPT_POSTFIELDS.3: Mention curl_easy_escape

.. 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
9 years agoopenssl: Use SSL_CTX_set_msg_callback and SSL_CTX_set_msg_callback_arg
Brian Prodoehl [Tue, 19 May 2015 15:10:28 +0000 (11:10 -0400)]
openssl: Use SSL_CTX_set_msg_callback and SSL_CTX_set_msg_callback_arg

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

9 years agocurl.1: fix missing space in section --data
Jay Satiro [Tue, 19 May 2015 19:43:11 +0000 (15:43 -0400)]
curl.1: fix missing space in section --data

9 years agotransfer: remove erroneous and misleading comment
Daniel Stenberg [Tue, 19 May 2015 11:58:13 +0000 (13:58 +0200)]
transfer: remove erroneous and misleading comment

9 years agohttp: silence compile-time warnings without USE_NGHTTP2
Kamil Dudka [Tue, 19 May 2015 10:51:40 +0000 (12:51 +0200)]
http: silence compile-time warnings without USE_NGHTTP2

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]

9 years agotransfer: Replace __func__ instances with function name
Jay Satiro [Tue, 19 May 2015 06:23:55 +0000 (02:23 -0400)]
transfer: Replace __func__ instances with function name

.. 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__.

9 years agobuild: bump version in default nghttp2 paths
Viktor Szakats [Mon, 18 May 2015 23:42:29 +0000 (01:42 +0200)]
build: bump version in default nghttp2 paths

9 years agoINTERNALS: we require nghttp2 1.0.0+ now
Daniel Stenberg [Tue, 19 May 2015 05:41:01 +0000 (07:41 +0200)]
INTERNALS: we require nghttp2 1.0.0+ now

9 years agohttp: Add some include guards for the new HTTP/2 stuff
Jay Satiro [Tue, 19 May 2015 00:53:58 +0000 (20:53 -0400)]
http: Add some include guards for the new HTTP/2 stuff

9 years agohttp2: store upload state per stream
Daniel Stenberg [Mon, 18 May 2015 12:01:38 +0000 (14:01 +0200)]
http2: store upload state per stream

Use a curl_off_t for upload left

9 years agohttp2: fix build when NOT h2-enabled
Daniel Stenberg [Mon, 18 May 2015 12:09:32 +0000 (14:09 +0200)]
http2: fix build when NOT h2-enabled

9 years agohttp2: switch to use Curl_hash_destroy()
Daniel Stenberg [Mon, 18 May 2015 09:41:16 +0000 (11:41 +0200)]
http2: switch to use Curl_hash_destroy()

as after 4883f7019d3, the *_clean() function only flushes the hash.

9 years agocurlver: restore LIBCURL_VERSION_NUM defined as a full number
Daniel Stenberg [Mon, 18 May 2015 09:28:44 +0000 (11:28 +0200)]
curlver: restore LIBCURL_VERSION_NUM defined as a full number

As it breaks configure, curl-config and test 1023 if not.

9 years agohostip: fix unintended destruction of hash table
Anthony Avina [Sat, 2 May 2015 18:49:55 +0000 (13:49 -0500)]
hostip: fix unintended destruction of hash table

.. and added unit1602 for hash.c

9 years agocurlver: introducing new version number (checking) macros
Daniel Stenberg [Thu, 14 May 2015 21:33:27 +0000 (23:33 +0200)]
curlver: introducing new version number (checking) macros

9 years agoruntests.pl: use 'h2c' now, no -14 anymore
Daniel Stenberg [Mon, 18 May 2015 08:10:55 +0000 (10:10 +0200)]
runtests.pl: use 'h2c' now, no -14 anymore

9 years agohttp2: Ignore if we have stream ID not in hash in on_stream_close
Tatsuhiro Tsujikawa [Sat, 16 May 2015 09:13:10 +0000 (18:13 +0900)]
http2: Ignore if we have stream ID not in hash in on_stream_close

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.

9 years agoRequire nghttp2 v1.0.0
Tatsuhiro Tsujikawa [Sat, 16 May 2015 09:03:47 +0000 (18:03 +0900)]
Require nghttp2 v1.0.0

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.

9 years agohttp2: infof length in on_frame_send()
Daniel Stenberg [Thu, 14 May 2015 12:02:21 +0000 (14:02 +0200)]
http2: infof length in on_frame_send()

9 years agopipeline: switch some code over to functions
Daniel Stenberg [Wed, 13 May 2015 12:24:30 +0000 (14:24 +0200)]
pipeline: switch some code over to functions

... to "compartmentalize" a bit and make it easier to change behavior
when multiplexing is used instead of good old pipelining.

9 years agosymbols-in-versions: add CURLOPT_PIPEWAIT
Daniel Stenberg [Tue, 12 May 2015 13:06:18 +0000 (15:06 +0200)]
symbols-in-versions: add CURLOPT_PIPEWAIT

9 years agoCURLOPT_PIPEWAIT: added
Daniel Stenberg [Tue, 12 May 2015 12:18:46 +0000 (14:18 +0200)]
CURLOPT_PIPEWAIT: added

By setting this option to 1 libcurl will wait for a connection to reveal
if it is possible to pipeline/multiplex on before it continues.

9 years agoCurl_http_readwrite_headers: minor code simplification
Daniel Stenberg [Tue, 12 May 2015 09:47:33 +0000 (11:47 +0200)]
Curl_http_readwrite_headers: minor code simplification

9 years agoIsPipeliningPossible: fixed for http2
Daniel Stenberg [Tue, 12 May 2015 09:46:58 +0000 (11:46 +0200)]
IsPipeliningPossible: fixed for http2

9 years agohttp2: bump the h2 buffer size to 32K for speed
Daniel Stenberg [Mon, 11 May 2015 21:18:21 +0000 (23:18 +0200)]
http2: bump the h2 buffer size to 32K for speed

9 years agohttp2: remove the stream from the hash in stream_close callback
Daniel Stenberg [Mon, 11 May 2015 21:17:36 +0000 (23:17 +0200)]
http2: remove the stream from the hash in stream_close callback

... and suddenly things work much better!

9 years agohttp2: if there is paused data, do not clear the drain field
Daniel Stenberg [Mon, 11 May 2015 13:54:50 +0000 (15:54 +0200)]
http2: if there is paused data, do not clear the drain field

9 years agohttp2: rename s/data/pausedata
Daniel Stenberg [Mon, 11 May 2015 13:10:21 +0000 (15:10 +0200)]
http2: rename s/data/pausedata

9 years agohttp2: "stream %x" in all outputs to make it easier to search for
Daniel Stenberg [Mon, 11 May 2015 12:30:36 +0000 (14:30 +0200)]
http2: "stream %x" in all outputs to make it easier to search for

9 years agohttp2: Curl_expire() all handles with incoming traffic
Daniel Stenberg [Mon, 11 May 2015 12:18:53 +0000 (14:18 +0200)]
http2: Curl_expire() all handles with incoming traffic

... so that they'll get handled next in the multi loop.

9 years agohttp2: don't signal settings change for same values
Daniel Stenberg [Mon, 11 May 2015 10:05:14 +0000 (12:05 +0200)]
http2: don't signal settings change for same values

9 years agohttp2: set default concurrency, fix ConnectionExists for multiplex
Daniel Stenberg [Mon, 11 May 2015 09:41:10 +0000 (11:41 +0200)]
http2: set default concurrency, fix ConnectionExists for multiplex

9 years agobundles: store no/default/pipeline/multiplex
Daniel Stenberg [Mon, 18 May 2015 06:56:29 +0000 (08:56 +0200)]
bundles: store no/default/pipeline/multiplex

to allow code to act differently on the situation.

Also added some more info message for the connection re-use function to
make it clearer when connections are not re-used.

9 years agohttp2: lazy init header_recvbuf
Daniel Stenberg [Fri, 8 May 2015 13:53:18 +0000 (15:53 +0200)]
http2: lazy init header_recvbuf

It makes us use less memory when not doing HTTP/2 and subsequently also
makes us not have to cleanup HTTP/2 related data when not using HTTP/2!

9 years agohttp2: separate multiplex/pipelining + cleanup memory leaks
Daniel Stenberg [Fri, 8 May 2015 12:42:59 +0000 (14:42 +0200)]
http2: separate multiplex/pipelining + cleanup memory leaks

9 years agoCURLMOPT_PIPELINE: bit 1 is for multiplexing
Daniel Stenberg [Fri, 8 May 2015 08:43:36 +0000 (10:43 +0200)]
CURLMOPT_PIPELINE: bit 1 is for multiplexing

9 years agohttp2: Fix bug that data to be drained are overwritten by pending "paused" data
Tatsuhiro Tsujikawa [Thu, 7 May 2015 14:19:08 +0000 (23:19 +0900)]
http2: Fix bug that data to be drained are overwritten by pending "paused" data

9 years agohttp2: Don't call nghttp2_session_mem_recv while it is paused by a stream
Tatsuhiro Tsujikawa [Thu, 7 May 2015 15:52:48 +0000 (17:52 +0200)]
http2: Don't call nghttp2_session_mem_recv while it is paused by a stream

9 years agohttp2: Read data left in connection buffer after pause
Tatsuhiro Tsujikawa [Thu, 7 May 2015 08:22:57 +0000 (17:22 +0900)]
http2: Read data left in connection buffer after pause

Previously when we do pause because of out of buffer, we just throw
away unread data in connection buffer.  This just broke protocol
framing, and I saw occasional FRAME_SIZE_ERROR.  This commit fix this
issue by remembering how much data read, and in the next iteration, we
process remaining data.

9 years agohttp2: Fix streams get stuck
Tatsuhiro Tsujikawa [Thu, 7 May 2015 05:51:32 +0000 (14:51 +0900)]
http2: Fix streams get stuck

This commit fixes the bug that streams get stuck if stream gets some
DATA, and stream->closed becomes true at the same time.  Previously,
in this condition, after we processed DATA, we are going to try to
read data from underlying transport, but there is no data, and gets
EAGAIN.  There was no code path to evaludate stream->closed.

9 years agohttp2: store incoming h2 SETTINGS
Daniel Stenberg [Thu, 7 May 2015 14:44:00 +0000 (16:44 +0200)]
http2: store incoming h2 SETTINGS

9 years agopipeline: move function to pipeline.c and make static
Daniel Stenberg [Thu, 7 May 2015 07:00:20 +0000 (09:00 +0200)]
pipeline: move function to pipeline.c and make static

... as it was only used from there.

9 years agoIsPipeliningPossible: http2 can always "pipeline" (multiplex)
Daniel Stenberg [Thu, 7 May 2015 06:59:28 +0000 (08:59 +0200)]
IsPipeliningPossible: http2 can always "pipeline" (multiplex)

9 years agohttp2: remove debug logging from on_frame_recv
Daniel Stenberg [Mon, 4 May 2015 12:03:29 +0000 (14:03 +0200)]
http2: remove debug logging from on_frame_recv

9 years agohttp2: remove the closed check in http2_recv
Daniel Stenberg [Mon, 4 May 2015 08:14:42 +0000 (10:14 +0200)]
http2: remove the closed check in http2_recv

With the "drained" functionality we can get here slightly asynchronously
so the stream have have been closed but there is pending data left to
read.

9 years agohttp2: bump the h2 buffer to 8K
Daniel Stenberg [Mon, 4 May 2015 08:14:27 +0000 (10:14 +0200)]
http2: bump the h2 buffer to 8K

9 years agohttp2: Curl_read should not use the single buffer
Daniel Stenberg [Mon, 4 May 2015 08:03:07 +0000 (10:03 +0200)]
http2: Curl_read should not use the single buffer

... as it does for pipelining when we're multiplexing, as we need the
different buffers to store incoming data correctly for all streams.

9 years agohttp2: more debug outputs
Daniel Stenberg [Thu, 30 Apr 2015 16:34:41 +0000 (18:34 +0200)]
http2: more debug outputs

9 years agohttp2: leave WAITPERFORM when conn is multiplexed
Daniel Stenberg [Thu, 30 Apr 2015 13:25:06 +0000 (15:25 +0200)]
http2: leave WAITPERFORM when conn is multiplexed

No need to wait for our "spot" like for pipelining

9 years agohttp2: force "drainage" of streams
Daniel Stenberg [Thu, 30 Apr 2015 06:20:49 +0000 (08:20 +0200)]
http2: force "drainage" of streams

... which is necessary since the socket won't be readable but there is
data waiting in the buffer.

9 years agohttp2: move the mem+len pair to the stream struct
Daniel Stenberg [Wed, 29 Apr 2015 15:00:53 +0000 (17:00 +0200)]
http2: move the mem+len pair to the stream struct

9 years agohttp2: more stream-oriented data, stream ID 0 is for connections
Daniel Stenberg [Wed, 29 Apr 2015 13:21:45 +0000 (15:21 +0200)]
http2: more stream-oriented data, stream ID 0 is for connections

9 years agohttp2: move lots of state data to the 'stream' struct
Daniel Stenberg [Wed, 29 Apr 2015 12:19:39 +0000 (14:19 +0200)]
http2: move lots of state data to the 'stream' struct

... from the connection struct. The stream one being the 'struct HTTP'
which is kept in the SessionHandle struct (easy handle).

lookup streams for incoming frames in the stream hash, hashing is based
on the stream id and we get the SessionHandle for the incoming stream
that way.

9 years agoHTTP: partial start at fixing up hash-lookups on http2 frame receival
Daniel Stenberg [Tue, 28 Apr 2015 18:39:47 +0000 (20:39 +0200)]
HTTP: partial start at fixing up hash-lookups on http2 frame receival