Fixed redirection with session brokers.

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

Do not clear last error if not reconnecting.

Refactored reconnect and redirect API

Reconnect and redirect share the same code on disconnect.
Move that to a single function and export it as it may be required
to terminate the session properly before reconnect is called.

Fixed channel duplicate disconnect handling

Fixed channel duplicate disconnect handling

Fixed channel duplicate disconnect handling

Fixed channel duplicate disconnect handling

Fixed channel duplicate disconnect handling

Fixed channel duplicate disconnect handling

Fixed channel (dis)connect on redirect or reconnect.

Removed unnecessarty string duplications.

Merge pull request #4540 from akallabeth/warning_fixes_v2

Warning fixes v2

Fixed RDP debug message invalid function.

Fixed format string mismatch.

Fixed type mismatches with H264 backend buffer pointers.

Fix pointer type mismatch struct sockaddr_storage* and struct sockaddr*

Merge pull request #4535 from akallabeth/warning_fixes

Warning fixes

Merge pull request #4481 from akallabeth/fastpath_fix

Fastpath uncompressed length issue

Merge pull request #4538 from mfleisz/win_ipv6_fix

core: Fix IPv6 handling on Windows

core: Fix IPv6 handling on Windows

On Windows the sockaddr struct is smaller than sockaddr_in6.
This causes getsockname to fail because the buffer is too small.
The new code uses sockaddr_storage which should be large enough to
hold any supported protocol address structure.

See: http://pubs.opengroup.org/onlinepubs/009696699/basedefs/sys/socket.h.html

Simplified fastpath buffer handling.

Fixed missing return value checks.

Merge pull request #4514 from akallabeth/com_lpt_fix

Com lpt fix

Fixed XPixmap checks.

Fixed format string types.

Removed unnecessary range check.

Fixed integer to float type cast.

Fixed printf format string.

Fixed intrinsic integer overflow.

Adding 128 in 8bit is not possible due to range [-128,127].
Instead subtract -128, that should eliminate the overflow.

Merge pull request #4457 from RangeeGmbH/fix_custom_sizes

Fix /size: /w: /h: with /monitors: (Fix custom sizes)

Merge pull request #4472 from akallabeth/win_mouse_button

[WIN] Added additional mouse mappings.

Merge pull request #4477 from akallabeth/command_line_fix

Detect command line flags in case RDP or MSINCIDENT files are used.

Merge pull request #4416 from akallabeth/audin_fix

Audin fix

Merge pull request #4529 from akallabeth/wlog_cleanup_fix

Fix #4524: Initialize with cleanup handler

Merge pull request #4519 from mfleisz/cssp_nonce_fix

cssp: Fix handling of nonce

Merge pull request #4532 from RangeeGmbH/cmdline_spelling

Fix spelling of network commandline values

Fix spelling of network commandline values

Use __attribute__((destructor)) on non windows systems.

atexit in shared libraries is not universally supported.

Removed all calls to WLog_Init and WLog_Uninit

Since the calls are no longer required remove their usage.

Removed winpr_exit

As the cleanup functions are called by atexit a dedicated
cleanup call is no longer required.

Merge pull request #4527 from jkaivo/master

PCB support

Fix #4524: Initialize with cleanup handler

Use singleton initializer and register cleanup handler for logger.

add support for the "pcb" block in .rdp files provided by Project Honolulu in the VM interface

Merge pull request #4522 from UndefBehavior/fix_SCardStatusW

Fix SCardStatusW function

Fix SCardStatusW function

Terminating null character was inserted in the middle of readerNames instead of
last position in the unicode version of SCardStatus function.
This commit fix it.

cssp: Fix handling of nonce

Fix /size: /w: /h: with /monitor: (Fix custom sizes)

Fix weird rectangle by removing checks

Fixed #4412: memory corruption in audin channel.

Audin channel use dynamic logger where possible.

Merge pull request #4513 from chipitsine/travis-new

initial travis-ci support

Merge pull request #4512 from chipitsine/master

resolve several possible null pointer dereference

initial travis-ci support

resolve several possible null pointer dereference

issue detected by cppcheck

[channels/drive/client/drive_main.c:454] -> [channels/drive/client/drive_main.c:443]: (warning) Either the condition '!irp' is redundant or there is possible null pointer dereference: irp.
[client/X11/xf_window.c:582] -> [client/X11/xf_window.c:580]: (warning) Either the condition '!xfc' is redundant or there is possible null pointer dereference: xfc.
[winpr/libwinpr/path/test/TestPathShell.c:40] -> [winpr/libwinpr/path/test/TestPathShell.c:43]: (warning) Either the condition '!path' is redundant or there is possible null pointer dereference: path.
[winpr/libwinpr/path/test/TestPathShell.c:49] -> [winpr/libwinpr/path/test/TestPathShell.c:52]: (warning) Either the condition '!path' is redundant or there is possible null pointer dereference: path.

Merge pull request #4510 from mfleisz/cssp_version_fix

cssp: Separate client/server version handling (#4502)

cssp: Separate client/server version handling (#4502)

Added debug message.

Removed reserved COM and LPT check.

Merge pull request #4508 from h3xx/fix-ccache-support

Fix ccache support

Fix ccache support

Setting the compiler launcher to "ccache" is the recommended way of
enabling ccache for the build.

If cmake is run with it defined, it causes an error when ccache tries to
run:

    ccache: error: Recursive invocation (the name of the ccache binary must be "ccache")

This was because the compiler was getting invoked as
"ccache ccache [COMPILER]"

Merge pull request #4469 from akallabeth/win_server_fix

Fixed windows shadow server update copy.

Merge pull request #4504 from mfleisz/cssp_warnings

cssp: Fix warnings (#4503)

cssp: Fix warnings (#4503)

Merge pull request #4505 from bmiklautz/nightlies_bionic

pkg freerdp-nightly: fix build on bionic

pkg freerdp-nightly: fix build on bionic

Merge pull request #4499 from mfleisz/cssp_v6

cssp: Add support for protocol version 6

cssp: Add support for protocol version 6

Merge pull request #4497 from akallabeth/kwin_multimon

Fix #2707: Set fullscreen first

Fix #2707: Set fullscreen first

Due to kwin bug https://bugs.kde.org/show_bug.cgi?id=391960
multimonitor fullscreen is not applied correctly with the
previous approach.

Merge pull request #4495 from akallabeth/header_fix

Fixed #4493: duplicate typedef.

Fixed #4493: duplicate typedef.

Merge pull request #4487 from bmiklautz/KB4088776

fix nla: don't use server version

fix nla: don't use server version

FreeRDP currently only supports CredSSP protocol version 3. However the
current implementation always sent back the version received by the
server indicating that this version was supported.
With recent windows updates applied the protocol changed and this approach
doesn't work anymore (see
https://msdn.microsoft.com/en-us/library/mt752485.aspx for protocol changes).

With this fix FreeRDP always sends version 3 as supported version.

Credit goes to @mfleisz.

Fixes #4449

Merge pull request #4473 from akallabeth/mac_timer

Added WaitableTimer implementation for mac OS.

Added WaitableTimer implementation for mac OS.

Detect command line flags in case RDP or MSINCIDENT files are used.

Added additional mouse mappings.

Do not clear invalid region if no client connected.

Fixed windows shadow server update copy.

Merge pull request #4471 from akallabeth/thread_function_fix

Thread and Threadpool work function fix

Fixed compiler warnings.

Fixed threadpool work function return and arguments.

Fixed thread function return and parameters.

Merge pull request #4470 from mfleisz/misc_fixes

Various fixes after running VS 2017 code analysis

Merge pull request #4464 from perkerk/misc-windows-client

Misc changes for Windows client

Remove NULL check before _aligned_free call

Clean up mf_uninit

Merge pull request #4454 from akallabeth/nightly

Use a custom command to generate helper tool

rdpei: Fix possible NULL ptr access

drive: Fix possible NULL ptr access

debug: Fix broken format specifiers

drdynvc: Fix possible NULL pointer accesses

codec: Another realloc fix

core: Fix possible out-of-bounds read

nla: Add NULL pointer check

codec: Prevent invalid mem access on realloc failure

codec: Fix broken reallocs

Merge pull request #4468 from akallabeth/transport_fix_handle

Fix invalid handle offset.

Fix invalid handle offset.

Fix memory leak of Decompress buffer in client wf_Bitmap_Free

Remove unnecessary WM_SETCURSOR handling from wf_event_proc