SEC_ENCRYPT check in rdp_client_connect_auto_detect

Fix by @wizwizaco for #3951

Merge pull request #3953 from hardening/kill_warnings

Drop some annoying warning

Drop some annoying warning

Probably that some unused functions should be removed, but at least it makes
it compile with no warnings.

Merge pull request #3952 from akallabeth/set_keyboard_ime

Added support for Set Keyboard IME Status

Merge pull request #3950 from hypherion2/cp_fixes

Fix handle leaks in encomsp and remdesk server

Added support for Set Keyboard IME Status

Fix handle leaks in encomsp and remdesk server

Ensure proper thread cleanup when these channel servers are stopped.

Merge pull request #3949 from akallabeth/gdi_surface_crash_fix

Check arguments for Bitmap_SetSurface

Merge pull request #3948 from hypherion2/master

Remove library prefix override for rdpei-server

Check arguments for Bitmap_SetSurface

The function may be called during connection phase
where there is no valid GDI available.

Merge pull request #3947 from akallabeth/openh264_fixes

Openh264 fixes

Remove library prefix override for rdpei-server

This library was missed in commit
059374457d801351d02ac81badb4aabbfa7689e7, so it was the only one that
stilll had a lib prefix

Initialize H264 context size for GFX surfaces.

Added pkg-config to cmake OpenH264 detection.

Merge pull request #3937 from hardening/h264_multimon

Fix h264 in multimon scenario

Merge pull request #3931 from hypherion2/master

Remove unused buffer in Mac shadow subsystem

Merge pull request #3943 from chipitsine/cppcheck

resolve an issue identified by cppcheck:

Merge pull request #3941 from mfleisz/fix_openh264_mt

codec: Fix multi-threading support for OpenH264 1.6 or later

resolve an issue identified by cppcheck:

[channels/urbdrc/client/libusb/libusb_udevice.c:1666] -> [channels/urbdrc/client/libusb/libusb_udevice.c:1661]: (warning) Either the condition 'request' is redundant or there is possible null pointer dereference: request.

codec: Fix multi-threading support for OpenH264 1.6 or later

Merge pull request #3924 from akallabeth/gfx_surface_area_update_callback

Added a GFX area update callback.

Merge pull request #3930 from realjiangms/fix_shadow_thread_leak

server/shadow: close the thread handle so that it is detached because…

Added missing callbacks.

Merge pull request #3938 from chipitsine/cppcheck

resolve an issue identified by cppcheck:

Fixed use after free of region.

Added a GFX area update callback.

resolve an issue identified by cppcheck:

[server/shadow/X11/x11_shadow.c:386] -> [server/shadow/X11/x11_shadow.c:390]: (warning) Either the condition '!subsystem' is redundant or there is possible null pointer dereference: subsystem.
[server/shadow/X11/x11_shadow.c:387] -> [server/shadow/X11/x11_shadow.c:390]: (warning) Either the condition '!subsystem' is redundant or there is possible null pointer dereference: subsystem.

Fixed progressive code style and added error handling

Fix H264 in multi monitor case

The H264 context is surface specific, so in multi-monitor (with multiple surfaces)
the decoding was failing. This patch fixes that by introducing a surface specific
h264 context.

Merge pull request #3932 from chipitsine/cppcheck

resolve issues identified by cppcheck

Merge pull request #3911 from akallabeth/avc444_v2

Added AVC444v2 client support.

Merge pull request #3934 from ilammy/x11-cliprdr/rail-support

Teach RAIL to use clipboard redirection

resolve issues identified by cppcheck

[channels/drive/client/drive_file.c:125]: (error) Memory leak: path_slash
[server/Windows/wf_dxgi.c:195]: (error) Invalid number of character '(' when these macros are defined: 'WITH_DXGI_1_2'.

client/X11: teach RAIL to use clipboard redirection

XSelection protocol does not define any global clipboard as there is on
Windows. Instead each window has its own property for clipboard content
(like CLIPBOARD or PRIMARY) and there is a global notion of clipboard
ownership.  Only one window can claim ownership of some clipboard type
at the moment.

FreeRDP uses CLIPBOARD for clipboard transfers (it's the one used by
applications when Ctrl+V is pressed). For regular desktop sessions the
session window itself is used for clipboard interactions via
xfc->drawable field. However, for remote app session there is no session
window. We cannot use the current remote app window as it may change or
be destroyed without closing the session. We also cannot use the root
window as it is already used for CF_RAW transfer protocol.

Therefore we create a simple dummy window to put into xfc->drawable for
this exact job: to act as a clipboard vessel on behalf of the entire
remote app session.

xf_create_window() usually creates the window as we immediately start in
RAIL mode when possible. xf_rail_enable_remoteapp_mode() is invoked only
when autologin failed or remote desktop had to show the session window
to the user for some reason.

Remove unused buffer which leaks

The buffer is not used in the code.

Merge pull request #3925 from akallabeth/avc_mf_decoder_fix

Fixed MF H264 buffer allocation.

Merge pull request #3919 from akallabeth/rdpdr_hotplug_fix

Fixed hotplug mount locations.

Fixed FindFirstFileA, do not strip trailing slash

Merge pull request #3927 from DavBfr/fix-win-printer

Bugfix: wrong test regarding the first GetPrinter

Fixed memory corruption in Find*FileW

Bugfix: wrong test regarding the first GetPrinter. This first call will always return FALSE because the goal of this call is to know the size needed.

Merge pull request #3926 from ShaunMaher/master

WS_EX_NOACTIVATE treated as unmanaged

WS_EX_NOACTIVATE treated as unmanaged\nWindows that carry the WS_EX_NOACTIVATE are treated the same as WS_EX_TOOLWINDOW in that they should not be handled as normal windows by the X window manager.  The prevents an issue with MS Outlook 2013 (and probably other applications) where attempting to resize the vertical columns causes some sort of WS_EX_NOACTIVATE window to be created which steals focus and aborts the resizing action.

Fixed MF H264 buffer allocation.

Merge pull request #3923 from rjcorrig/kb4015193

winpr: Updates time zone data to April 2017

Fix #3922: Use stat insead of lstat

Cleaned up includes.

Update timezone data to Apr 2017

Fixed AVC444v2 NEON chroma decoder.

Fixed SSE AVC444v2 chroma decoder.

Added surface width and heigth for YUV combine.

Add legacy mount locations, limited max depth.

Fixed WCHAR/CHAR conversion for drive path.

Fixed hotplug mount locations.

Added SSE AVC444v2

Fixed AVC444 NEON, added AVC444v2.

server/shadow: close the thread handle so that it is detached because no one is going to join it. Therefore the thread would release its resources automatically when exit

Merge pull request #3901 from akallabeth/openssl_1_1_no_legacy

Fixed OpenSSL 1.1 no legacy compile issues.

Merge pull request #3912 from nfedera/fix-gfx-gdihw-multimon

client/x11: fix gfx drawing bug with /multimon

Merge pull request #3902 from akallabeth/file_leak_fix

Fixed a memory leak and bad free.

client/x11: fix gfx drawing bug with /multimon

The surface's damage region is not relative to the output but
lives in its own universe starting at origin 0,0.

Also fixed the drawing coordinates used in the XRENDER code path
which is used with /smart-sizing

Upaded YUV tests.

Added error return in GetFileAttributesExA.

Added AVC444v2 client support.

Fixed broken scoping.

Fixed a memory leak and bad free.

Merge pull request #3864 from DavBfr/rewrite-disk-redirection

Rewrite disk redirection using WinPR

Fixed OpenSSL 1.1 no legacy compile issues.

Merge pull request #3897 from akallabeth/mac_os_fix

OSX: Updated RPATH settings for each lib.

Fixed RPATH settings for OS X

Fix indentation

Merge pull request #3899 from akallabeth/neon_fix

Fixed NEON YUV to RGB conversion issues.

Fixed NEON YUV to RGB conversion issues.

Merge pull request #3896 from nfedera/fix_gfx_progressive_decoding

rdpegfx: fix RemoteFX Progressive Codec decoding

rdpegfx: fix RemoteFX Progressive Codec decoding

Since this comes via a Wire-To-Surface-2 PDU we don't have
any left/top/right/bottom destination values.
The current code has always dealt with zeros when updating the
invalid region which resulted in black rectangles.
The correct update region is determined during decompression.

Fix FindFirstFile return values

Merge pull request #3892 from akallabeth/x11_gfx_surface_update

Update changed rectangles instead of extent.

Merge pull request #3894 from akallabeth/cache_to_surface_fix

Use exclusive coordinates in CacheToSurface.

Use exclusive coordinates in CacheToSurface.

Update changed rectangles instead of extent.

Merge pull request #3891 from nfedera/fix-rdpegfx-gdisw-drawing

improve rdpegfx drawing for software gdi

improve rdpegfx drawing for software gdi

process the region's rectangles individually instead of
just copying the extents

Merge pull request #3865 from DavBfr/unicode-printer-name

Bugfix: Wrong encoding method used for printer name

Merge pull request #3886 from akallabeth/warning_fixes_and_tests

Fixes for AVC420, Warnings and tests

Fixed memory leak and return value check issue.

Fixed AVC420 SSE implementation and test.

Fixed tests and dead store warnings.

Merge pull request #3884 from akallabeth/memleak_fix

Fixed leak of client random.

Fixed leak of client random.

Merge pull request #3883 from akallabeth/gfx_x11_speedup

Gfx x11 speedup

Let bitmaps keep XBitmap during lifetime.

 X11 GFX: Restrict to region, ignore alpha

* Restrict the screen update to the changed region
* Ignore differences in color format alpha

Fixed invalid return from xf_rail_window_icon

Merge pull request #3882 from nfedera/fix-x11-client-perf

some performance fixes

Fix memory leaks, Mixed declarations

some performance fixes

- draw only the updated region in the gdi and x11 surface bits implementation
- don't repeatedly call IsProcessorFeaturePresentEx in rfx rlgr decoder
- fix ugly and unaligned profiler print layout and remove an unnecessary value

Remove unused variable and check return codes

Merge pull request #3875 from hardening/client_random

Client random

Store client_random in server mode

We need the client_random in server mode when the client does RDP security and
tries to reconnect using the cookie.

Fixed typo