Merge pull request #3996 from akallabeth/drive_size_fix

Fix #3995: File size information

Fix #3995: File size information

Merge pull request #3905 from ilammy/x11-cliprdr/file-clipping

Local-to-remote file clipping for xfreerdp

channels/cliprdr: avoid possible integer overflow

If the server sends us garbage (or the client provides it) then it is
possible for the multiplication to overflow (as it is performed on
unsigned 32-bit values) which will result in a false positive failure of
the sanity check. Avoid it by rearranging arithmetics a little.

Keep the multiplication in the error message because we are interested
in the number of bytes in the stream and how it compares to the number
we have expected based on the presumed file count.

Merge pull request #3987 from chipitsine/cppcheck

resolve an issue found by cppcheck

resolve an issue found by cppcheck:

[libfreerdp/crypto/certificate.c:315] -> [libfreerdp/crypto/certificate.c:316]: (warning) Either the condition 'if(fingerprint&&fprint)' is redundant or there is possible null pointer dereference: fingerprint.

Merge pull request #3969 from chipitsine/cppcheck

resolve issue found by cppcheck

Merge pull request #3986 from akallabeth/debug_fix

Fixed missing external declaration (#3982)

Fixed missing external declaration (#3982)

Merge pull request #3985 from ilammy/x11-cliprdr/fix-html

Do not fixup HTML data length

resolve an issue found by cppcheck:

[channels/tsmf/client/tsmf_main.c:89] -> [channels/tsmf/client/tsmf_main.c:95]: (warning) Either the condition '!callback' is redundant or there is possible null pointer dereference: callback.

client/X11: do not fixup HTML data length

The data provided by local applications can be actually encoded in
UTF-16 (e.g., Firefox does this to HTML). UTF-16 allows embedded null
bytes so we should not use strlen() to fix up the data. The HTML format
synthesizer can handle trailing null bytes just fine and can detect
whether it deals with UTF-8 or UTF-16.

Merge pull request #3980 from uplusplus/patch-5

fixed issue:#3974

fixed issue:#3974

ConvertToUnicode failed

winpr/file: add missing NULL check

ValidFileNameComponent() has been missing a NULL check for its argument.
It's pretty obvious that NULL is not a valid file name component.

wClipboard: drop WITH_DEBUG_WCLIPBOARD option

This preprocessor definition has been initially intended to disable some
computationally expensive logging, however it turned out that there is
not much computation involved in the resulting implementation of new
wClipboard subsystems. Therefore we do not actually need the compilation
option, the logs can be filtered by "com.winpr.wclipboard.*" tag at
runtime if necessary. So drop the WITH_DEBUG_WCLIPBOARD CMake option and
convert all detailed logs to use WLOG_TRACE level via WLog_VRB macro.

x11/cliprdr: make callbacks static

Most of the functions is this file are internal-use callbacks so they do
not need to be exported from the compilation unit. Mark functions static
as appropriate.

Merge pull request #3906 from akallabeth/addin_cast_fix

Addin cast fix

Merge pull request #3963 from akallabeth/qoe_ack

Added client RDPGFX_QOE_FRAME_ACKNOWLEDGE_PDU

Merge pull request #3967 from akallabeth/sec_encrypt_fix

SEC_ENCRYPT check in rdp_client_connect_auto_detect

Merge pull request #3965 from akallabeth/drive_fixes_unicode

Drive fixes unicode

Merge pull request #3966 from davewheel/ntlm_getHash

Add a callback to provide NTLM hashes on server-side

Add a callback to provide NTLM hashes on server-side

Adds a callback that allows servers to compute NTLM hashes by themselves. The typical
use of this callback is to provide a function that gives precomputed hash values.

Sponsored by: Wheel Systems (http://www.wheelsystems.com)

Added setting for QoeAck.

Added client RDPGFX_QOE_FRAME_ACKNOWLEDGE_PDU

Merge pull request #3964 from akallabeth/update_reset_state_fix

Update reset state fix

Fixed path length check.

Fix #3962: Remove trailing slashes from path.

Use unicode constants for WCHAR

Fixed missing state reset.

Fixed missing argument checks.

Merge pull request #3957 from akallabeth/gfx_use_logger_instance

Use a preallocated logger instance for GFX

Use a preallocated logger instance for GFX

The static logger functions of WLog have quite a heavy performance
penalty. Use a pointer to an allocated logger to speed things up.

Merge pull request #3942 from akallabeth/shift_warn_fix

Fixed right shift on 32bit platforms.

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

Added 64bit file support flag for linux.

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.

Fixed right shift on 32bit platforms.

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.

winpr: fix field names of FILEDESCRIPTOR struct

The file name field is actually called cFileName on Windows. Use this
name in WinPR's struct definition as well for compatibility.

winpr: include Windows headers in <winpr/shell.h>

This header file (currently) provides definitions of FILEDESCRIPTOR
structure and GetUserProfileDirectory() function. However, it does so
only when included on non-Windows platforms. The code which includes it
fails to build on Windows because the definitions are absent and it
causes weird compilation errors (like FILEDESCRIPTOR being treated as
the name of a function argument).

Inculde <shlobj.h> to get FILEDESCRIPTOR and <userenv.h> for the
GetUserProfileDirectory() function. (And hope that this will not
pull more Windows headers than we need in the files which include
<winpr/shell.h>.)

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.