Separate private defines in lws_config_private.h

So we don't expose all HAVE_ macros and such to the world.

Fix potential memory leak

- Got rid of ifdef _WIN32 stuff adn moved to plat_ files instead.
- Also, check all calls to lws_zalloc, was potential failure on WIN32
- Made context destory enable to destroy a half inited context as well. This way I got get rid of some of the error handling complexity in libwebsocket_create_context
- Added TODOs for some potential problems I see where things might be leaking and such

Bugfix: SSL code should not free wsi

The SSL code should not do lws_free(new_wsi) here. The caller should clean up the wsi in libwebsocket_close_and_free_session. Instead this can potentially cause a crash during that call.

Also if we fail to create the SSL for hte listener we should indicate failure to the caller.

Turn off building windows installer, breaks everything

From 0a0b9a6d02037e93dabadad5e04d39c1c6af0c96 Mon Sep 17 00:00:00 2001
Subject: [PATCH] Update Openssl lib version

Don't mess with system types

is no problem on a unix system with sys/types.h

However on Windows it will result in a #undef pid_t ... When libwebsockets is used with pthreads32 which typedefs pid_t itself, it will fail to build with some weird errors.

Also since we're exporitng these macros to the entire world via libwebsockets.h we should not define a VERSION macro. Which is defined improperly anyway.

Use LWS_HAVE_ instead of just HAVE_

Since we include lws_config.h in the public headers, at least our HAVE_ macros should be kind of unique, so that we don't get redefinitions when used with other libraries using config files as well.

Fix windows build errors

NSI installer errors causing failed windows builds

use send MSG_NOSIGNAL

after

https://github.com/fancycode/libwebsockets/commit/810dbadd6b4cc3e192d050c81fc3ec8e652c41c6

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

Fix shared library name for all platforms

This is a further fix to f1b1254, building on 26fd0d2.

Tidy up the LWS_WITH_SHARED and LWS_WITH_STATIC options.

Allow using CyaSSL in stead of wolfSSL

Update http-link to wolfSSL in README.build.md

Move from CyaSSL to wolfSSL

This patch lets libwebsockets use the lastest version of wolfSSL (the new name for CyaSSL).
The reason for the patch is that allthough wolfSSL provides compatibility headers for (old) projects using CyaSSL,
these are incomplete and do not work for libwebsockets.

The patch also fixes a typo in CMakeLists.txt where CYASSL_LIBRARIES was added to include_directories() instead of CYASSL_INCLUDE_DIRS.

Signed-off-by: ABruines <alexander.bruines@gmail.com>

fix #309. Receive large messages over WSS

Read the full incoming TLS/SSL record at once in libwebsocket_service_fd().

SSL_read() is called until no more pending data for the current record is buffered in SSL.
SSL_read() is never requested more than the pending data size for the current record
to ensure that the fd is not read again for new data, which would be copied in the SSL buffer otherwise.

Pass HTTP reject error code returned by server to client

AG: added changelog and documentation comment

Signed-off-by: Gadkari Mugdha <mugdha.gadkari@siemens.com>
Reviewed-by: Roger Meier <r.meier@siemens.com>
HTTP reject error code returned by server during HTTP handshake is extracted and sent to the client application.
With this fix, the Connection error callback will include the HTTP reject status code and response frame received from server. This string passed in Connection error callback can be read or ignored by client application based on client application requirement

Fixing occasional failure of connect() on Windows

See: https://msdn.microsoft.com/en-us/library/windows/desktop/ms737625(v=vs.85).aspx

Specifically the section reading:

"Until the connection attempt completes on a nonblocking socket, all subsequent calls to connect on the same socket will fail with the error code WSAEALREADY, and WSAEISCONN when the connection completes successfully. Due to ambiguities in version 1.1 of the Windows Sockets specification, error codes returned from connect while a connection is already pending may vary among implementations. As a result, it is not recommended that applications use multiple calls to connect to detect connection completion. If they do, they must be prepared to handle WSAEINVAL and WSAEWOULDBLOCK error values the same way that they handle WSAEALREADY, to assure robust operation."

appveyor update windows openssl blob url

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

Added callback for PONG responses

[PATCH] added timeout reset in serve_http_file_fragment to prevent timeout on big files

cmake remove targets from install path

No package on my machine populates cmake files, I think it's extraneous.

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

cmake revert change to linux so

Put back the stanza fixing the .so name to what it has always been
on Linux, trashed in f1b12544

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

close CLIENT_WAITING_CONNECT should callback CONNECTION_ERROR

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

close also enable pollout processing for post send close state

This should solve the 100% cpu at close problem reported by Bruce Perens

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

close send close ack control packet type

Noticed by Andrey Pokrovskiy

Close processing reused ping processing to save and send the payload,
and sets a flag to know it's close, but forgot to change the control
packet accordingly.

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

Subject: [PATCH] Multiple changes in the build process

* Default CMAKE_BUILD_TYPE to Release
* Use CMAKE_BUILD_TYPE to define _DEBUG (NDEBUG is more standard though,
  but cmake defines it)
* Drop LWS_WITHOUT_DEBUG (use CMAKE_BUILD_TYPE for that)
* Drop LWS_NO_EXTERNAL_POLL (was not used)
* Drop CMAKE_BUILD (CMake is the only build system now)
* Add LWS_WITH_STATIC and LWS_WITH_SHARED to choose what version(s) to
  build
* Add LWS_XXX_LIBRARIES and LWS_XXX_INCLUDE_DIRS for each library
  dependency (zlib, openssl, libev, cyassl)
* Support setting of XXX_LIBRARIES, XXX_LIBRARIES and XXX_INCLUDE_DIRS
  by parent project (when included with add_subdirectory())
* Rename LWS_USE_EXTERNAL_ZLIB to LWS_USE_BUNDLED_ZLIB and default it to
  NO (since it's Windows only)
* Default LWS_WITHOUT_DAEMONIZE to NO (since network lib shouldn't know
  how to do it anyway)
* Rename config.h.cmake to lws_config.h.in
* Rename shared library to websockets_shared so linker will not be
  confused
* Fix inline keyword detection in clang
* Explicitly set MACOSX_RPATH to YES on MacOS

Subject: [PATCH] Fix unused n variable warning in http2.c

Subject: [PATCH] Add missing lib/header.c file

File was lost during merge to upstream.

Subject: [PATCH] Be quite when executing openssl commands

From 04da2ccd1e8c5b582c4e2a77ee53f929ae8f22a0 Mon Sep 17 00:00:00 2001
Subject: [PATCH] Always include lws_config.h since now we have only CMake
 build

Subject: [PATCH] Fix broken libev support in client mode

refactor allow http parse also for client

Subject: [PATCH] Don't fail on non-existing files

Subject: [PATCH] Don't build installer on noserver build.

Building the installer like this is a bad idea... Better to move it into
the CMakeLists.txt and be aware of what is being build and so on using
CPack.

Subject: [PATCH] Fix windows build

New version of OpenSSL

Ensure proper cleanup is performed if last websocket is
 closed.

Fix broken libev support in client mode

Maybe it will be better to move all lws_libev_io() inside
lws_change_pollfd() to avoid similar problems in future.
The same for lws_libev_accept() and insert_wsi_socket_into_fds().

close reply must use writable control reply path

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

client error on timeout before connection

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

windows check for NULL getprotobyname

From maxkaro on trac #110
https://libwebsockets.org/trac/libwebsockets/ticket/110

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

timeout counts as pernmanently unusable close

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

client getaddrinfo valgrind fix

Signed-off-by: Danomi Czaski <djczaski@gmail.com>

changed expected shared lib version from 4.0.0 to 5 in spec file

parser restrict unknown header paranoia to server methods

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

test echo put proper origin

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

close cover flush stored send with timeout

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

unsigned char by default compiler compatibility

http://ml.libwebsockets.org/pipermail/libwebsockets/2015-March/001705.html

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

specfile openssl devel only needed on buildrequires

https://github.com/warmcat/libwebsockets/issues/259

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

prepare for v1.4

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

context destroy just kill wsis ugh add missing bit

Unsaved file...

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

set server listening wsi use_ssl status

If the server option to allow non-ssl is enabled,
it will force this cleared when it sees that has happened

context destroy just kill wsis

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

ping client ping handling use same infrastructure as server

First part of this:

https://github.com/warmcat/libwebsockets/issues/256

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

ping allow zero length PING

Part of

https://github.com/warmcat/libwebsockets/issues/256

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

Subject: [PATCH] Windows installer related fixes.

Remove incorrectly committed cpack changes.
Fix call to makensis.
Update README filenames.

documentation add info http_proxy members

From https://libwebsockets.org/trac/libwebsockets/ticket/104

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

Subject: [PATCH 1/2] fix win32 context memory leak

windows installer missing quote

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

Add Windows installer file and configure appveyor to create it

Also update OpenSSL on Windows to latest version.

test server only mention pollfd if EXTERNAL_POLL

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

clean remove x perm on two sources

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

use initgroups in plat unix

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

Subject: [PATCH] call lws_plat_inet_ntop instead of inet_ntop

LWS_WITHOUT_DEBUG fix unused var

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

Subject: [PATCH] Simplify soversion.

From 03b93ec864c29dfbc41f2adbdf1b639f5403c95f Mon Sep 17 00:00:00 2001
Subject: [PATCH] Added c include

Fixes #220 and #223

client avoid double free on error path

via dj1yfk on trac #99

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

Subject: [PATCH] Fix bug parsing unknown headers.

Bug was introduced in 49f72aa45.

win32 use hashtable for fd management

At least some win32 uses an opaque pointer for fd that is not
an ordinal like it is in unix.

Resurrect the old hashtable management for that platform to use
instead, and introduce a helper to get the wsi from the fd "somehow".

Signed-off-by: Bud Davis <bdavis9659@gmail.com>

ssl pendding buffered reads dont dereference first time

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

ssl pending buffered reads unbreak NO_SSL

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

ssl pending buffered reads use linked list

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

gethostbyname removal from lws get peer addresses

This replaces gethostbyname in libwebsockets_get_peer_addresses and
in the client handshake path.

There's one left in lws-plat-win but that can be done another time.

Let's see how much damage that did to the cross-platform and option
builds...

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

Callback with reason LWS_CALLBACK_CLOSED_HTTP when HTTP closed before serve.

If the remote HTTP client closes the connection before serving commences, the
only notification client code receives is LWS_CALLBACK_WSI_DESTROY. This commit
gives LWS_CALLBACK_CLOSED_HTTP as well, as would happen if HTTP serving had
already commenced.

Add compiler checking for visibility flag

Signed-off-by: Steven Lai <steven.lai@liricco.com>

getifaddrs needs private libwebsockets.h

via trac

Signed-off-by: Steven Y Lai <stevenylai@INVALID.com>

plat unix 64 bit timestamps on 32-bit os

Signed-off-by: Imo Farcher <ifar@gmx.net>

Add PATCH, PUT, DELETE methods

AG: rewrite method code to use method arrays

cmake ipv6 default to off

Signed-off-by: Quinlan Pfiffer <qpfiffer@gmail.com>

Subject: [PATCH] Fix error handling when calling "REQUIRES PRIVATE KEY"
 callback.

A recent patch in ssl.c introduced a callback for setting the
private key of the SSL context. This code contained a bug, which
resulted in lws_context_init_server_ssl() returning always
with a return value of 1, indicating an error.

This patch introduces the missing curly braces to fix the code's
intended behaviour.

Subject: [PATCH] Fix deflate-frame extension TX log statement

Subject: [PATCH] New OpenSSL version for appveyor build!

fix #220. Add missing include statement for <cstdarg>

Subject: [PATCH] Adds "REQUIRES PRIVATE KEY" callback

In some situations the private key is not directly available via
filesystem (for example, when stored on a smartcard). If this is
the case, the user can set the private key filepath to NULL and
expect this callback reason to set the key directly via openSSL
library calls.

clean unmix declarations and code

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

Subject: [PATCH] proxy protocol (version 1) parsing implementation

The proxy line is dumped in the token buffer as any other HTTP header

Adapted by AG to move token id to be last in the parsed list

Subject: [PATCH] add user context pointer to the protocol structure

Subject: [PATCH] add _lws_logv to make loging accessible to other wrappers.
 use _lws_logv in _lws_log

improve c++ compatibility

Adapted by Joakim

client: fix verification of the accepted protocol

to properly take ", *" as delimiter

Signed-off-by: Alejandro Mery <amery@geeks.cl>

README: fix coverity link

coverity link was pointing to the wrong project

Signed-off-by: Alejandro Mery <amery@geeks.cl>

libev: clean cast abuse

assuming that you can cast to the first field of an struct is very
risky

Signed-off-by: Alejandro Mery <amery@geeks.cl>

add container_of() macro

to be used to remove cast abuse and finding parent structs later

Signed-off-by: Alejandro Mery <amery@geeks.cl>

Subject: [PATCH] Attempt at fixing Travis for OSX.

Move install into external shell script. We cannot use apt-get on OSX, use brew instead... Also valgrind isn't used at the moment so skip intalling it.

Subject: [PATCH] Fixes occasional segfault in lws_chage_pollfd

* so it happens that sometimes lws_change_pollfd is called with
  bad or incomplete data structures, mainly the context pointer
  obtained from wsi->protocol is null or wsi->position_in_fds_table
  is -1; checks have been added to avoid this condition

http2 remove some dead code

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

NO_EXTENSION clean out handled var in handle_POLLOUT_event

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

NO_SERVER_fix cosmetic warning

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

Subject: [PATCH] Fix the travis.yml to build everything

When fixing the indentation for the coverity chagnes in 924bcfbc995e5c862216f6ad47a734a2c8f14370
the build matrix got turned off. This re-enables this.

Also change from using gcc/clang. What we want is to build on both Linux
and OSX. Not just run Linux with gcc+clang.

However to enable multi-os support one has to mail the travis support:
http://docs.travis-ci.com/user/multi-os/

"This feature needs to be enabled manually. If you wish to take advantage
of this feature, please send email to support@travis-ci.com, indicating
which repository you would like to have it enabled."

Subject: [PATCH] Introduce lws_free2() helper to free and re-NULL pointers

Signed-off-by: Alejandro Mery <amery@geeks.cl>

Subject: [PATCH] Use custom allocator

Signed-off-by: Alejandro Mery <amery@geeks.cl>