Merge pull request #4032 from blino/clipboard-build

Fix clipboard POSIX build because of basename conflict

Fix clipboard POSIX build because of basename conflict

FreeRDP/winpr/libwinpr/clipboard/posix.c:397:20: error: conflicting types for ‘basename’
 static const char* basename(const char* name)
                    ^
In file included from FreeRDP/winpr/include/winpr/collections.h:25:0,
                 from FreeRDP/winpr/libwinpr/clipboard/posix.c:37:
/usr/include/string.h:599:14: note: previous declaration of ‘basename’ was here
 extern char *basename (const char *__filename) __THROW __nonnull ((1));

Merge pull request #4024 from bmiklautz/fix_4022

Fix a regression introduced with PR #4013

Fix a regression introduced with PR #4013

If numWindowRects/numVisibilityRects is zero a realloc might either
return NULL or a free able memory. In the first case the introduced
regression caused a double free.
As 0 is a possible value that can be received in both cases rail was
broken.

Fixes #4022

Merge pull request #4008 from hardening/h264_split

Split h264 subsystems in different files

Merge pull request #4013 from weizhenwei/master

fix FreeRDP some memory leak problem

code format adjustment

remove redundant NULL pointer check

remove duplicate code when freeing drive

remove useless NULL pointer check before free

add NULL pointer check and set freed pointers to NULL afterward

refactor to remove duplicate code and replace free+malloc with realloc

refactor on redundant code copy

fix compile error on win64.vs2010 platform

fix memroy leak of window_icon->iconInfo at update_read_window_icon_order()

fix memory leak of pKeys at xf_rail_invalidate_region()

fix memroy leak of fd at FindFirstFileW()

fix memory leak at update->window->window_state.windowRects/visibilityRects at update_read_window_state_order()

fix memory leak at update->window->window_state.titleInfo.string at update_read_window_state_order()

fix memory leak on rail_virtual_channel_event_data_received() -> StreamNew()

fix memory leak at drive_register_drive_path() and drive_free()

fix memory leak at rail_client_execute()

fix memroy leak on rail_recv_exec_result_order() execResult

code clean on free(update->window->monitored_desktop.windowIds);

fix memory leak on update->window->monitored_desktop.windowIds which is realloced at update_read_desktop_actively_monitored_order()

fix a memroy leak on rdpBitmap free

Fix openh264 backend compilation

Fix variable declaration and win32 compilation

Split h264 subsystems in different files

Merge pull request #4003 from akallabeth/macos_mouse_wheel_shadow

Fixed shadow server mouse wheel mapping on MacOS

Fixed shadow server mouse wheel mapping on MacOS

Merge pull request #3989 from akallabeth/avc444_alignment_fix

Align width and height for AVC444 decoding to 16.

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.

Align width and height for AVC444 decoding to 16.

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.