curl / curl / 9676
75%
master: 37%

LAST BUILD BRANCH: patch-1
DEFAULT BRANCH: master
Ran
Jobs 1
Files 108
Run time 15s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
9676

Pull #3448

travis-ci

web-flow 
multi: multiplexing improvements

Fixes #3436

After LOTS of scratching my head, I eventually realized that even when doing
10 uploads in parallel, sometimes the socket callback to the application that
tells it what to wait for on the socket, looked like it would reflect the
status of just the single transfer that just changed state.

Digging into the code revealed that this was indeed the truth. When multiple
transfers are using the same connection, the application did not correctly get
the *combined* flags for all transfers which then could make it switch to READ
(only) when in fact most transfers wanted to get told when the socket was
WRITEABLE.

A separate but related regression had also been introduced by me when I
cleared connection/transfer association better a while ago, as now the logic
couldn't find the connection and see if that was marked as used by more
transfers and then it would also prematurely remove the socket from the socket
hash table even in times other transfers were still using it!

Make sure that each socket stored in the socket hash has a "combined" action
field of what to ask the application to wait for, that is potentially the ORed
action of multiple parallel transfers. And remove that socket hash entry only
if there are no transfers left using it.

The socket hash entry stored an association to a single transfer using that
socket - and when curl_multi_socket_action() was called to tell libcurl about
activities on that specific socket only that transfer was "handled".

This was WRONG, as a single socket/connection can be used by numerous parallel
transfers and not necessarily a single one.

We now store a list of handles in the socket hashtable entry and when libcurl
is told there's traffic for a particular socket, it now iterates over all
known transfers using that single socket.
Pull Request #3448: multi: multiplexing improvements

3602 of 4968 relevant lines covered (72.5%)

778697.57 hits per line

Uncovered Existing Lines

Lines Coverage ∆ File
1
80.68
 -0.05% lib/url.c
2
86.27
 -0.32% lib/smtp.c
Jobs
ID Job ID Ran Files Coverage
23 9676.23 (T=coverage) 273
74.77
 Travis Job 9676.23
Source Files on build 9676