Merge pull request #4551 from oshogbo/nego_0

Fix setting of negotiated security protocol.

Fix setting of negotiated security protocol.

PROTOCOL_RDP is equals 0 so if we and it with anything is still 0.
Correct way of doing it is to compare two values.

Merge pull request #4549 from oshogbo/krb

Fix checking of krb in encrypt public key echo.

Fix checking of krb in encrypt public key echo.

In commit 0e1a073384ddd1382f6a0619177cf0b601ad973c there was a mistake -
originally code said different then kerberos. Because of that NLA authentication
of server side didn't work for me.

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

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.