Merge pull request #2594 from baruchsiach/fix-math-2

Don't disable C99 long double math on non Unix platforms

Merge pull request #2577 from hardening/listen_from_socket

Add the ability to have listener from an existing socket

Return FALSE under Win32

Test the number of listening handles

Add the ability to have listener from an existing socket

Merge pull request #2596 from akallabeth/wlog_default_out_fix

Wlog console output to stderr

Merge pull request #2601 from nfedera/fix-2015-05-07-01

Fix some unchecked create calls

Merge pull request #2597 from clivest/bitstream_read_fix

Fix bitstream reading bounds check

Merge pull request #2598 from clivest/stream_leak_fix

Fix two cases of stream resource leak

Merge pull request #2599 from hardening/fix_wayland_colors

Fix colors with the wayland client

Merge pull request #2603 from hardening/rail_empty_title_fix

Fix empty window title

Merge pull request #2604 from hardening/rail_fix

Fix remoteApps

server/shadow: added missing error message

Fix empty window title

When the size of a RAIL string was 0, we were reporting an OOM error.

Fix remoteApps

The assert that have been removed were preventing RAIL from working when you
have an intermediary screen before seeing the application.

Fix unchecked CreateDirectory calls

Fix unchecked CreateSemaphore calls

Fix unchecked CreateMutex calls

Fix colors with the wayland client

Fix whitespace

Fix bitstream reading

The bounds checks where previously incorrect

Fix two cases of potentially leaked streams

Fixed broken indent.

Not writing messages if level is WLOG_OFF.

Console logging to stdout and stderr

With this patch the default for the console logger
changes to writing to stdout for TRACE, DEBUG and INFO
messages and stderr for WARNING, ERROR and FATAL messages.

Merge pull request #2592 from nfedera/fix-2015-05-05-01

Fix unchecked CreateThread calls and misc fixes

Don't disable C99 long double math on non Unix platforms

CMakeLists.txt may enable HAVE_MATH_C99_LONG_DOUBLE only on Unix platforms.
Limit the effect of this symbol accordingly.

Merge pull request #2587 from floppym/armfix

Remove the ARM_FP_ABI option

Merge pull request #2589 from baruchsiach/fix-math

Don't use unavailable C99 long double math functions

Fix unchecked CreateThread calls and misc fixes

Merge pull request #2582 from nfedera/fix-2015-04-28-01

Fix unchecked CreateEvent calls and misc fixes

Don't use unavailable C99 long double math functions

uClibc variants do not provide the C99 long double math functions like ceill,
powl, etc.. For future compatibility use check_symbol_exists() to check
whether these functions are available, and keep the result in
HAVE_MATH_C99_LONG_DOUBLE. Use that instead of the fragile Cygwin version
check in triodef.h.

Fixes build failures under uClibc(-ng) like:

../../libwinpr/utils/libwinpr-utils.so.0.1.0: undefined reference to `powl'
../../libwinpr/utils/libwinpr-utils.so.0.1.0: undefined reference to `fmodl'
../../libwinpr/utils/libwinpr-utils.so.0.1.0: undefined reference to `ceill'
../../libwinpr/utils/libwinpr-utils.so.0.1.0: undefined reference to `log10l'
../../libwinpr/utils/libwinpr-utils.so.0.1.0: undefined reference to `floorl'
collect2: error: ld returned 1 exit status

Signed-off-by: Baruch Siach <baruch@tkos.co.il>

Remove the ARM_FP_ABI option

It's unclear why this option would be necessary, and it causes problems
when people do not match it to their toolchain and CFLAGS.

To set the float abi, either use a toolchain with an appropriate default
or set the float-abi option in the CFLAGS environment variable.

This should resolve #2586.

Merge pull request #2584 from baruchsiach/fix-eventfd-2

Fix build with newer uclibc versions

Increase API version to 1.2.3

Fix build with newer uclibc versions

uClibc master branch, as well as uclibc-ng add eventfd_{read,write}
definitions. Instead of testing for __UCLIBC__, have cmake explicitly check
for the existence of eventfd_read and save the result in
WITH_EVENTFD_READ_WRITE.

Fixes build errors like:

.../winpr/libwinpr/synch/event.c:120:12: error: static declaration of 'eventfd_read' follows non-static declaration
 static int eventfd_read(int fd, eventfd_t* value)
            ^
In file included from .../winpr/libwinpr/synch/event.c:39:0:
.../usr/include/sys/eventfd.h:37:12: note: previous declaration of 'eventfd_read' was here
 extern int eventfd_read (int __fd, eventfd_t *__value);
            ^
.../winpr/libwinpr/synch/event.c:125:12: error: static declaration of 'eventfd_write' follows non-static declaration
 static int eventfd_write(int fd, eventfd_t value)
            ^
In file included from .../winpr/libwinpr/synch/event.c:39:0:
.../usr/include/sys/eventfd.h:40:12: note: previous declaration of 'eventfd_write' was here
 extern int eventfd_write (int __fd, eventfd_t __value);

Fix unchecked CreateEvent calls and misc fixes

1)
Added missing checks for CreateEvent which also required the
following related changes:

- changed freerdp_context_new API to BOOL
- changed freerdp_peer_context_new API to BOOL
- changed pRdpClientNew callback to BOOL
- changed pContextNew callback to BOOL
- changed psPeerAccepted callback to BOOL
- changed psPeerContextNew callback to BOOL

2)
Fixed lots of missing alloc and error checks in the
changed code's neighbourhood.

3)
Check freerdp_client_codecs_prepare result to avoid segfaults
caused by using non-initialized codecs.

4)
Fixed deadlocks in x11 caused by missing xf_unlock_x11() calls
in some error handlers

5)
Some fixes in thread pool:
- DEFAULT_POOL assignment did not match TP_POOL definition
- don't free the pool pointer if it points to the static DEFAULT_POOL
- added error handling and cleanup in InitializeThreadpool

Merge pull request #2579 from akallabeth/mutex_debug_log

winpr mutex unlock on destroy

Added EBUSY to success states.

Better error handling on mutex destroy.

Merge pull request #2578 from jonathankap/master

Fix issue when numWindowIds is 0

Ensuring mutex is unlocked on destroy.

Fix issue when numWindowIds is 0

Added mutex debug flag.

When mutex debugging is enabled now a stack trace
is logged, if a mutex is locked on destruction.

Merge pull request #2575 from bmiklautz/gdi_fix

gdi: fix problem with gdi_SelectObject check(s)

gdi: fix problem with gdi_SelectObject check(s)

When a hdc is created no initial or default objects are created
therefore can the first call of gdi_SelectObject return NULL.
Because of this checking the return value of  gdi_SelectObject failed
for newly create hdc causing errors (disconnects).

Since all types of HGDIOBJECT are handled and the return value of
gdi_SelectObject isn't used the recently added checks were removed
again.

Merge pull request #2568 from nfedera/fix-2015-04-23-01

codec/rfx: error checking and various fixes

codec/rfx: simplification, segfault/malloc fixes

rfx_process_message_sync:
- simplified the check if the header messages got processed

rfx_process_message_tileset:
- ObjectPool_Take result was not checked
- fail if TS_RFX_TILE block type is not CBT_TILE
- CreateThreadpoolWork result was not checked
- post decoding loop code segfaulted in error case

rfx_decoder_tile_new:
- missing malloc check

rfx_message_free:
- segfault protection

rfx_write_message_tileset:
- segfault protection

Merge pull request #2572 from bceverly/OpenBSD-2015-04-24

Added support for manpage construction for OpenBSD

Added support for manpage construction for OpenBSD

codec/rfx: error checking and various fixes

- removed some unneeded null checks for free()
- fixed a memory leak in shadow_client
- removed rfx_compose_message_header from API

Changed the following functions to BOOL, check the result
where they are called and handle failures:
- rfx_compose_message
- rfx_compose_message_header
- rfx_write_tile
- rfx_write_message_tileset
- rfx_write_message_frame_begin
- rfx_write_message_region
- rfx_write_message_frame_end
- rfx_write_message

rfx_process_message:
- check memory allocation failures
- verify protocol-conform order of data messages to prevents memory
  leaks caused by repeated allocations
- verify that header messages were parsed/received before the
  data messages
- treat unknown rlgr mode as error
- fixed/added error handling
- fixed all callers to check/handle result

rfx_encode_message:
- fixed incorrect usage of realloc
- missing malloc check
- missing check of CreateThreadpoolWork
- correct cleanup on failure (threadpool, memory)
- check rfx_encode_message result

rfx_encode_messages:
- check rfx_split_message result
- correct cleanup on failure
- prevent memory leak on failure

rfx_write_message_context:
- fixed invalid channelId value (must be 0xFF for WBT_CONTEXT)

rfx_process_message_codec_versions:
- fixed invalid read size of codec_version (it is 16bit)

rfx_process_message_channels:
- verify protocol conform channelId value

rfx_process_message_region:
- replaced invalid reallocs with malloc
- read and verify regionType and numTileSets from stream

rfx_process_message_tileset:
- check allocation results
- fixed incorrect usages of realloc

setupWorkers:
- fixed incorrect usages of realloc

rfx_split_message:
- removed dead code
- missing malloc check

rfx_compose_message:
- fixed a memory leak
- check/handle rfx_encode_message result

Merge pull request #2566 from bceverly/OpenBSD-2015-04-22

Further cleanup on the OpenBSD build

Removed compiler warning about obsolete header file in OpenBSD

Moved declaration of OPENBSD to higher in the file to clean some things up.  Also, removed -Wredundant-decls from OpenBSD because the OpenBSD standard include files have some redundant declarations that generate a large number of compiler warnings.

Merge pull request #2564 from akallabeth/pthread_mutex_timedjoin_detection

Fixed pthread_mutex_timedjoin detection.

Merge pull request #2555 from akallabeth/wayland_warning_fix

Updated wayland client to new API.

Updated to new freerdp_get_event_handles API.

Updated wayland client to new API.

Merge pull request #2554 from akallabeth/event_fix

Fix for transport event reset

Merge pull request #2544 from bmiklautz/boolenization

change return types of callbacks to BOOL

Increase API version to 1.2.2

Fixed pthread_mutex_timedjoin detection.

Merge pull request #2562 from bceverly/OpenBSD-2015-04-11

Open bsd 2015 04 11

Modified code to use bulitin pthread functions on OpenBSD

Necessary changes to get latest branch working on OpenBSD 5.6

Necessary changes to get latest branch working on OpenBSD 5.6

Merge pull request #2537 from MartinHaimberger/mh-autodetect-fix

autodetect: debug fixed and level adjusted

Merge pull request #2553 from akallabeth/leak_fixes

Leak fixes

Merge pull request #2560 from akallabeth/get_event_handle_api_fix

Add count to ```*_get_event_handles``` function

Changed listener callback arguments and return.

Fixed handle copy and index.

TestStream: set function parameters to void

Fix formating

gdi: fix broken return type checks

Integrates comments from pull request.

Fix GDI return values and other fixes

* top level GDI functions return 0 on error and != 0 otherwise but the
  low level functions (16bpp.c, 8bpp.c 32bpp.c) which are called did it
exactly the other way around. Those were adapted.
* change gdi_InvalidateRegion to BOOL and check calls where appropriate
* integrate comments from pull request

change return types of callbacks to BOOL

* change all client/server callbacks to BOOL
* update all clients accordingly
* add multiple return value checks
* small fixes

TestStream: fix compiler warning

Remove unnecessary argument from read function call.

ios: updates to build with xcode6+

* update tool chain to use clang instead of gcc
* add compatibility function for fwrite

Fixed some type mismatches.

Add *get_event_handles handle count, return check.

Modified GetEventHandles of listener

Now using nCount as in and out argument.
When called, set nCount to the number of available handles.
This value is checked and an error returned, if not enough
handles are available.

Merge pull request #2558 from llyzs/llyzs

h264: change encoder api and add multi-thread support.

h264: change encoder api and add multi-thread support.

Merge pull request #2538 from MartinHaimberger/mh-add-transport-get-bytes

transport: added function to get bytes written

Merge pull request #2552 from akallabeth/sample_update

Sample update

Fixed reading bitmap from buffer.

Using auto reset event now.

Added proper bitmap and png image test.

Fixed bitmap reading bug.

Using new API for socket listener.

Moved client handler to new API.

Moved cleanup code, removed unused.

Updated sample for new API.

Merge pull request #2547 from nfedera/fix-2015-04-16-01

Fullscreen fixes and smart sizing resuscitation

xfreerdp: fixed initial smart-sizing dimensions

The window width/height variables and in turn SmartSizingWidth/Height
get automatically updated in th the configure notify event handler.
If the window is created initially the values don't differ from the
event values and therefore SmartSizing was not applied until the
window was resized.

Fixed NULL dereference.

Fixed NULL dereference and double free

Fixed NULL dereference

Fixed memory leaks and NULL dereferences.

xfreerdp: FS fixes & smart-sizing resuscitation

- make smart-sizing work again which was killed in previous commits
- removed several unnecessary/ugly workarounds
- miscellaneous small fixes
- new feature: restore previous window position when toggling out of
  fullscreen mode
- new feature: if /f is specified in combination with /smart-sizing:WxH
  we run the session in the /smart-sizing dimensions scaled to full screen