rdpBrushCache now opaque

Merge pull request #5722 from akallabeth/primitives_const_source

Modified primitives function pointer to take const pointer to const data

Modified primitives function pointer to take const pointer to const data

fix [build]: don't export rdp2tcp as cmake target

rdp2tcp was wrongly exposed as cmake target. This could lead to errors
like the following when the FreeRDP2 cmake file was required:

CMake Error at ../lib/cmake/FreeRDP2/FreeRDPTargets.cmake:73 (message):

  The imported target "rdp2tcp-client" references the file
     ".../lib/freerdp2/librdp2tcp-client.a"
  but this file does not exist.  Possible reasons include:
* The file was deleted, renamed, or moved to another location.
* An install or uninstall procedure did not complete successfully.
* The installation package was faulty and contained
     ".../lib/cmake/FreeRDP2/FreeRDPTargets.cmake"
  but not all the files it references.

Merge pull request #5718 from akallabeth/remdesk_fix

Fixed cleanup of remdesk in CHANNEL_EVENT_WRITE_COMPLETE

Fixed cleanup of remdesk in CHANNEL_EVENT_WRITE_COMPLETE

cliprdr: common: fix validation of file contents request pdu

server: proxy: fix sync issue in gfx opening

Merge pull request #5711 from akallabeth/diff

Fixed broken strnlen checks

Fixed broken strnlen checks

Merge pull request #5709 from akallabeth/diff

Fixed broken length calculation in rdp_write_extended_info_packet

Fixed broken length calculation in rdp_write_extended_info_packet

Merge pull request #5703 from akallabeth/reformat

Reformatted to new style

Added cast and checks for info packet.

Fixed NULL checks for strnlen calls, limit static channel name length.

As reported by @metametaclass in #5687 check the strings for NULL before
using strnlen. Also extend the same checks to static channel loading.

Reformatted to new style

Updated format file.

Merge pull request #5704 from XiaopengZHOU/cursor-xor-padding

Fix padding of encoded scan-line for XOR mask of pointer.

Fix padding of encoded scan-line for XOR mask of pointer.

Merge pull request #5687 from akallabeth/channel_write_leaks

Fixed VirtualChannelWriteEx error case leaks.

Added CHANNEL_EVENT_WRITE_CANCELLED for remdesk channel

Fixed userdata pointer and CHANNEL_EVENT_WRITE_CANCELLED message.

Listen to CHANNEL_EVENT_WRITE_CANCELLED and emit if queue is cleared.

Fixed #4809: Properly process CHANNEL_EVENT_WRITE_COMPLETE

Fixed VirtualChannelWriteEx error case leaks.

Merge pull request #5685 from akallabeth/multi_instance_arg_parse

COMMAND_LINE_ARGUMENT structs contain parser results, use one per instance

Merge pull request #5689 from akallabeth/cleanups

Cleanups

Fixed strnlen issues.

Merge pull request #5690 from akallabeth/winpr_ini_cleanup

Cleaned up ini.h and added error checks.

Removed invalid use of strnlen

Use BOOL return for IniFile_Open_File and IniFile_Load_File

Merge pull request #5553 from akallabeth/win_console_fix

Reattach to cmd if wfreerdp was started from one.

Fixed review remarks.

Merge pull request #5699 from akallabeth/proxy_link_fix

Fix #5693: Add missing link to gfx client channel library

rdp: fix byte and packet counters
Best to count bytes directly at the transport level.

fix typo

Fix #5693: Add missing link to gfx client channel library

Merge pull request #5692 from kubistika/proxy/when_possible_do_nla_fallback

server: proxy: client: always do NLA fallback

server: proxy: client: always do NLA fallback

Fixed warnings from TestEnvironmentGetEnvironmentStrings

Cleaned up TestCryptoHash

Removed strlen from GetComputerNameExA

Fixed length checks for GetComputerNameA

Removed strlen use in wlog (and speed up check)

Fixed int/size_t use for channel add functions.

Fixed tmpnam warning.

Removed strcpy use.

Code cleanups (strlen, casts, size_t, ...)

Cleaned up ini.h and added error checks.

Merge pull request #5673 from akallabeth/audin_format_leak_fix

Fixed memory leak when freeing up audin formats.

Merge pull request #5670 from akallabeth/cairo_detection_fix

Fix #5669: If cairo is detected as part of other dependencies ignore it

Merge pull request #5668 from akallabeth/xfreerdp_mon_check

Added missing check for monitor indices

server: proxy: export config utils functions to be used by modules

server: proxy: stop loading session captures config if it is disabled

Use COMMAND_LINE_ARGUMENT_A copy also in shadow server code.

COMMAND_LINE_ARGUMENT structs contain parser results, use one per instance

Remove the old global structs as the parser modifies them. When using
multiple instances in the same process space this could break parsing.

Merge pull request #5676 from akallabeth/hotplug_leak_fix

Hotplug leak fix

Merge pull request #5680 from akallabeth/fixes2

Some smaller Fixes

Merge pull request #5684 from akallabeth/disp_static_fkt

Made disp channel internal functions static.

Made disp channel internal functions static.

Merge pull request #5679 from akallabeth/gdi_clamp

Fix gdi coordinate clamping.

Fix #5687: check for file extensions case insensitive.

wts_read_drdynvc_close_response quit message queue.

Added MessageQueue_Dispatch argument checks

Fix gdi coordinate clamping.

Removed experimental warning from gfx

devman_load_device_service device argument can be const.

server: proxy: refactor config loader

server: proxy: capture: refactor path creation

server: proxy: add python script to create a video from session frames

server: proxy: implement session capture feature

server: proxy: client use PIXEL_FORMAT_BGRA32 (easier to write frames to bitmap)

channels: rdpgfx: export API to create rdpgfx client

winpr: utils: image: fix const correctness

Fixed memory leak when freeing up audin formats.

Fix #5669: If cairo is detected as part of other dependencies ignore it

If WITH_CAIRO=OFF it was still used when the library was detected as
part of some other dependency. With this commit the dependency is
only added if WITH_CAIRO=ON

Added missing check for monitor indices

Fixes hub was not found! error

Again as with auto usb detection strtol needs to use base 10 and force to use base 10 else the conversion does not work correctly.

Fixes Auto USB Redirection

Base 0 does not work when converting the string to long int. This ends up not showing the correct device id and sometimes the wrong bus id. By changing it to base 10 conversion instead of base 0 this fixes the issue of auto redirect usb devices.

rdp: count in/out bytes and packets

Fix some static analizer warnings

server: proxy: client: terminate the connection if pre connect hook fails

channels: cliprdr: client: fix leak in cliprdr_packet_send

channels: cliprdr: improved common cliprdr api

channels: cliprdr: reformat cliprdr_common.c

channels: cliprdr: get rid of duplicated server and client code

channels: cliprdr: remove formatDataResp.dwFlags

According to the channel docs, this field is only used in format data
request. Therefore, there's no need to hold it in the response. cliprdr
server code was copy-pasted from client code, therefore this must be
some leftover.

Merge pull request #5657 from akallabeth/warning_fixes2

Warning fixes2

Merge pull request #5648 from akallabeth/h264_dyn

H264 runtime detection

Fixed compilation warnings.

Fixed missing initializer

Fixed floating point calculations.

Merge pull request #5646 from akallabeth/realloc_fixes

Fixed #5645: realloc return handling

Let shadow server detect H264 support at runtime

Allow freerdp clients to function with WITH_OPENH264_LOADING

If no library can be loaded at runtime ignore H264 capabilities.
This allows a build with only OpenH264 support and dynamic loading
enabled to function when no openh264 library was detected.

Merge pull request #5644 from akallabeth/nsc_cleanup

Cleaned up NSC API

Fixed windows shadow server startup issue.

Fixed leak in verify_cb.

Added /buildconfig command line for shadow.

Fixed #5645: realloc return handling