Merge pull request #3428 from nfedera/fix-getcomputernameex-lentgh-calculation

winpr/sspi/ntlm: fix computer name len calculation

winpr/sspi/ntlm: fix computer name len calculation

The lpnSize parameter for GetComputerNameEx specifies the total
size of the buffer (in characters).
However, the current code calculated the amount of bytes.
Since only GetComputerNameExA was used and because sizeof(CHAR) == 1
the result was correct but the math was wrong.
Credit goes to @byteboon

Merge pull request #3426 from nfedera/fix-read-extended-info-packet

protocol violation: rdp_read_extended_info_packet

protocol violation: rdp_read_extended_info_packet

The clientTimeZone and all subsequent fields in the Extended Info Packet
(MS RDPBCGR 2.2.1.11.1.1.1) are optional but were not handled as such.

Merge pull request #3418 from llyzs/rdpsnd_server_latency

rdpsnd/server: decrease audio latency and make it configurable.

rdpsnd/server: decrease audio latency and make it configurable.

Merge pull request #3408 from hendwolt/master

fix errors in xdg-shell-xml for protocol creation

Merge pull request #3410 from nfedera/fix-symbols

Fix win32 linker hell and more

winpr: fix win32 linking issues

Depending on the windows target version (_WIN32_WINNT), the used
SDK and the build configuration the linker will see multiple
libraries exporting the same symbols.
To prevent ugly hacks (e.g. modifying cmake's default system
libraries or fragile library linking order chains) we prefix
these functions with "winpr_" and create corresponding defines
to keep the current api names.

freerdp: remove static inet_ntop

inet_ntop is already implemented in winpr

cmake: remove some void statements and fix winpr source group

cmake: replaced STATIC_CHANNELS with BUILTIN_CHANNELS

channels/audin/client/winmm: requires winpr

fix errors in xdg-shell-xml for protocol creation

Merge pull request #3407 from nfedera/enable-tests-on-windows

ci: build tests on windows from now on

ci: build tests on windows from now on

Merge pull request #3406 from nfedera/fix-winpr-ntlm

winpr/sspi/ntlm: fix GetComputerNameExA parameters

winpr/sspi/ntlm: fix GetComputerNameExA parameters

On input, the lpnSize [in, out] parameter for GetComputerNameEx()
specifies the total size of the buffer (in characters).
Several functions in ntlm.c were off by one which caused ntlm to fail
if the netbios hostname's strlen was exactly MAX_COMPUTERNAME_LENGTH.

Merge pull request #3404 from nfedera/fix-pathmakepath

winpr: fix PathMakePathA and TestWLog

winpr: fix PathMakePathA and TestWLog

PathMakePathA:
- This function had an endless loop if no native delimiter was in the string
- Use SHCreateDirectoryExA on Windows
- Replaced old code with a new implementation

TestWLog:
- Windows has no "/tmp" by default
- Use GetKnownPath(KNOWN_PATH_TEMP) for the WLog "outputfilepath"

Merge pull request #3403 from TingPing/freerdp-client.pc

Fix include path in freerdp-client.pc

Fix include path in freerdp-client.pc

Merge pull request #3399 from nfedera/fix-winpr-library2

winpr/library: fix GetModuleFileName and tests

winpr/library: fix GetModuleFileName and tests

- Use correct SetLastError values in GetModuleFileName
- Fix wrong return codes in GetModuleFileName
- Build the TestLibraryA/TestLibraryB libraries always shared and
  put them in the test output directory
- TestLibraryGetModuleFileName always returned success
- Improve TestLibraryGetModuleFileName to also check last error values
  and insufficient buffer sizes
- Change TestLibraryGetProcAddress and TestLibraryLoadLibrary to load
  the TestLibrary from the test executable's directory

Merge pull request #3397 from awakecoding/master

add missing inet_pton implementation on Windows

wayk-now: add missing inet_pton implementation on Windows

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

winpr: fix some tests

Merge pull request #3394 from nfedera/fix-client-x11-locked-mutex

client/x11: don't close locked mutex

client/x11: don't close locked mutex

winpr: fix some tests

TestNtCreateFile, TestPipeCreateNamedPipeOverlapped
- These tests are currently only expected to succeed on _WIN32
- Also reflect the reverse meaning of this fact in the return values

TestSynchWaitableTimer, TestSynchWaitableTimerAPC:
- These tests are currently expected to fail on __APPLE__
- Also reflect the reverse meaning of this fact in the return values

This logic makes sure that we don't forget to fix the tests if the
corresponding WinPR implementations are fixed.

TestLibrary:
- TestLibraryA and TestLibraryB must always get built as shared libraries

Merge pull request #3392 from nfedera/fix-winpr-barrier-deadlock

winpr/synch: fix barrier deadlock in release build

winpr/synch: fix barrier deadlock in release build

Let the compiler know that we're comparing a volatile value.
Otherwise the compiler might nuke the comparison operation
and produce code that will spin endlessly.

Merge pull request #3391 from nfedera/fix-initializewaitabletimer-result

winpr/synch: fix InitializeWaitableTimer result

winpr/synch: fix InitializeWaitableTimer result

InitializeWaitableTimer must not reported success if the operating
sytem does not provide the required functionality.

Merge pull request #3390 from nfedera/fix-winpr-barrier2

winpr/synch: improve barrier test

winpr/synch: improve barrier test

The SYNCHRONIZATION_BARRIER_FLAGS_SPIN_ONLY flag caused this test
to run extremely long if the system has very few processors.
Although this is expected (thread starvation) this will cause a
intolerably long execution time for automated tests.
Changed the number of threads to be calculated dyamically based
on the number of processors.
Also do proper cleanup to prevent memory leaks.

Merge pull request #3388 from nfedera/fix-winpr-pool-synch

winpr: several pool and synch fixes

Merge pull request #3386 from nfedera/fix-winpr-barrier

winpr/synch: rewrite barrier implementation & test

Merge pull request #3389 from nfedera/fix-winpr-synch-timer-queue-segfault

winpr/synch: fix timerqueue segfault

winpr/synch: fix timerqueue segfault

The current experimental/incomplete WinPR timer queue implementation
has several race conditions.
This commit fixes a segfault caused by not unklinking freed timers
from the timer queue timers list.

Merge pull request #3387 from nfedera/fix-wfreerdp-verify-cert-callback

wfreerdp: fix invalid VerifyCertificate callback

wfreerdp: fix invalid VerifyCertificate callback

winpr: several pool and synch fixes

pool:
- the winpr implementation fallback was not used on older windows editions
- drop useless and conflicting TP_CALLBACK_ENVIRON_V3
- fix race conditions by using use proper one-time initialization
- on win32 WinPR tried to load several pool/callback_environment functions
  from kernel32.dll but since these are defined as inline functions in the
  windows headers, no windows edition has ever exported them in any dll.
- removed callback_environment.c and added corresponding static inline
  function to pool.h
- fix segfault in TestPoolWork: CloseThreadpoolWork() must not be called
  if there is a cleanup group associated with the work object since calling
  CloseThreadpoolCleanupGroupMember() already releases the work object

sync:
- The windows headers incorrectly define InitializeCriticalEx support if
  _WIN32_WINNT >= 0x0403 instead of >= 0x0600 (Vista)
- created a compatible define to deal with this issue

winpr/synch/barrier: fix return value and test

- According to the msdn docs DeleteSynchronizationBarrier always returns TRUE
- Added additional error checks to the barrier test

winpr/synch: add spinning support to barrier

Implemented the SYNCHRONIZATION_BARRIER_FLAGS_BLOCK_ONLY and
SYNCHRONIZATION_BARRIER_FLAGS_SPIN_ONLY flags.

winpr/synch: rewrite barrier implementation & test

The synchronization barrier test as well as the actual WinPR
implementation were completely broken.

Merge pull request #3381 from nfedera/fix-winpr-misc

fix race conditions, tests and some invalid return values

winpr/thread: fix SwitchToThread

On some operating systems sched_yield is a stub returning returning -1.
In that case use usleep which should at least trigger a context switch
if any thread is waiting.

Merge pull request #3382 from giox069/master

Fix some memory leaks in freerdp_settings_free()

Fix some memory leaks in freerdp_settings_free()

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