freerdp: make modifications to NLA server-side fixes according to PR comments

freerdp: add configurable NTLM SAM file option for server-side NLA

freerdp: fix sending of TLS alert on NLA failure, add better handling of server-side NLA in shadow server

Merge pull request #3445 from lanoxx/master

docs: Make doxygen silent

Merge pull request #3447 from bmiklautz/package_updates

pkg: update packages

pkg: update packages

* include wlog man page

docs: Make doxygen silent

Merge pull request #3438 from akallabeth/android_build_paths

Fixed android libfreerdp install paths.

Merge pull request #3436 from akallabeth/openh264_1_6_support

Added support for OpenH264 1.6

Merge pull request #3405 from hardening/wlog_man

Add a dedicated man page for wLog

Added gradle.properties to extend heap size.

Fixed android libfreerdp install paths.

Merge pull request #3384 from akallabeth/android64_build

Android64 build fixes

Updated gradle.

Disabled OpenH264 build for pull request tester.

Using android build-tools 24 (23 is broken)

Merge pull request #3402 from doughdemon/master

Compile without <sys/queue.h>

Merge pull request #3433 from akallabeth/wlog_callbac_init_fix

Wlog callback init fix

Proper wLogMessage initialisation.

Initialise WLog PrefixString for each message type.

Added support for OpenH264 1.6

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.

Add an environment variables section to the man

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

man pages needed love

This patch cleans things up so that generated man pages for argument are nicer.

Add a dedicated man page for wLog

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

Make iOS workaround iOS specific

Fixes compilation on systems without <sys/queue.h>.

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.

Updated readme for android.

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

Fixed defined symbols for function check.

Setting lib64 preference now.

Updated toolchain file, added new variable for lib64 detection.

Fixed android library detection.

Fixed missing arch.

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.

Fixed pthread_mutex_timedlock detection.

Added android64 support.

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