deal with SSL_ERROR_WANT_ in client connect action

"4b0e01f Retry SSL_connect when SSL_get_error requests it. " from David Galeano
noticed the problem that client connect may receive SSL_ERROR_WANT_* from
SSL_connect, which is basically WOULDBLOCK.  That patch tried to deal with it
by blocking in a while(1) until the condition went away.

That's problematic because of it blocks service of anything else (including
the host application sockets in the external socket poll sharing case) for
up to 5s controlled by conditions at one client.

After fiddling with and researching this, the actual problem with the code is
we are not getting the SSL layer error correctly, it is not contained in the
code returned from the Connect api directly.

I was unable to get a renegotiation forced on my modern SSL libs, it complained
about protocol error are reopened the connection instead.  So I think the stuff
found in the docs and the web about the SSL_ERROR_WANT_ is probably not something
we will see in reality (if we check the right error code...)

Signed-off-by: Andy Green <andy.green@linaro.org>

add longlived option to test client

Needed to confirm pending timeouts won't kill the connection, by default
it spams the server with connections that live less than 5s

Signed-off-by: Andy Green <andy.green@linaro.org>

logging ensure everyone has a newline

Signed-off-by: Andy Green <andy.green@linaro.org>

replace ifdefs around close socket with compatible_close

Signed-off-by: Andy Green <andy.green@linaro.org>

ssl client certs fix crash

I run a web socket server that requires clients to present a certificate.

context_ssl_ = libwebsocket_create_context(wssPort_, wssIpAddr_.c_str(), protocols_ssl,

                           libwebsocket_internal_extensions,

                           cert_path.c_str(), key_path.c_str(), -1, -1,

LWS_SERVER_OPTION_REQUIRE_VALID_OPENSSL_CLIENT_CERT);

I am getting a crash in the OpenSSL_verify_callback().

The SSL_get_ex_data() call is returning NULL

I could not find a call to SSL_set_ex_data() for server mode operation.

Has anyone seen this crash in the newer versions?

Signed-off-by: Larry Hayes <larry.hayes@prodeasystems.com>

absorb README.rst into main README and code

Some of the advice in README.rst became deprecated with recent patches,
the (good) advice about http connection close is better demonstrated
in the code and API docs, and the remainder can go in the main README,
which will have to be refactored itself at some point.

Signed-off-by: Andy Green <andy.green@linaro.org>

expose compiletime constants to setting from configure

This patch allows control of the main compiletime constants in libwebsockets
from the configure commandline.

README is updated with documentation on what's available, how to set them
and the defaults.

The constants are logged with "info" severity (not visible by default) at
context create time.

The zlib constant previously exposed like this is moved to private-libwebsockets.h
so it can be printed along with the rest.

Signed-off-by: Andy Green <andy.green@linaro.org>

renable deflate frame buffer expansion fixing DoS

This reverts the removal of the deflate_frame code that was crashing after porting
David Galeano's code: he pointed out there's a typo in the merged version causing
the crash which is fixed here.

However the fixed code has a problem, there's no limit (other than int size) to the
amount of memory it will try to malloc, which can allow a DoS of the server by the
client sending malicious compression states that inflate to a large amount.  I have
added checking for OOM already that will avert the segfault that would otherwise follow
but the server will be unusuable if malicious connections were made repeatedly each
forcing it to allocate large buffers and cause small allocations on other connections
to fail.

The patch changes the code to use realloc(), and introduces a configurable limit
on the amount of memory one connection may need for zlib before the server hangs
up the connection.  It defaults to 64KBytes but can be set from ./configure as
described now in the README.

Signed-off-by: Andy Green <andy.green@linaro.org>

fix config enable name for getifaddrs

copy-paste...

Signed-off-by: Andy Green <andy.green@linaro.org>

introduce getifaddrs for toolchains without it

David found that uclibc did not provide this slightly esoteric api
and provided one from BSD that can be built by the library internally.

AG: Made contingent on configure option --enable-builtin-getifaddrs

Signed-off-by: David <cymerio@gmail.com>
Signed-off-by: Andy Green <andy.green@linaro.org>

audit and make all malloc check for OOM

Signed-off-by: Andy Green <andy.green@linaro.org>

logging add timestamp

Signed-off-by: Andy Green <andy.green@linaro.org>

logging extend level set api to allow setting emission function

Signed-off-by: Andy Green <andy.green@linaro.org>

update README with info on new logging scheme

Signed-off-by: Andy Green <andy.green@linaro.org>

allow enabling debug contexts from test apps

Adds a -d switch to everything so you can set the log level bitfeld.

Signed-off-by: Andy Green <andy.green@linaro.org>

introduce logging api and convert all library output to use it

 - multiple debug context calls lwsl_ err, warn, debug, parser, ext, client

 - api added to set which contexts output to stderr using a bitfield log_level

 - --disable-debug on configure removes all code that is not err or warn severity

 - err and warn contexts always output to stderr unless disabled by log_level

 - err and warn enabled by default in log_level

Signed-off-by: Andy Green <andy@warmcat.com>

compile in xcode, privatize debug macro

update soname and configure to v1.0

Signed-off-by: Andy Green <andy.green@linaro.org>

correct autotools warning

Signed-off-by: Andy Green <andy.green@linaro.org>

zlib code add OOM checks remove buffer expansion on rx path

Here testing with the test serer and chrome 25, the buffer expansion
code on Rx was triggered by a valid no data output condition and looped
until it exhausted all memory.

This patch adds OOM check to all malloc()s and removes the buffer expansion
code on the rx path... leaving the code on tx path for now.

Signed-off-by: Andy Green <andy.green@linaro.org>

Avoid leaking a socket when SSL_accept fails.

Print error string on accept failure.

Increased MAX_BROADCAST_PAYLOAD to match MAX_USER_RX_BUFFER.

Added README file with some useful tips for using the library.

Added support for continuation frames on the server.

Close connection if LWS_CALLBACK_HTTP returns non-zero.

Fixed to keep reading data until the SSL internal buffer is empty.
Before this fix only 2048 bytes were read,
the rest were buffered inside SSL until another message arrived!!!

Added no-cache headers to client handshake:
http://www.ietf.org/mail-archive/web/hybi/current/msg09841.html

Separate compression levels for server and client,
increased the later one to zlib default (6).

More correct handling of inflate result.

Fixed crash when HTTP requests method is not GET.

Check if macro SSL_OP_NO_COMPRESSION is defined before trying to use it.

Using size_t instead of int for deflate-frame offsets and length.

Added private macro CIPHERS_LIST_STRING to define ciphers list string.

When choosing a cipher, use the server's preferences.

Pass URI length to LWS_CALLBACK_HTTP.

Disable compression for SSL socket,
it is a waste of CPU when using compression extensions.

Using "SSL_CTX_use_certificate_chain_file" instead of "SSL_CTX_use_certificate_file"
to support server certificates signed by intermediaries.

Better definition of "debug" macro for Win32 builds.

Use __inline for Win32 builds.

Avoid checking choked pipe if no extension has more data to write.

zlib update 1.2.7

Signed-off-by: Andy Green <andy.green@linaro.org>

Set listen backlog to SOMAXCONN.

Fixed operator precedence bug.

Avoid deflate of small packets.

Support compressed packets bigger than MAX_USER_RX_BUFFER.
(AG adapted style and removed logging stuff)

Allow extensions when no protocol was specified.

Added extension "deflate-frame".
Using by default instead of "deflate-stream".

Added support for extensions that only manipulate application data.

Fixed deflate-stream extension.
When the output buffer was exhausted the input buffer was discarded
without checking if zlib had actually consumed all the input,
now we copy the remaining input data for the next call.

Added private macro AWAITING_TIMEOUT instead of harcoded value 5.

Fixed spacing.

Added context creation parameter for CA certificates file.

Return NULL if the handshake failed to complete,
libwebsocket_service_fd closes and frees the websocket in that case.

Ignoring linux build files

Use feature check rather than browser check.

Changed client handshake to use "Origin" instead of "Sec-WebSocket-Origin" as defined by RFC 6455 when using version 13 of the protocol.

Fixed compiler warnings on Windows.

Added new status codes from RFC 6455.

Fixed compiler warning on Windows.

required version of autoconf can be lower

Static variable is now const.

add context construction option to skip server hostname lookup

Signed-off-by: Andy Green <andy.green@linaro.org>

add missing docs for new context user pointer

Signed-off-by: Andy Green <andy.green@linaro.org>

libwebsocket_service_fd: EAGAIN is harmless, treat like EINTR

Only tested on linux.

Signed-off-by: Alon Levy <alevy@redhat.com>

libwebsocket_context: add userspace pointer for use before wsi creation

Signed-off-by: Alon Levy <alevy@redhat.com>

lib/Makefile.am: whitespace fix

Signed-off-by: Alon Levy <alevy@redhat.com>

add pkg-config file libwebsockets.pc

Signed-off-by: Alon Levy <alevy@redhat.com>

Added test.html favicon.ico to EXTRA_DIST.

Add missing .h files to sources.

Add kernel doc to extra_dist.

always taking an interest in ppid wont hurt

Signed-off-by: Andy Green <andy.green@linaro.org>

remove depcomp

Signed-off-by: Andy Green <andy.green@linaro.org>

stop being so fragile on socket lifecycle

Signed-off-by: Andy Green <andy.green@linaro.org>

use autogen.sh

Signed-off-by: Andy Green <andy.green@linaro.org>

Updating changelog

Adding Tizen packaging files

add not about autoreconf and libtoolize

Signed-off-by: Andy Green <andy.green@linaro.org>

restore accept error as closure signal

Signed-off-by: Andy Green <andy.green@linaro.org>

check for prctl, poll parent PID if not present

this allows forking code to be used on non-linux systems

Fixing uninitialised memory

These were found with valgrind tool.
Signed-off-by: Paulo Roberto Urio <paulourio@gmail.com>

some toolchains need stddef

Signed-off-by: Paulo Roberto Urio <paulourio@gmail.com>

add missing sa_data init to canonical hostname code

Shay noticed we're no longer initializing the initial lookup of
server canonical hostname correctly

Reported-by: Shay Zuker <shay@boxee.tv>
Signed-off-by: Andy Green <andy.green@linaro.org>

Fixed segfault in libwebsocket_context_destroy.

When creating a context with NULL extensions list,
a segmentation fault was yelled when trying to
destroy the context.  This checks if the
extension list is NULL before go through the list.
Signed-off-by: Paulo Roberto Urio <paulourio@gmail.com>

sockets were left open if connection could not be
 established resulting in a mass of unusable open file
 descriptors

Signed-off-by: Tobias Maier <tobias.maier@netplace.com>

link lz explicitly for mips

    fix link error since libz is not linked in

Signed-off-by: yuval tal <yuvalt@boxee.tv>

null exception with null extensions list

I was under the impression extensions could be null, so heres a patch to fix this error in libwebsockets. Cheers!

Signed-off-by: Andrew Chambers <andrewchamberss@gmail.com>
--

refresh autotools stuff

Signed-off-by: Andy Green <andy.green@linaro.org>

update test html now mozilla uses normal ws apis

Signed-off-by: Andy Green <andy.green@linaro.org>

start migration to getnameinfo

Signed-off-by: Andy Green <andy.green@linaro.org>

openssl allow connect multiple packets for SSL_connect

Ken found over the internet with real delays, SSL_connect can
fail to work.  This patch adapts his workaround to stay in the
connect state until we either run out of time for the connect
or succeed.

Signed-off-by: Andy Green <andy.green@linaro.org>
Signed-off-by: Ken Atherton <katherton@echofirst.com>

cleanup extra wrapping needed on braced define

Signed-off-by: Andy Green <andy.green@linaro.org>

android toolchain SHA1 endian fix

Android toolchain needs an extra include if it's not to confuse
SHA-1 code probably with incorrect endian-ness from missing /bits/

Signed-off-by: Peter Hillier <peterhillier@yahoo.com>
Signed-off-by: Andy Green <andy@warmcat.com>

ssize_t needs help on windows

ssize_t is needed, but absent in Windows.

This patch typedefs it to an int in that case as recommended by Tobias.

Signed-off-by: Andy Green <andy@warmcat.com>
Signed-off-by: David Brooks <dave@bcs.co.nz>
Signed-off-by: Tobias Maier <tobias.maier@netplace.com>
Reported-by: Rich Gossweiler <rich.gossweiler@gmail.com>

introduce libwebsocket_client_connect_extended

Signed-off-by: David Brooks <dave@bcs.co.nz>
Signed-off-by: Andy Green <andy@warmcat.com>
--

introduce LWS_CALLBACK_CLIENT_CONNECTION_ERROR

Signed-off-by: David Brooks <dave@bcs.co.nz>
Signed-off-by: Andy Green <andy@warmcat.com>
--

set connection callback before connection completed to allow early messages

Signed-off-by: David Brooks <dave@bcs.co.nz>
Signed-off-by: Andy Green <andy@warmcat.com>
--

 lib/client-handshake.c |   19 +++++++++++++++++++
 1 files changed, 19 insertions(+), 0 deletions(-)

improve callback prototypes to use single definition

Not sure all compilers can hack this, but give it a try

Signed-off-by: David Brooks <dave@bcs.co.nz>
Signed-off-by: Andy Green <andy@warmcat.com>
--

reduce debug spew

Signed-off-by: David Brooks <dave@bcs.co.nz>
Signed-off-by: Andy Green <andy@warmcat.com>
--

workaround osx compiler bug

Signed-off-by: Pasi Mankinen <pasi.t.mankinen@student.jyu.fi>