shadow_client: dispose encomsp and remdesk

Their threads were kept running after the client's exit.

Merge pull request #2638 from kwitaszczyk/master

Assign peer to settings when a new RDP module is being initialized.

Merge pull request #2640 from xsumbe00/master

winsock.c Solaris build fixes

winsock.c Solaris build fixes

Assign peer to settings when a new RDP module is being initialized.

Merge pull request #2637 from nfedera/fix-shadow_client_context_cleanup

shadow/client context cleanup and unused vars fix

Merge pull request #2583 from bceverly/OpenBSD-cleanup

Two minor source code changes to remove warnings

shadow/client context cleanup and unused vars fix

Merge pull request #2630 from hardening/new_2616

Add checks for some XXX_New and XXX_Add functions

Merge pull request #2557 from realjiangms/shadow_fix_hang2395

shadow fix multi-client hang

Merge pull request #2632 from akallabeth/client_bandwidth_measurement_callback

Added client bandwidth measurement result callback.

Fixed declaration of xf_event_action_script_init()

Take in account nfedera's review

Fixed mistake with bitwise operator in previous two commits

Moved thread code to OpenBSD only

Added client bandwidth measurement result callback.

Merge pull request #2633 from clivest/progressive_codec_new_surface_fix

Fix rounding error in progressive codec surface new

Merge pull request #2635 from clivest/gdi_gfx_leaked_regions_fix

Fix two memory leaks in gdi/gfx

Fix two cases of leaked regions in gdi/gfx

Fix rounding error in progressive codec

The grid is composed of 64x64 blocks and should not be smaller
than the surface. If width or height were not a multiple of 64
the previous rounding resulted in a grid smaller than the surface.

Fixed last VirtualChannelEntry

Fixed functions that return BOOLs

Merge pull request #2627 from nfedera/fix-x11-polyregion

client/x11: fix + simplify invalidate_poly_region

client/x11: fix + simplify invalidate_poly_region

Merge pull request #2624 from kwitaszczyk/master

QueryContextAttributes shouldn't return an error when a user or a domain is empty.

smartcard: fix compiler warnings

gcc 4.7.2:
warning: passing argument 1 of ‘free’ discards ‘const’ qualifier from
pointer target type

Add checks for some XXX_New and XXX_Add functions

Based on PR #2616

QueryContextAttributes shouldn't return an error when a user or a domain
is empty.

Merge pull request #2623 from nfedera/fix-xfclient-if-semicolons

client/x11: fixed if statements with empty body

client/x11: fixed if statements with empty body

Merge pull request #2622 from kwitaszczyk/master

Pass valid context to CompleteAuthToken function in Negotiate Security Package.

Pass valid context to CompleteAuthToken function in Negotiate Security Package.

Merge pull request #2621 from nfedera/fix-mac-async-transport

transport/mfreerdp: fix async transport

transport/mfreerdp: fix async transport

- handle WAIT_TIMEOUT result as error in async transport thread
  if an INFINITE timeout was specified in WaitForMultipleObjects
- fix mfreerdp's async transport handling to not use
  freerdp_get_event_handles/freerdp_check_event_handles if async
  transport is activated

Merge pull request #2620 from nfedera/fix-gdi-cache

gdi: fix nullification of instance->context->cache

gdi: fix nullification of instance->context->cache

Merge pull request #2611 from nfedera/fix-async-transport-thread

transport: fixed multiple errors in async thread

Merge pull request #2613 from oshogbo/gethostname

At least at FreeBSD - the gethostname(3) function can fail.

Merge pull request #2612 from oshogbo/style

Function return bool, so lets return FALSE if 0 or TRUE if 1.

At least at FreeBSD - the gethostname(3) function can fail.
Check returned value.

Function return bool, so lets return FALSE if 0 or TRUE if 1.

transport: fixed multiple errors in async thread

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

fixed multiple missing gdi return value checks

Merge pull request #2607 from nfedera/remove_free_null_checks

free can handle NULL perfectly fine

free can handle NULL perfectly fine

shadow fix multi-client hang

Merge from commit d4df3a952b
Conflicts:

server/shadow/shadow_subsystem.c

fixed multiple missing gdi return value checks

mainly gdi_Create* functions

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.

Removed unnecessary include file.

Simplified random byte stream generation code to use SSL RAND_bytes() and also eliminated two more compiler warnings.

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

Freed memory buffer that was allocated in previous commit.

Fixed stream leak in modified code

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>

Fixed unchecked malloc() error

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.

Changed the xf_tsmf.c code to test for zero visible rectangles for all platforms.  Modified arc4random() change to actually have a skeletal implementation in winpr for BCryptGenRandom() on all platforms.

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.

Fixed linker warning about insecure crypto and fixed compiler warning about unsigned being compared to less than zero

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.