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)

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

Misc fixes for Windows clients

Support EmbeddedWindow in wf_resize_window

Fix H.264 codec init access violation on Windows N/KN systems

Merge pull request #4459 from akallabeth/surface_cmd_fix

Fixed read/write of surface bits command.

Fixed read/write of surface bits command.

The optional field exBitmapDataHeader of TS_ BITMAP_DATA_EX was ignored.
Read and expose the data (currently unused)

Merge pull request #4450 from akallabeth/assist_fix

Assistance and RDP file parser fix

Use a custom command to generate helper tool

Removing elements from CMAKE_C_FLAGS is not reliable,
to use a custom command to compile the helper tool
with default CFLAGS for the system.

Merge pull request #4451 from akallabeth/gsm_leak_fix

Clean up GSM context on channel close.

Merge pull request #4441 from akallabeth/paa

[cleanup] Support for gatewayaccesstoken / PAA

Free gsmBuffer on channel close.

Clean up GSM context on channel close.

Fixed .rdp and .msrcIncident checks.

The command line detection fails, if only one of the aforementioned
files is used as an argument. Detect those first and ignore command
line detection if found.

Added assistance file name NULL check.

Merge pull request #4448 from akallabeth/nightly_manpage_fix

Nightly manpage fix

Reenable -DWITH_SANITIZE_ADDRESS=ON for i386 builds.

Now that the manpage helper tool is always build without
address sanitizer there is no longer a reason to deactivate.

manpage helper tool generation fixes

The generator tool is called by CMake and is incompatible
with certain options. Therefore this commit
* removes WinPR dependency
* removes -fsanitize=* from CFLAGS

Merge pull request #4447 from akallabeth/apple_rpath_namespace

Apple rpath namespace fixes

Fixed apple rpath settings.

Removed apple flat-namespace hack.

Merge pull request #4446 from akallabeth/sam_crash_fix

Fix #3335: Do not call fclose on NULL file.

Fix #4445: Do not call fclose on NULL file.

Merge pull request #4440 from akallabeth/silence_unused_profiler

Disable unused variable warning from profiler.

Fix PROFILER_* macros

Add the semicolon to the macro to allow clean
undefinition if the profiler is not used.
This used to break VisualC compilers.

Merge pull request #4414 from akallabeth/pthread_cleanup_fix

Pthread cleanup fix

Modified RDG connection setup using PAA Cookie.
Skip NTLM auth on the IN/OUT channels. Add PAACookie to TUNNEL_CREATE.

Added to rdp file parsing: gatewayaccesstoken:s:<token>
Added to commandline parsing: /gat:<token>
Added GatewayAccessToken to sings

Merge pull request #4325 from akallabeth/fixes_com

Fixes serial redirection name check and SAM file parser

Merge pull request #4415 from akallabeth/drdynvc_dyn_log

Prefer dynamic logger in dynamic channel.

Merge pull request #4434 from akallabeth/winpr_digest_param

Fixed winpr digest type.

Merge pull request #4438 from akallabeth/gw_doublefree_fix

Fixed #4436: double free of http response.