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

Merge pull request #5639 from akallabeth/error_category

Added function to get an error category for better error handling.

Removed check for wrong buffer size.

Merge pull request #5643 from akallabeth/openh264_fixes

fixed openh264 related issues, thx to mythos from IRC

Merge pull request #5641 from akallabeth/settings_use_after_free

Fixed use after free.

Cleaned up NSC API

fixed openh264 related issues, thx to mythos from IRC

Fixed use after free.

Added function to get an error category for better error handling.

Merge pull request #5634 from akallabeth/error_code_update

Added missing error codes.

Merge pull request #5636 from akallabeth/silence_cmake

Fix #5635: Silence double WITH_CUPS definition.

Merge pull request #5633 from akallabeth/event_fd_fix

GetEventFileDescriptor return handle internal event.

Fix #5635: Silence double WITH_CUPS definition.