| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
To optimize storage slightly.
|
| |
|
|
|
|
| |
... and rely only on the value already set in Curl_easy.
|
|
|
|
|
| |
As the info is already stored in the transfer handle anyway, there's no
need to carry around a duplicate buffer for the life time of the handle.
|
|
|
|
|
|
| |
... and use 'int' for ports. We don't use 'unsigned short' since -1 is
still often used internally to signify "unknown value" and 0 - 65535 are
all valid port numbers.
|
|
|
|
| |
... as the numerical IP address is already stored and kept in 'primary_ip'.
|
|
|
|
|
|
|
| |
... as it's superfluous now when Curl_easy is passed in and we can
derive the connection from that instead and avoid the duplicate copy.
Closes #6525
|
|
|
|
| |
Closes #6513
|
|
|
|
|
|
| |
... there are a few left but let's keep them to last
Closes #6512
|
| |
|
|
|
|
|
|
|
|
|
|
| |
- Pass the very long request header via file instead of command line.
Prior to this change the 49k very long request header string was passed
via command line and on Windows that is too long so it was truncated and
the test would fail (specifically msys CI).
Closes https://github.com/curl/curl/pull/6516
|
|
|
|
|
|
|
|
| |
Readdir data, filenames and attributes are strictly related to the
transfer and not the connection. This also reduces the total size of the
fixed connectdata struct.
Closes #6519
|
| |
|
|
|
|
| |
Closes #6499
|
|
|
|
| |
Closes #6508
|
|
|
|
|
|
|
| |
... since they disable it by default since a few days back.
Closes #6506
Fixes #6493
|
|
|
|
|
|
|
| |
This also moves the 'async' struct from the connectdata struct into the
Curl_easy struct, which seems like a better home for it.
Closes #6497
|
|
|
|
|
|
|
|
|
| |
On Windows an error number may be greater than INT_MAX and negative once
cast to int.
The assertion is checked only in debug builds.
Closes https://github.com/curl/curl/pull/6504
|
|
|
|
|
|
|
|
| |
... if Curl_doh() returned a NULL, this function gets called anyway as
in a asynch procedure. Then the doh struct pointer is NULL and signifies
an OOM situation.
Follow-up to 6246a1d8c6776
|
|
|
|
|
|
| |
... and repair recent build breakage
Closes #6507
|
|
|
|
|
|
|
|
|
| |
... so mark the connection as closed in this condition to prevent that
verbose message to wrongly appear.
Reported-by: Matt Holt
Bug: https://twitter.com/mholt6/status/1352130240265375744
Closes #6503
|
|
|
|
|
|
| |
... by anchoring more functions on Curl_easy instead of connectdata
Closes #6498
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
- Reorder some internal struct members so that less padding is used.
This is an attempt at saving a bit of space by packing some structs
(using pahole to find the holes) where it might make sense to do
so without losing readability.
I.e., I tried to avoid separating fields that seem grouped
together (like the cwd... fields in struct ftp_conn for instance).
Also abstained from touching fields behind conditional macros as
that quickly can get complicated.
Closes https://github.com/curl/curl/pull/6483
|
|
|
|
| |
Found-by: Marcel Raad
|
|
|
|
|
|
| |
Added test 1613 to verify.
Closes #6490
|
|\ |
|
| | |
|
| |
| |
| |
| | |
Closes #6496
|
|/ |
|
|
|
|
| |
Follow-up to 2bdec0b3
|
|
|
|
| |
Follow-up to 219d9f8620d
|
|
|
|
|
|
|
|
| |
... instead of having it static within the Curl_easy struct. This takes
away 1176 bytes (18%) from the Curl_easy struct that aren't used very
often and instead makes the code allocate it when needed.
Closes #6492
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The SOCKS code now uses the generic download buffer for temporary
storage during the connection procedure, instead of having its own
private 600 byte buffer that adds to the connectdata struct size. This
works fine because this point the buffer is allocated but is not use for
download yet since the connection hasn't completed.
This reduces the connection struct size by 22% on a 64bit arch!
The SOCKS buffer needs to be at least 600 bytes, and the download buffer
is guaranteed to never be smaller than 1000 bytes.
Closes #6491
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
By making the `magic` identifier the same size and at the same place
within the structs (easy, multi, share), libcurl will be able to more
reliably detect and safely error out if an application passes in the
wrong handle to APIs. Easier to detect and less likely to cause crashes
if done.
Such mixups can't be detected at compile-time due to them being
typedefed void pointers - unless `CURL_STRICTER` is defined.
Closes #6484
|
|
|
|
|
|
| |
... and also rename the define for max length.
Closes #6489
|
|
|
|
| |
Closes #6487
|
|
|
|
| |
Closes #6486
|
|
|
|
| |
Closes #6485
|
| |
|
|
|
|
|
| |
At the moment the test fails as curl sends two Proxy-Connection
headers.
|
| |
|
|
|
|
|
|
|
|
| |
Fixes test 1180
Bug: https://curl.se/mail/lib-2021-01/0095.html
Reported-by: Fabian Keil
Closes #6472
|
|
|
|
|
|
|
| |
... with this function provided, we can disable tests for specific
environments and setups directly within this file.
Closes #6477
|
|
|
|
|
| |
... and remove all other variable substitutions as they're now done once
and for all in the preprocessor.
|
|
|
|
|
|
| |
... to make it diff friendlier and easier to read.
Closes #6448
|
|
|
|
| |
Closes #6482
|
|
|
|
| |
Closes #6479
|
|
|
|
| |
Closes #6474
|
|
|
|
|
|
|
|
|
| |
... just as the native code path does. Avoids sending too large data
chunks in the callback and more.
Reported-by: Gisle Vanem
Fixes #6462
Closes #6473
|
| |
|