Handle API updated.

WSAEventSelect: Handle lNetworkEvents == 0

WSAEventSelect did ignore the lNetworkEvents argument.
In case this argument is 0, the non blocking socket must
be set to blocking again to mimic windows behavior.

Added write event support to handle functions.

Allows the WinPR HANDLE functions WaitForSingleObject and
WaitForMultipleObjects to signal in case of write events.
This is used by CreateFileDescriptor and SetEventFileDescriptor,
which got an API change accomodating for this new feature.

Added convenience function winpr_log_backtrace.

Merge pull request #2759 from bmiklautz/fix/cliprdp

x11/cliprdr: handle empty format names

x11/cliprdr: handle empty format names

The recently added strdup checks ignored the fact that format names
can be NULL.

Merge pull request #2747 from ilammy/egfx-missing-events

Do not discard events when updating EGFX surface mapping

Merge pull request #2758 from bmiklautz/fix/CreateProcess

CreateProcess misc fixes

CreateProcess: fix setting of default handler

If SA_SIGINFO isn't set in the flags sa_handler is used
instead of sa_sigaction.

This fixes also the compiler warning:

FreeRDP/winpr/libwinpr/thread/process.c: In function ‘_CreateProcessExA’:
FreeRDP/winpr/libwinpr/thread/process.c:282:20: warning: assignment from
       incompatible pointer type [enabled by default]

CreateProcess: two fixes

* change to lpCurrentDirectory if set even if no token was supplied
* fix wrong check - add missing !

This was part of akallabeth's PR #2714.

Merge pull request #2757 from nfedera/nf-fix-createprocess-signals

winpr: fix CreateProcess signal handler issues

winpr: fix CreateProcess signal handler issues

The calling thread of CreateProcess can be in any library and
can have arbitrary signal masks and handlers.

We now save the caller's mask and  block all signals before forking.
After fork:
- child resets the handlers and unblocks all signals.
- parent restores the caller's original signal mask.

Merge pull request #2754 from akallabeth/mixed_code

Fixed mixed declarations and code.

Merge pull request #2750 from realjiangms/fix_win32_wsaevent

Fix event created for WSAEventSelect

Merge pull request #2753 from akallabeth/nla_null_auth_fix

nla: Fixed handling of optional TSPasswordCreds field.

Fixed mixed declarations and code.

NLA: Fixed length check.

Fixed handling of optional TSPasswordCreds field.

Merge pull request #2752 from akallabeth/nla_user_length_fix

NLA decrypt credentials fixed.

NLA decrypt credentials fixed.

Merge pull request #2738 from bmiklautz/ffuncs

hardening: check fread and fwrite return values

Integrate pull request feedback

* unify fwrite usage - set nmemb to 1 and the size to the size to write.

Fix event created for WSAEventSelect: The event we pass to WSAEventSelect should be WSAEVENT instead of normal event.
From MSDN, it looks same as CreateEvent(NULL, FALSE, FALSE, NULL):
The WSACreateEvent function creates a manual-reset event object with an initial state of nonsignaled. The event object is unnamed.
However they are not really equivalent. When we use normal event, the WSAEventSelect still works but the event appears to be 'auto-reset'.

client/x11: do not discard events when updating EGFX surface mapping

Passing True to XSync() discards any pending X11 events. Occasionally
this caused ButtonRelease or KeyRelease to be lost and not forwarded
to the remote computed, leading to stuck keys and buttons.

This should resolve issue #2391

Merge pull request #2745 from mfleisz/fix_makecert

makecert: Fix double free crashes

makecert: Fix incorrect return value check

makecert: Formatting

makecert: Add strdup checks

makecert: Fix double free crashes

Merge pull request #2740 from realjiangms/shadow_fix_cert_2708

server/shadow: Fix incorrect handle of makecert_context_process.

Merge pull request #2741 from awakecoding/master

egfx progressive codec frame handling

server/shadow: Fix incorrect handle of makecert_context_process.
Also fix another typo for makecert_context_set_output_file_name

libfreerdp-codec: fix egfx artifacts resulting from incorrect handling of rects inside the same egfx frame

hardening: check fread and fwrite return values

Merge branch 'master' of github.com:FreeRDP/FreeRDP

channels/rdpgfx: update debug output

Merge pull request #2724 from bmiklautz/leak_fix

Fix leaks in certificate and identity handling

Merge pull request #2737 from nfedera/nf-fix-gdi-return-value-weirdness

libfreerdp/gdi: fixed gdi return value madness

nla: clear identity memory before releasing

Merge pull request #2727 from akallabeth/leak_fixes_reformat

Fixed leaks, NULL dereferences and broken init.

libfreerdp/gdi: fixed gdi return value madness

Mostly booleanization to comply with the MS API

Merge pull request #2719 from bmiklautz/pull/2481

OSS, tsmf, usb and BSD fixes and improvements

Merge pull request #2736 from akallabeth/windows_client_fix

Fixed windows client pre_connect return.

Merge pull request #2734 from giox069/master

Fixes for software GDI issues #2732 and #2137

Fixed windows client pre_connect return.

Merge pull request #2735 from akallabeth/clipboard_temp_file_name_conversion_fix

[Clipboard] Fixed out of bound array access.

Merge pull request #2729 from akallabeth/win_reg_key_by_vendor_product_define

Replaced hard coded registry keys with cmake defines.

Fixed out of bound array access.

Returing OK when there is nothing to draw after clipping, fixes #2732

Signed glyph offset calculation, fixes #2137

Merge pull request #2731 from akallabeth/mutex_cleanup_fix

Fixed mutex trylock in MutexCloseHandle.

Fixed mutex trylock in MutexCloseHandle.

Merge pull request #2728 from bmiklautz/unsigned

Fix possible endless loops on cleanup.

clipboard: fix of by one error on cleanup

Integrate pull request feedback

Use while instead of for with additional variable where appropriate.

Fixed cleanup of resources, only in error case now.

Replaced registry keys with cmake defines.

Fix possible endless loops on cleanup.

Some cleanup code possibly create endless loops because an unsigned
type was used as run variable but the check was >= 0 in the for loop.

Fixed cleanup of MachineAddresses and MachinePorts.

Fixed loop condition.

Fixed memory leak.

Fixed sizeof misuse.

Fixed a resource leak.

Fixed a resource leak.

Fixed a resource leak.

Fixed a resource leak.

Merge pull request #2717 from realjiangms/shadow_encoder_fps

server/shadow: shadow encoder related enhancement/fix.

Fixed leaks, NULL dereferences and broken init.

Merge pull request #2725 from akallabeth/warning_fixes_more_more_more

Warning fixes more more more

Removed unused variable and incorrect comment.

ifdef variables used with specific build options.

Fix leaks in certificate and identity handling

Merge pull request #2721 from akallabeth/warning_fixes_some_more

Some warnings fixed

Fixed memory leak.

Added missing argument checks.

Fixed malloc with 0 size.

Fixed uninitialized value.

Fixed uninitialized value.

Fixed grabage return value.

Fixed NULL pointer dereference.

tsmf_oss: fix compiler warning

Merge pull request #2708 from bmiklautz/dupdup

alloc and strdup checks

core: use error instead of debug

When the function would return with an error print an error message
instead of a debug message

nla and cmdline: integrated feedback

* fix possible problems with 0 size lengths
* add return value checks

server/shadow: shadow encoder related enhancement/fix.
1. Export fps related API so that subsystem implementation no longer need to know about details in encoder structure.
2. Discard frameList dictionary.
The 'value' in this dictionary is never used and not properly free'ed when client is disconnected.
The dictionary was used to calculate 'inflight' frame count. Once an ACK is received from client, an item in the dictionary is removed.
We then calculate 'inflight' frame by the count of the items in the dictionary.
However, some rdp clients (win7 mstsc) skips frame ACK if it is inactive, ACK of some frame would actually never arrive.
We actually don't need the dictionary. We only need to record the latest acknowledged frame id, and the difference between last sent frame id is the inflight frame count.
3. Minor fix in default fps calculation. encoder->frameAck is wrongly used as integer while it's actually bool flag.

oss: fix formating

windows: fix compilation and warnings

misc: integrate pull request feedback

client/locale: properly check popen return value

popen returns NULL if an error occurs and not < 0.

client/android: reset debug level to default

* WLOG debug level was hard coded to DEBUG which generates a lot of
  noise
* remove indecent and unneeded debug message

sspi/ntlm: integrate pull request comments

cmd line: add missing checks

* strdup
* some allocs

cache: fix regression

Fix a regression in cache_new introduced in commit
b4f147e242ea396ef92082d29866e5ab7d041b4e

codec/planar: regression - fix conflicting types

Update planar.h to fix the conflicting types of
freerdp_bitmap_planar_delta_encode_planes.

Fixes for malloc / calloc + other fixes

This patch contains:

* checks for malloc return value + treat callers;
* modified malloc() + ZeroMemory() to calloc();
* misc fixes of micro errors seen during the code audit:
** some invalid checks in gcc.c, also there were some possible
integer overflow. This is interesting because at the end the data are parsed
and freed directly, so it's a vulnerability in some kind of dead code (at least
useless);
** fixed usage of GetComputerNameExA with just one call, when 2 were used
in misc places. According to MSDN GetComputerNameA() is supposed to return
an error when called with NULL;
** there were a bug in the command line parsing of shadow;
** in freerdp_dynamic_channel_collection_add() the size of array was multiplied
by 4 instead of 2 on resize

A malloc() pass on WinPR

This patch treats remaining non-checked mallocs. And changes to calloc in places
where it makes sense

client/android: misc updates and fixes

* increase minimum required SDK to 14
* change functions from void to BOOL where required to have proper
  return values
* update the JAVA classes and JNI accordingly
* add return value checks for *alloc and strdup

android: fix misc compiler warnings

with gcc version arm-linux-androideabi-gcc (GCC) 4.8

winpr/wtsapi: disable tests that block on windows

winpr/thread: fix test compilation on windows

Use DWORD instead of QWORD.