Merge pull request #4711 from akallabeth/command_line_cleanup_rework

Command line cleanup rework

Fixed memory leak in command line value parsing.

FIX: moved declarations above the statements for strict C 90 compliance.

Cleaned up const char** -> char** for argv, since we definitely do modify the argv!
(we overwrite the password and pin arguments).
This implies changes in the argument parsing tests that now must pass a mutable argv
(copied from the statically declared test argvs).
Some other const inconsistency have been dealt with too.

Merge pull request #4694 from akallabeth/mic_crash_fix

Fixed #4693: Fix integer underflow in encoder loop.

Fixed #4693: Fix integer underflow in encoder loop.

Merge pull request #4683 from akallabeth/ffmpeg_found_fix

Nightly build fix

Fixed case of variable names.

Merge pull request #4677 from akallabeth/build_dep_fix

Fix #4672: Check if FFMPEG and OpenH264 detected

Fix #4672: Check if FFMPEG and OpenH264 detected

When FFMPEG or OpenH264 was requested but not detected abort
the cmake run with an error.

Merge pull request #4665 from FreeRDP/akallabeth-patch-1

Update issue templates

Update issue templates

primitives: ensure primitives_get() returns a populated struct

Fixes: https://github.com/FreeRDP/FreeRDP/issues/4658

Merge pull request #4654 from akallabeth/alsa_latency_patch

Return real alsa latency if buffer is filled.

Return real alsa latency if buffer is filled.

Merge pull request #4653 from rjcorrig/kb4093753

winpr: Updates time zone data to April 2018

Update time zone data to April 2018

Merge pull request #4651 from akallabeth/loadepng_memleak_fix

Fixed loadepng memory leak.

Merge pull request #4648 from akallabeth/nsc_free_fix

Fixed #4647: nsc_context_free must not access possibly uninitialized fields

Fix missing sound arrival time for wave2 PDU

Fixed loadepng memory leak.

Merge pull request #4637 from akallabeth/win_ninja_support

VS2017 support: set pdb output directory for Ninja generator.

Merge pull request #4631 from akallabeth/santizer_fixes

Fix #4628: CommandLineFindNextArgumentA must check the current argument

Fixed #4647: nsc_context_free must not access possibly uninitialized fields.

Merge pull request #4638 from akallabeth/known_hosts_comment_support

Added comment support for known_hosts format.

Added comment support for known_hosts format.

Fixed invalid function argument for ntlm_compute_message_integrity_check

Fixed compiler warnings (unused)

Fixed #4629: Only call freerdp_channels_post_connect when it was connected.

In rdp_client_redirect or rdp_client_reconnect freerdp_channels_post_connect must
be called if the channels were connected previously.
This might not be the case, skip that call then.

VS2017 support: set pdb output directory for Ninja generator.

Fix #4628: CommandLineFindNextArgumentA must check the current argument

When determining if there is a next argument first check the current one
for abort criteria.

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