Age | Commit message (Collapse) | Author |
|
https://github.com/warmcat/libwebsockets/issues/2524
|
|
set the buffer size (rpath) for header processing during
proxying equal to the value in max_http_header_data
|
|
|
|
The loop check shouldn't care about bl any more, since we snipped bl from n
already.
|
|
|
|
|
|
https://github.com/warmcat/libwebsockets/issues/2500
|
|
|
|
|
|
|
|
|
|
|
|
|
|
This can't be reached currently, so remove it and simplify the setting of c
to the exact equivalent.
|
|
|
|
We can call mark immortal multiple times for a child stream, make sure it
is only processed (and increments the nwsi refcount) the once.
|
|
|
|
Coverity belatedly noticed that client_reset no longer changes *pwsi
|
|
|
|
It can fail, we should check it.
|
|
h cannot be NULL since it's set by address offset earlier.
Help coverity understand that.
|
|
The wsi is always created on a valid vhost. Add a needless NULL check on
it to satisfy coverity.
|
|
Coverity does not understand that once we checked that the header has
a non-zero length, the associated pointer can never be NULL. Add a
pointless check to make it happy.
|
|
ip can't be NULL, it's set to the address of another object.
|
|
continue here makes no difference than using break, but continue gets us a
pointless complaint "statement continue does not have any effect" and break
does not.
|
|
Coverity gets antsy that we were checking wsi->a.protocol for NULL earlier
then start using it... explicitly bail if it's NULL before we start using
it.
|
|
->san is a union, in this case we set the type to indicate we use
SAN_OTHER_NAME member, but set the unstructured_name union member, that is
smaller.
This doesn't cause any problem, since the union has space for it. But
Coverity noticed, it is wrong, so fix it.
|
|
Coverity doesn't understand that since we already handled
akid.keyIdentifier.MBEDTLS_PRIVATE(len) being zero, we don't need to
check for akid.keyIdentifier.MBEDTLS_PRIVATE(p) being NULL.
So explicitly check it, even though it is a NOP.
|
|
mbedtls_x509_get_name() does not clean up properly after itself in the case
of OOM on multi-segment name. We have to add extra handling and cleaning.
|
|
|
|
It's not exported, it seems nothing wants it any more
|
|
Either explicit option selection, or multiple SMP service threads, should
enable SO_REUSEPORT
https://github.com/warmcat/libwebsockets/issues/2470
|
|
There's no problem for library build, also with LWS_WITH_MINIMAL_EXAMPLES,
but after install at least on OSX, there are problems finding the installed
lws include dir (concealed on most platforms by the path being in the
default search list for the toolchain), and the references in the lws
includes to the tls includes meaning that explicit paths for that must be
available at consuming cmakes.
This patch enhances the cmake config installed by lws to deal with adding
the lws include paths to CMAKE_REQUIRED_INCLUDES and include_directories,
so it can be found before the target is introduced.
The tls include is passed back up the CMakeLists layers and the lws targets
marked with target_include_directories(PUBLIC) with them, so they are
understood as needed by consumers.
More boilerplate is moved out of the example consuming cmakes.
After this, on machines with previous installs of older lws, you may have to
clean out the cmake install path, that is usually something like
/usr/local/lib/cmake/libwebsockets/*
before make installing lws and putting the latest content in there.
|
|
Fix minimal-mqtt-client and minimal-mqtt-client-multi from crashes
by allowing indicating username and password are not on heap.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
When the client goes away, on some platforms all we get is POLLIN revent
stuck on... we have to read it to find out a zero length result and
understand it's gone.
Add SSE mode's DOING_TRANSACTION to the list of states we will read for.
|
|
|
|
|
|
Of note: A single use of lwsl_debug() remains because the function in
question is documented as a public API, but the passed-in parameter
doesn’t offer a path to a logging context.
|
|
If lws_s_a_t() is zero, there's a path where n stays -1 from its
declaration and we return -1.
|
|
|
|
|
|
|
|
|