Merge pull request #4512 from chipitsine/master

resolve several possible null pointer dereference

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.

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.

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.