fix race conditions, tests and some invalid return values

Since the current winpr implementation for overlapped operations is
incomplete and buggy, all affected functions will now fail if they are
called with a set FILE_FLAG_OVERLAPPED flag or a non-null pointer to
a OVERLAPPED structure.

winpr/nt:
- use proper one-time initialization on win32
- fix TestNtCreateFile
- fix broken/incomplete _RtlAnsiStringToUnicodeString
- unimplemented functions return appropriate error codes

winpr/pipe:
- improved TestPipeCreateNamedPipe
- rewrite the completely broken TestPipeCreateNamedPipeOverlapped test

rdtk:
- improve test and don't blindly return success

winpr/synch:
- fix race condition in TestSynchTimerQueue

winpr/ssspi:
- fix TestEnumerateSecurityPackages printf output
- fix TestQuerySecurityPackageInfo printf output

winpr/environment:
- fix GetEnvironmentStrings printf output

winpr/comm:
- unimplemented functions return appropriate error codes

winpr/io:
- unimplemented functions return appropriate error codes

winpr/thread:
- implement SwitchToThread() via sched_yield()

Merge pull request #3379 from nfedera/fix-winpr-wtsapi

winpr/wtsapi: fixed race conditions and tests

Merge pull request #3376 from realjiangms/fix_client_gfx

channels/rdpgfx: Fix for gfx client

Merge pull request #3371 from realjiangms/fix_x11_shadow_race

Server/shadow: Protect X operations on shared subystem X11 display with X11 display lock

winpr/wtsapi: fixed race conditions and tests

Merge pull request #3377 from nfedera/fix-channel-name-parsing

freerdp/core/gcc: channel name hardening

freerdp/core/gcc: channel name hardening

According to [MS-RDPBCGR 2.2.1.3.4.1 Channel Definition Structure]
the channel name must be an 8-byte array containing a null-terminated
collection of seven ANSI characters that uniquely identify the channel.

We did not check if the transmitted name was null-terminated which
could have the usual severe effects on stabiliy and security since
the channel name is used in several functions expecting a null-
terminated string (strlen, printf, etc.)

Merge pull request #3375 from nfedera/fix-sizet-format-segfaults

fix segfaults casused by size_t format specifier

channels/rdpgfx: Make freerdp_client_codecs_prepare calls codec reset. It fix broken h264 client from #3328. Also it doesn't make sense that we don't need width and height for codec initialization while we need them for codec reset

fix segfaults casused by size_t format specifier

win32/msvc cc does not recognize the %z format specifier which caused
invalid references and segfaults on win32.
Until FreeRDP gets format specifier macros we'll cast size_t to
unsigned long and use the %lu specifier.

Also simplified winpr_backtrace_symbols() a little bit and fixed it
to allocate the correct amount of bytes for the return buffer.

Merge pull request #3372 from nfedera/fix-winpr-test-library

winpr/library: fix win32 test and some cleanup

winpr/library: fix win32 test and some cleanup

On Windows we seem to have to load the TestLibrary[AB] test libraries
from in same folder the test executable runs.

Also removed the empty RemoveDllDirectory, SetDefaultDllDirectories,
AddDllDirectory tests and the redundant FreeLibrary test.

TestLibrary now works and succeeds on Win32.

sadasd

Merge pull request #3374 from nfedera/fix-winpr-path

winpr/path: fix PathCchFindExtensionA and more

winpr/path: fix PathCchFindExtensionA and more

- PathCchFindExtensionA had an off-by-one error when verifying the
  required null termination
- TestPathCchFindExtension used unicode strings when testing the
  *A (ASCII) functions
- The PathAllocCombineW implementation (which is still buggy has
  hell) used strlen to calculate the lenght of unicode strings

TestPath now succeeds on WIN32

Merge pull request #3373 from nfedera/fix-winpr-test-pipe

winpr/pipe: fix CreateNamedPipe test

winpr/pipe: fix CreateNamedPipe test

GetLastError() was not always checked for ERROR_PIPE_CONNECTED which
indicates success if ConnectNamePipe returns FALSE.

TestPipe now also succeeds on Win32

Server/shadow: Protect X operations on shared subystem X11 display with X11 display lock

Merge pull request #3369 from realjiangms/fix_mouseback_2796

Server/shadow: Fix issue 2796. Mouse cursor draw back issue

Server/shadow: Fix issue 2796. Mouse cursor draw back issue

Merge pull request #3370 from nfedera/fix-winpr-test-thread-createprocess

winpr/thread: fix TestThreadCreateProcess

winpr/thread: fix TestThreadCreateProcess

On WIN32 TestThread now works and is expected to succeed

Merge pull request #3365 from nfedera/fix-winpr-synch-tests

winpr/synch: fix tests

winpr/synch: fix tests

TestSynchTimerQueue:
- fixed race condition

TestSynchWaitableTimerAPC:
- Use WaitForSingleObjectEx since the thread must be in an alterable state

TestSynch is now expected to succeed on WIN32

Merge pull request #3363 from nfedera/fix-mutex-implementation

winpr/synch: fix mutex implementation

winpr/synch: fix mutex implementation

- Mutex is recursive on Windows; as a consequence we have to use
  the pthread PTHREAD_MUTEX_RECURSIVE type
- Adapt MutexCloseHandle accordingly
- ReleaseMutex returned TRUE even if pthread_mutex_unlock failed
- Fixed and improved the TestSynchMutex ctest

Merge pull request #3357 from realjiangms/fix_rdpsnd_race

shadow/rdpsnd: Fix race condition in rdpsnd channel server.

Merge pull request #3359 from bmiklautz/eventfd_leak

winpr/event: fix file descriptor leak

Merge pull request #3349 from byteboon/master

kerberos authentication fix

winpr/event: fix file descriptor leak

SetEventFileDescriptor overrides the internal file descriptor of the
event but didn't close it. Now if the descriptor is closed if it isn't
marked as attached.

shadow/rdpsnd: Fix race condition in rdpsnd channel server. The output buffer and format parameters are not protected. This cause some data inconsistence if rdpsnd has own thread

Merge pull request #3356 from rinov/master

Update mf_mountain_lion.c

Update mf_mountain_lion.c

Remove duplicate return statement in mf_mlion_stop_getting_screen_updates()
This change is no impact anywhere.

Merge pull request #3353 from bmiklautz/no_shared_libs

build: fix builds with -DBUILD_SHARED_LIBS=OFF

build: fix builds with -DBUILD_SHARED_LIBS=OFF

When no shared libs are built is required to export all generated
archive files otherwise programs linking against
freerdp-client/freerdp-server might miss symbols.

Fixes #3350

fixed kerberos authentication

Details: cbSecurityTrailer was assumed to be a fixed length for all signatures, however for Kerberos authentication the signature may generate smaller than this value

Merge pull request #3333 from akallabeth/memleak_fixes

Memleak fixes

Merge pull request #3340 from rjcorrig/kb3153731

winpr: KB3153731: May 2016 DST update for Azerbaijan, Chile, Haiti and Morocco

Merge pull request #3341 from awakecoding/xcrush_fix

proper xcrush context reset fix

libfreerdp-codec: make proper xcrush context reset fix

KB3153731: May 2016 DST update for Azerbaijan, Chile, Haiti and Morocco

Merge pull request #3335 from bmiklautz/wfreerdp_naming

wfreerdp: fix name build without client interface

Merge pull request #3339 from awakecoding/xcrush_fix

libfreerdp-codec: fix xcrush decompression, revert remote control fix

libfreerdp-codec: fix xcrush decompression, revert remote control fix

wfreerdp: fix name build without client interface

When built without client-interface the binary should be called wfreerdp
as well (currently it is wfreerdp-client)

Fixed memory leaks.

Fixed NULL pointer access.

Merge pull request #3329 from awakecoding/hv_fix

freerdp: fix Hyper-V connectivity, fix issues #2421 and #3325

Merge pull request #3330 from awakecoding/xcrush_fix

xcrush context flush fix

libfreerdp-codec: fix xcrush context flush

freerdp: fix Hyper-V connectivity, fix issues #2421 and #3325

Merge pull request #3328 from awakecoding/gfx_fix

egfx codec context reset fix, Windows Server 2012 RemoteFX empty tile set fix

libfreerdp-codec: fix error on empty RFX tileset, which can sometimes happen

channels/rdpgfx: fix resetting of codec contexts

Merge pull request #3260 from awakecoding/master

Universal Windows Platform (UWP) Port

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

winpr: fix definitions of synchronization barrier and interlocked functions

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

Conflicts:
winpr/libwinpr/bcrypt/CMakeLists.txt

Merge pull request #3318 from bmiklautz/multilayoutpdu

core: don't advertise multi layout PDUs

Merge pull request #3321 from bmiklautz/client_interface

client: disable the client interface per default

client: disable the client interface per default

libxfreerdp-client doesn't necessarily provide a stable interface
therefore it isn't built and installed anymore per default.

To archive the same behavior as before this change -
libxfreerdp-client.so built and a library version set - use
-DWITH_CLIENT_INTERFACE=ON -DCLIENT_INTERFACE_SHARED=ON
when running cmake.

This also fixes the build on windows without WITH_CLIENT_INTERFACE
enabled.

core: don't advertise multi layout PDUs

If support for multi layout PDUs is advertised there are known issues
with Windows 7 and Windows 2008R2. Until those are resolved generally
disable multi layout PDUs.

See #3114 for details.

Merge pull request #3282 from akallabeth/input_arg_checks

 Added argument checks for exported input functions.

Merge pull request #3263 from akallabeth/tofu

Added command line option /cert-tofu

Merge pull request #3314 from bmiklautz/refactoringx

Refactoring and fixes

Added argument checks for exported input functions.

pkg/deb: update the package

Add library libwinpr-tools

libwinpr-tools is a replacement for winpr-makecert-tool.a. Currently
it's basically the same as winpr-makecert-tool.a but in future
functionality that doesn't fit directly in winpr will be added here.

client/X11: fix build without client interface

Fix build when -DWITH_CLIENT_INTERFACE=OFF is used.

Merge pull request #3313 from bmiklautz/typos

libfreerd/color: fix typo

libfreerd/color: fix typo

Source: http://anonscm.debian.org/cgit/collab-maint/freerdp2.git/commit/?id=f3c1f25bbfe0f17a0d1317096984b73db6fa1f38

Merge pull request #3311 from bmiklautz/static_rdtk

Don't export/install rdtk

Merge pull request #3312 from bmiklautz/commtests

Disable comm tests per default

rdtk: fix windows build

Don't do export any functions.

Disable comm tests per default

comm tests require a serial device for testing. If the test environment
isn't available the tests will return errors therefore the tests are
now disabled per default. They can be (re-)enabled by using the cmake
option BUILD_COMM_TESTS.

Don't export/install rdtk

Although exported rdtk isn't really mature and stable yet therefore it is
not exported/installed anymore. In case FreeRDP is built without
static libraries librdtk is installed as part or the
FreeRDP-ShadowTargets exports as this is the only place it is used right
now.

Merge pull request #3305 from akallabeth/dynamic_channel_crash_fix

Dynamic channel crash fix

Merge pull request #3250 from mfleisz/cssp_v3

core: Add support for CredSSP version 3

Dynamic channel internal functions now static.

ListDictionary handle NULL list argument.

Merge pull request #3296 from akallabeth/clipboard_file_fix

Fix CB_FILECONTENTS_REQUEST message decoding.

Fix CB_FILECONTENTS_REQUEST message decoding.

Do not require optional fields to be present as required by
[MS-RDPECLIP] 2.2.5.3 File Contents Request PDU (CLIPRDR_FILECONTENTS_REQUEST)

Merge pull request #3294 from bmiklautz/misc_build

Misc build and test fixes

Merge pull request #3287 from bmiklautz/linking_fixes2

build: add missing dependencies

Merge pull request #3293 from nfedera/mitigate-cbclientaddress-verification

core: allow TS Gateway protocol violation

build: disable -Wl,--no-undefined per default

As building with -Wl,--no-undefined might cause issues when
combined with other linker flags (e.g. with -fsanitize=address see #3289)
it is now disabled per default but can be re-enabled with
-DWITH_NO_UNDEFINED=ON.

It is enabled for ci builds on linux to find undefined symbols.

core: allow TS Gateway protocol violation

According to [MS-RDPBCGR 2.2.1.11.1.1.1] the TS_EXTENDED_INFO_PACKET
structure's cbClientAddress field must include the _mandatory_ NULL
terminator of the clientAddress field in its byte count.
However, connections proxied via Microsoft's TS Gateway set the
cbClientDir value to 0.

Tests: Disable TESTS_WTSAPI_EXTRA per default

TESTS_WTSAPI_EXTRA was mistakenly always enabled if BUILD_TESTING
was enabled.

Merge pull request #3285 from rjcorrig/kb3148851

winpr: KB3148851: Updates to Russian Time Zones for 2016

build: add missing dependencies

Add all missing dependencies found when building with
STATIC_CHANNELS=OFF.

Fixes #3281

KB3148851: Updates to Russian Time Zones for 2016

Merge pull request #3274 from bmiklautz/linking_fixes

Fix build regressions

Merge pull request #3276 from nfedera/nf-fix-ms-android-protocol-violation

core: allow ms android client protocol violation

core: allow ms android client protocol violation

According to [MS-RDPBCGR 2.2.1.11.1.1.1] the TS_EXTENDED_INFO_PACKET
structure's cbClientDir field must include the _mandatory_ NULL
terminator of the ClientDir field.
However, since version 8.1.31.44, the Microsoft Remote Desktop Client
for Android sets cbClientDir to 0.

Merge pull request #3275 from nfedera/fix-server-establish-keys-typo

core: fix broken rdp security (server side)

core: fix broken rdp security (server side)

- fixed typo in rdp_server_establish_keys

build: add missing dependencies

Add all missing dependencies found with --no-undefined. Since
dependencies aren't exported anymore (if not required) it is no necessary
to explicitly list all required libraries.

Merge pull request #3273 from realjiangms/fix_android_crash_rotate_3258

Fix android rotation crash

build: use --no-undefined if available

Fix android rotation crash

Added command line option /cert-tofu