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.

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.

Fixed #4436: double free of http response.
Fixed #4436: reset of token split.

Merge pull request #4435 from akallabeth/refguid_static

fix windows nightly

Fixed missing error return.

Do not export internal variables.

Merge pull request #4432 from akallabeth/pubsub_fix

Fixed PubSub function pointer casts.

Fixed winpr digest type.

Fixed broken extern C

Fixed PubSub function pointer casts.

Merge pull request #4428 from hardening/video_fixes

fixes for video and other minor fixes

http: fix compilation with valgrind memcheck helpers

video, geometry: fix prototypes for timer callbacks

video: fix invalid geometry and invalid argument passing to time pubsub

Merge pull request #4430 from bmiklautz/rpmversion

pkg rpm: add support for .source_version in freerdp-nightly

pkg rpm: fix perapre script for freerdp-nightly

pkg rpm: add version information to freerdp-nightly

pkg rpm: add prepare script for freerdp-nightly

Merge pull request #4425 from akallabeth/windows_nego_fix

Fixed windows unicode authentication.

Merge pull request #4426 from akallabeth/pointer_null_fix

Check if xfreerdp window != NULL

Merge pull request #4364 from akallabeth/gateway_refactor

Gateway refactor

Prefer dynamic logger in dynamic channel.

Merge pull request #4376 from hardening/ms-rdpevor

Implement MS-RDPEVOR

Check if xfreerdp window != NULL

When xfreerdp aborts a connection it may be before xfc->window was allocated.
Don't access it in that case.

Fixed windows unicode authentication.

Added missing FREERDP_API function export.

Fixed gemoetry mapping.

Fixed copy offset issue.

Fixed code duplication.

Added software decoding fallback for RDPVOR

Fixed scaled drawing of video content.

video, geometry: fixed geometry handling

It was not working when moving the video window.

video: mutualize things in the common channel code

video: do cleanups in post_connect

drdynvc: fixed stream leak on error

geometry: drop server-side option for geometry

video: fixed uninit of channel

video+geometry: fixed leaks, a segfault and concurrent accesses

video: fix prototype and cleanup at exit

video: an implementation of MS-RDPEVOR for X11

Implements the decoding of video streams using common H264 decoders. We also implement
a trivial feedback algorithm.

Sponsored by: Rangee GmbH (http://www.rangee.de)

video: a skeleton for MS-RDPEVOR client

Basics for implementing the client side of the MS-RDPEVOR channel.

Sponsored by: Rangee GmbH (http://www.rangee.de)

geometry: change the channel API to mutualize things

Try to mutualize things that are platform independant in the geometry client
channel.

Sponsored by: Rangee GmbH (http://www.rangee.de)

Merge pull request #4423 from akallabeth/double_free

bitmap free now always deallocating all resources.

bitmap free now always deallocating all resources.

Merge pull request #4421 from akallabeth/x11_asan_fix_v2

Fixed accidental free of XImage data.

Fixed accidental free of XImage data.

Exit main thread with winpr_exit to trigger resource cleanup.

When using pthread_once with destructors they are only called,
if each thread (including the main thread) is exited with pthread_exit.
Introducing winpr_exit as a wrapper for that purpose.