Merge pull request #3501 from DavBfr/fix-few-memory-leaks

Fix few memory leaks

Merge pull request #3512 from nathankidd/redundant-rfx-conversion

client/X11: Only upconvert < 24 bit RFX tiles

client/X11: Only upconvert < 24 bit RFX tiles

Fixing this logical typo gives a slight performance boost /
CPU usage reduction on heavy loads.

Fix smartcard channel disconnect

Fix Memory Leaks in Printer Redirection

Fix Memory Leaks in RdpSnd

Fix Memory Leaks in RdpDR

Fix Memory Leaks in Disp channel

Merge pull request #3482 from realjiangms/fix_gfx_win8

server/shadow: Fix regression with rdp8.0

server/shadow: Fix regression with rdp8.0
Shadow server crash with rdp8.0 with gfx enabled.
Root Cause: rdp8.0 is not support in shadow gfx and rdpgfx_caps_advertise returns an error. However setChannelError crashs because context->errorDescription is NULL
1. Fix shadow gfx to handle rdp8.0
2. Initialize context->errorDescription for server side new-context

Merge pull request #3435 from akallabeth/uuid_find_fix

CMake FindUUID fixes.

Merge pull request #3284 from ondrejholy/endianness

Endianness fixes

Merge pull request #3316 from realjiangms/server_gfx

rdpgfx: Implementation for server side channel

Merge pull request #3476 from FreeRDP/mh-fix-transport

libfreerdp/core: transport_write unchecked parameters

libfreerdp/core: transport_write unchecked parameters

transport_write did not check if transport or bio are
set. The transport read checks it. In using fastpath
and for example a mouse input is sent, the transport
is never checked and can cause a segfault.

rdpgfx/server: Fix for windows10 clients
1. Fix order of gfx reset and new-surface. Windows10 client will show black screen with this issue(FreeRDP itself is dramatically immune to this issue)
2. Handle RDPGFX_QOE_FRAME_ACKNOWLEDGE_PDU for FPS control

rdpgfx/server: More minor fix

rdpgfx/server: More minor fix according to comments:
Not use sizeof on packet memory structure for stream length check/estimating and use number directly.
Most of the packet size could be obviously calculated/verified from code and comments in same function.
Macro is defined for those were referenced multiple times.

rdpgfx/server: Minor fix: allow configurable codec parameter

rdpgfx/server: Minor fix: use switch for pdu length estimation

rdpgfx: more minor changes
1. Fix style for channels/rdpgfx/server/rdpgfx_main.c according to scripts/format_code.sh
2. Refactor and introduce APIs to allow rdpgfx server channel running in existing thread (similar to rdpsnd)

rdpgfx: various fixes according to comments
1. Fix stream leak in rdpgfx
2. Make src data const in zgfx. Harden zgfx to be independent to byte order
3. Fix written bytes return value in channel write
4. Add check for return value in shadow_client.c
5. Add gfx callback to send surface command with frame marker pdu.
6. Check remain length for recv subroutine
7. Fix compile errors

rdpgfx: Implementation for server side channel
server/shadow: support h264 codec with gfx channel

Merge pull request #3451 from awakecoding/nla_server_fix

NLA Server Side Improvements

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 #3449 from awakecoding/android_toolchain_fix

cmake: fix Android toolchain missing usage of CMAKE_PREFIX_PATH

cmake: fix Android toolchain missing usage of CMAKE_PREFIX_PATH

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.

Fixed UUID find.

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>.

winpr/crt: Make TestString endian-independent

Wide char strings use always little endian encoding since commit f722dc5.
Use only strings in little endian to make the tests endian-independent.

winpr/crt: Fix endianness in string utils

All WCHAR strings are stored as little endian after commit f722dc5,
therefor WCHAR string utils have to be changed appropriately.

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.