Merge pull request #4610 from akallabeth/cmd_strtol_fix

Fix #4597: Do string argument checks before a possible strtol

Merge pull request #4623 from akallabeth/mac_sound_backend_fix

Fixed mac sound backend initialization.

Merge pull request #4627 from akallabeth/clang_warning_fixes

Clang warning fixes

Fixed uninitialized return and early resource cleanup.

Fixed uninitialized value.

Fixed missing NULL pointer checks.

Merge pull request #4624 from akallabeth/pth_fix

Fixed /pth: Consistently treat the hash offset to password length.

Do not set password to identity if pth is used.

Merge pull request #4611 from akallabeth/argument_warnings

Argument warnings

Merge pull request #4625 from dualmoon/fix-hisher-pronouns

First person "they" pronouns in logoff message

Fixed formats supported by backend.

Added error log messages.

First person "they" pronouns in logoff message

Fixed /pth: Consistently treat the hash offset to password length.

Fixed mac sound backend initialization.

Merge pull request #4620 from akallabeth/bio_ctrl_fix

Bio ctrl fix

Fixed formatting.

Fixed #4476: broken casts/variable sizes for custom BIO calls.

Merge pull request #4453 from akallabeth/sound_channel_refactor

Sound channel refactoring

Merge pull request #4617 from kevans91/spurious

Upstream the rest of our local FreeBSD patching

Merge pull request #4613 from kevans91/file64

Enable 64-bit file support on FreeBSD as well

Merge pull request #4616 from kevans91/timerfd-nonblock

Pass TFD_NONBLOCK to timerfd_create to avoid later fcntl

Merge pull request #4596 from p-pautov/rdg_ssl_fixes

RDG related fixes for better compatibility with mstsc

Fixed treat_wave sample length and checks.

Merge pull request #4614 from kevans91/cmakemod

Abstract away CMake module installation setting, install differently on FreeBSD

Merge pull request #4618 from kevans91/buildtest

Pull in the libepoll-shim includes for libwinpr's synch/test

Pull in the libepoll-shim includes for libwinpr's synch/test

This allows FreeBSD to successfully build with BUILD_TESTING enabled. Currently,
only 3/184 tests fail:

 13 - TestLibraryLoadLibrary (Failed)
 14 - TestLibraryGetProcAddress (Failed)
 15 - TestLibraryGetModuleFileName (Failed)

These failures are probably due to a lack of GetModuleFileNameA implementation
on FreeBSD.

Upstream the rest of our local FreeBSD patching

FreeBSD/DragonflyBSD are also UNIX platforms, as far as trio is concerned, and
we don't need to be setting the CMAKE_INSTALL_RPATH on FreeBSD.

Pass TFD_NONBLOCK to timerfd_create to avoid later fcntl

Merge pull request #4576 from ccpp/bugfix-rdg-poll

Fix polling in RDG

Merge pull request #4612 from kevans91/mandest

Follow OpenBSD convention when installing manpages on FreeBSD

Enable 64-bit file support on FreeBSD as well

Use SetFreeRDPCMakeInstallDir where CMake modules are installed

Add SetFreeRDPCMakeInstallDir function to abstract away platform differences

Different platforms/systems may install CMake modules into different places. For
instance, FreeBSD will install modules into ${PREFIX}/share/cmake/Modules while
Linux distributions might install into ${PREFIX}/lib/cmake.

This leaves us with a useful abstraction- consumers only need to care about
the name of their subdirectory, and only one place needs to be patched to change
where this subdirectory lands.

Follow OpenBSD convention when installing manpages on FreeBSD

Both OS install third party man pages into ${PREFIX}/man/man${SECTION}, rather
than the convention established in the base system of share/man/man${SECTION}.

Fixed invalid argument parameter

Fixed argument warning.

Fix #4597: Do string argument checks before a possible strtol

strtol sets errno, which must be reset if it is no abort condition.
Invert the comparisons to avoid that.

Merge pull request #4608 from akallabeth/posix_timer_cmake_check

Determine posix timer use by function availability

Merge pull request #4570 from akallabeth/toolchain_fix

android Toolchain fix

Merge pull request #4603 from hardening/socksplus

SOCKS5 proxy support

Determine posix timer availability by function availability

To avoid having to whitelist every os starting to implement posix
timer API just detect if the functions are available.

proxy: cleanup SOCKS support and add user/password support

Merge pull request #4606 from kevans91/libressl-fixes

Pull in the LibreSSL compatibility patches from FreeBSD

Merge pull request #4609 from akallabeth/clipboard_return_check

Added proper return value checks for clipboard data.

Merge pull request #4607 from kevans91/epoll

FreeBSD: epoll/timerfd support

socks proxy reply fix

Added proper return value checks for clipboard data.

Use shared memory instead of temp files on FreeBSD-compatible OS

Pull in libepoll-shim for the bits that require timerfd/epoll

libepoll-shim is our implementation of this API on top of kevent. It supplies
the same headers and a library, but we don't install it in any of the default
include search paths when it comes in through ports on an as-needed basis.

This set of changes is restricted to FREEBSD-compatible OS, which includes
DragonflyBSD and FreeBSD.

Set BSD/FREEBSD for DragonflyBSD

DragonflyBSD was a fork of FreeBSD once upon a time. As far as FreeRDP is
concerned, DragonflyBSD is FreeBSD-compatible and DragonflyBSD in-fact uses
the FreeBSD ports tree (+ patches)

Pull in the LibreSSL compatibility patches from FreeBSD

Added libavcodec version check for dsp-ffmpeg component.

Abort compilation with a specific error if the libavcodec module
detected is too old.

Merge pull request #4548 from akallabeth/autoreconnect_fix

Autoreconnect fix

Merge pull request #4589 from oshogbo/token

Recognize only the cookie format anything else treat as token.

Fix Windows build.

Fallback to RDG RPC transport only if server does not support RDG HTTP and
error out in other cases - invalid RDG SSL cert, bad credentials, PAA failue, etc.

Remove some unused functions.

Apply "authentication level" RDP property only to non-RDG connections (as mstsc does).

Don't use CertificateName setting for RDG connections.

Merge pull request #4586 from Awingu/cbytes-fix

Fix inconsistent cchReaderLen value between WinScard and pcsc-lite.

Merge pull request #4588 from oshogbo/tokencookie

Fix comments where is cookie and where is token.

Fail on unimplemented BIO_ctrl for /gt:rpc

Implement BIO_ctrl more correctly for RDG

See #3602

This change is discussed in https://github.com/FreeRDP/FreeRDP/pull/4576#pullrequestreview-113381733

Revert useless part of the bugfix

This reverts commit 589d2ec62a6efc3af18b628c5f545c73c90dd331.

https://github.com/FreeRDP/FreeRDP/pull/4576#pullrequestreview-113378805

Recognize only the cookie format anything else treat as token.

If in the RDP file we will set loadbalanceinfo.
Instead of getting the cookie value we will get load balance info.

For example:
0000 03 00 00 2a 25 e0 00 00 00 00 00 74 73 76 3a 2f ...*%......tsv:/
0010 2f 56 4d 52 65 73 6f 75 72 63 65 2e 31 2e 41 48 /VMResource.1.AH
0020 0d 0a 01 00 08 00 0b 00 00 00                   ..........

The MSFT-SDLBTS document don't describe this behavior.

For this reason lets treat the token as anything ended with seqance
CR and CL. To be honest we already did that because in the core/connection.c
file where we are seting the routing_token to the LoadBalanceInfo.

Fix comments where is cookie and where is token.

Fix inconsistent cBytes value between WinScard and pcsc-lite.

SOCKS proxy support

Merge pull request #4539 from p-pautov/rdg_fixes

RDG HTTP transport fixes and refactoring

Fix BIO_get_fd for RDG, again.

Merge pull request #4543 from oshogbo/master

Fix variable passsed to HashCallback with MIC.

Fix #3602 by implementing BIO_get_fd correctly for RDG

Fix timeout for polling (partly fixes #3602)

Remove MessageIntegrityCheck from context.

Merge pull request #4572 from weberhofer/patch-1

Include geometry.h in video channel

Include geometry.h in video channel

Fix for #4562

Avoid buffer to struct cast.

Send correct packet size in case of PAA. Some cleanup.

Content-Length and Transfer-Encoding are mutually exclusive.

Consolidate IN/OUT data connections establishment into common function and clean up related code.

Move NTLM auth related code into dedicated functions.

Consolidate rdg_tls_out_connect/rdg_tls_in_connect into single function. This also fixes connections to RDG server via proxy.

Removed unused "readEvent" - no one checks if it was set.
Removed needless rdg_check_event_handles declaration.

Consolidate rdg_send_in_channel_request/rdg_send_out_channel_request into single function.

Explicitly skip "seed" payload on RDG OUT connection, to avoid issues when it's split over several SSL records.

Moved some repeated "read all" code into function.

Fixed redirection with session brokers.

* Only reconnect channels on redirect, if they have already been connected.
* Prefer TargetNetAddress over FQDN to connect.

Add default values for keystore variables.

Ensure audin channel uses supported protocol version 1.

rdpsnd: add support for wave2 PDU in client (#2)

To achieve this the announced version number is bumped to 8.

Updated OpenSSL version in build scripts.

Added patch from f-droid repo. (Christian Plattner)

Merge pull request #4567 from akallabeth/sanitizer_checks_refined

Refined checks for sanitizers.

Merge pull request #4568 from akallabeth/rails_bad_window_fix

Check for window existence in remote app mode before access

Check for window existence in remote app mode before access

X11 is asynchronous, so events arriving in xf_event_PropertyNotify
may access a window that has already been discarded. Check that before
reading properties.

Refined checks for sanitizers.

Merge pull request #4557 from akallabeth/connect_error_fix

Set connection error if TCP connect fails.

Merge pull request #4544 from oshogbo/nSize

Don't restrict the hostname size.

Merge pull request #4547 from andreesteve/gatewayip

Gateway client (RDG) must connect to same IP/server for both channels