| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
| |
My Watt-32 tcp/ip stack works on Windows but it does not have `WSAIoctl()`
|
|
|
|
|
|
| |
Added test 3020 to verify
Closes #7183
|
|
|
|
|
|
|
| |
They're too similar to the gcc-9 ones to be useful (and seems to not
work anymore).
Closes #7187
|
|
|
|
|
|
|
| |
--ignore-content-length / CURLOPT_IGNORE_CONTENT_LENGTH doesn't work
with hyper.
Closes #7184
|
|
|
|
|
|
|
|
|
|
|
| |
The 'hyper mode' makes line-ending checks work in the test suite for
when hyper is used. Now it also requires that HTTP or HTTPS are
mentioned as keywords to be enabled so that it doesn't wrongly adjusts
tests for other protocols.
This makes test 271 (TFTP) work again in hyper enabled builds.
Closes #7185
|
|
|
|
|
|
| |
Added test 3019
Fixes #7170
Closes #7174
|
|
|
|
|
| |
Fix a typo in the sorting comment, and while in there elaborate slightly
on why creationtime can be used as a tiebreaker.
|
|
|
|
|
|
|
|
|
| |
Commit 1c1d9f1affbd3367bcb24062e261d0ea5d185e3a removed the last use
for the inet_pton.h headerfile, this removes the inclusion of the
header.
Closes: #7182
Reviewed-by: Daniel Stenberg <daniel@haxx.se>
|
|
|
|
|
|
| |
This reverts commit c769d1eab4de8b9f1bd84d992c63692fdc43c5be.
See #7144 for details
|
|
|
|
|
|
|
|
|
|
|
| |
Fixes potential hang in accept by using select + non-blocking accept.
Fixes potential hang in peer check by replacing the send/recv check with
a getsockname/getpeername check.
Adds length check for returned sockaddr data.
Closes #7144
|
|
|
|
|
|
|
|
|
|
|
| |
The warning about missing entries in that file then doesn't require that
the Makefile has been regenerated which was confusing.
The scan for the test num is a little more error prone than before
(since now it doesn't actually verify that it is legitimate Makefile
syntax), but I think it is good enough.
Closes #7177
|
|
|
|
| |
Closes #7179
|
|
|
|
|
|
|
|
|
| |
For options that pass in lists or strings that are subsequently parsed
and must be correct. This broadens the scope for the option previously
known as CURLE_TELNET_OPTION_SYNTAX but the old name is of course still
provided as a #define for existing applications.
Closes #7175
|
|
|
|
|
|
|
|
|
|
| |
The previous strip also removed the CR which turned problematic.
valgrind.supp: add zstd suppression using hyper
Reported-and-analyzed-by: Kevin Burke
Fixes #7169
Closes #7171
|
|
|
|
|
| |
Assisted-by: Marc Hoersken
Closes #7173
|
|
|
|
|
| |
Add test 3017 and 3018 to verify.
Closes #7166
|
|
|
|
| |
Closes #7028
|
|
|
|
| |
Closes #7028
|
|
|
|
|
|
|
| |
This flag will be further exposed by adding build options.
Reverts #6809
Closes #7028
|
| |
|
|
|
|
|
| |
Reviewed-by: Daniel Stenberg
Closes #7167
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Follow-up to 1a0ebf6632f889eed
- Check the return code to Curl_inet_pton() in two instances, even
though we know the input is valid so the functions won't fail.
- Clear the 'struct sockaddr_in' struct before use so that the
'sin_zero' field isn't left uninitialized.
Detected by Coverity.
Assisted-by: Harry Sintonen
Closes #7163
|
|
|
|
| |
Closes #7154
|
|
|
|
|
|
|
|
|
|
| |
As host names are case insensitive, the use of case sensitive hashing
caused unnecesary cache misses and therefore lost performance. This
lowercases the hash key.
Reported-by: Harry Sintonen
Fixes #7159
Closes #7161
|
|
|
|
|
|
|
| |
If the function doesn't exist, provide a macro that just clears the
error message. Removes #ifdef uses from the code.
Closes #7162
|
|
|
|
|
|
|
|
| |
Follow-up to b249592d29ae0
Avoids NULL pointer derefs.
Closes #7165
|
|
|
|
| |
Closes #7164
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This avoids a TCP reset (RST) if the server initiates a connection
shutdown by sending an SSL close notify alert and then closes the TCP
connection.
For SSL connections, usually the server announces that it will close the
connection with an SSL close notify alert. curl should read this alert.
If curl does not read this alert and just closes the connection, some
operating systems close the TCP connection with an RST flag.
See RFC 1122, section 4.2.2.13
If curl reads the close notify alert, the TCP connection is closed
normally with a FIN flag.
The new code is similar to existing code in the "SSL shutdown" function:
try to read an alert (non-blocking), and ignore any read errors.
Closes #7095
|
|
|
|
| |
Closes #7157
|
|
|
|
|
|
|
| |
CURLOPT_SSLCERT_BLOB and CURLOPT_SSLKEY_BLOB weren't usable with
mbedtls backend, so the support was added.
Closes #7157
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The CMake `try_compile` command is especially slow for
the Xcode generator. With this patch applied it first tests
for the currently used (and Open Group specified) send/recv
signature. In case this fails testing falls-back to the
permutations.
speed-up:
```
time cmake .. -GNinja -DCMAKE_USE_SECTRANSP=ON -DHTTP_ONLY=ON -DCMAKE_USE_LIBSSH2=OFF
before: 11.64s user 11.09s system 55% cpu 40.754 total
after: 7.84s user 6.57s system 51% cpu 28.074 total
```
```
time cmake .. -GXcode -DCMAKE_USE_SECTRANSP=ON -DHTTP_ONLY=ON -DCMAKE_USE_LIBSSH2=OFF
before: 217.07s user 104.15s system 60% cpu 8:51.79 total
after: 108.76s user 51.80s system 58% cpu 4:32.58 total
```
Closes #7158
|
|
|
|
|
|
|
|
| |
Debug builds would warn that these structs were not initialized properly
for pushed streams.
Ref: #7148
Closes #7153
|
|
|
|
|
|
|
|
|
| |
This function might get called for an easy handle for which the session
cache hasn't been setup. It now just returns a "miss" in that case.
Reported-by: Christoph M. Becker
Fixes #7148
Closes #7153
|
|
|
|
|
|
| |
As those are commonly used terms in the project.
Closes #7151
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Resolving the case insensitive host name 'localhost' now returns the
addresses 127.0.0.1 and (if IPv6 is enabled) ::1 without using any
resolver.
This removes the risk that users accidentally resolves 'localhost' to
something else. By making sure 'localhost' is always local, we can
assume a "secure context" for such transfers (for cookies etc).
Closes #7039
|
| |
|
|
|
|
|
|
|
|
|
| |
Also, use a single function library-wide for detecting if a given hostname is
a numerical IP address.
Reported-by: Harry Sintonen
Fixes #7146
Closes #7149
|
|
|
|
|
|
|
|
|
|
|
| |
Hyper returns the same error for wrong HTTP version as for negative
content-length. Test 178 verifies that negative content-length is
rejected but the hyper backend will return a different error for it (and
without any helpful message telling why the message was bad). It will
also not return any headers at all for the response, not even the ones
that arrived before the error.
Closes #7147
|
| |
|
|
|
|
| |
Closes #7143
|
| |
|
|
|
|
|
|
|
|
| |
In INSTALL.MD, it's currently set to CURL-DISABLE-md instead of
CURL-DISABLE.md. This generates a 404 on the cURL website as well as
when viewing the docs through Github.
Closes #7142
|
| |
|
|
|
|
|
|
| |
Makes test 129 work (HTTP/1.2 response).
Closes #7141
|
|
|
|
|
|
| |
Makes test 94 and 95 work
Closes #7141
|
|
|
|
|
|
|
|
|
| |
To prevent previous ones to get reused on subsequent requests. Matches
how the built-in HTTP code works. Makes test 90 to 93 work.
Add test 90 to 93 in travis.
Closes #7139
|
|
|
|
|
|
|
|
|
|
| |
In a3268eca792f1 this code was changed to use the ALPN_H2 constant
instead of the NGHTTP2_PROTO_ALPN constant. However, these constants are
not the same. The nghttp2 constant included the length of the string,
like this: "\x2h2". The ALPN_H2 constant is just "h2". Therefore we need
to re-add the length of the string to the ALPN buffer.
Closes #7138
|
|
|
|
| |
Closes #7137
|
|
|
|
|
|
|
|
| |
This reverts commit c3eefa95c31f55657f0af422e8268d738f689066.
Reported-by: Kevin Burke
Fixes #7122
Closes #7136
|