Merge pull request #5857 from akallabeth/scanbuild_fixes

Scanbuild fixes

Merge pull request #5877 from akallabeth/urbdrc_export_symbol_fix

Urbdrc export symbol fix

Exporting URBDRC symbols used by helper library

When building the usb channel with different shared libraries some
necessary symbols were not exported.

Merge pull request #5875 from akallabeth/smartcard_missing_null_check

Fix #5847: Missing length check to trace smartcard buffers.

core: server: fix a bad free in channel->queue cleanup

Fix #5847: Missing length check to trace smartcard buffers.

Merge pull request #5865 from akallabeth/write_cancelled_leak

Fix #5852: handle CHANNEL_EVENT_WRITE_CANCELLED

Merge pull request #5869 from llyzs/winmm_fixes

rdpsnd/winmm: fix some memory issues.

server: proxy: improve logs

Merge pull request #5830 from akallabeth/smartcard_updates

Smartcard updates

rdpsnd/winmm: fix some memory issues.

Fix #5852: handle CHANNEL_EVENT_WRITE_CANCELLED

When channels are cleaned up OpenHandle is most of the time
already 0. Ignore the argument checks for
CHANNEL_EVENT_WRITE_CANCELLED and CHANNEL_EVENT_WRITE_COMPLETE
only check where actually required (CHANNEL_EVENT_DATA_RECEIVED)

FreeRDP_WTSVirtualChannelOpen unify channel new/free

server: make sure all messages in channel->queue are freed when queue is freed

server: proxy: show indicative log on bind failure.

Added unified NDR pointer reading.

Fixed missing length checks for smartcard IRP read.

Merge pull request #5859 from mfleisz/multi_trans_fix

core: Always send CS_MULTITRANSPORT PDU to server

core: Always send CS_MULTITRANSPORT PDU to server

This PDU is required by Microsoft servers in order for bandwidth
management to work correctly. Even if we do not support multi-transport
for now we should just send a PDU with flags set to 0 to enable correct
handing of bandwidth measurement PDUs.

Fixed rdpei unused function warning.

Fixed smartcard type mismatches.

Fixed type of index variables.

Fixed type of index variable.

Fixed unused variable warning.

Fixed copy of opaque data

do not use strcncpy for opaque data.

Fixed unused argument warnings.

Added NULL checks to silence clang-scanbuild.

Fixed NULL dereference.

Fixed NULL dereference.

Fixed NULL dereference.

Unified smartcart multistring conversion.

Fixed type mismatches on allocation.

Fixed smartcard_SetAttrib_Decode call type

Merge pull request #5856 from akallabeth/neon_ycocg

Fixed #5855: YCoCg to RGB conversion color order

Fixed #5855: YCoCg to RGB conversion color order

core: Fix and extend populate rdp file from settings

This PR contains the following changes:
- Get rid of unused SettingsModified array (kept in the settings struct for ABI
compatibility)
- Fix and extend freerdp_client_populate_rdp_file_form_settings (wrote <null> strings to the rdp file, missed a lot of settings)
- Set KeyboardHook default value to 2 (hook in fullscreen) just as mstsc
does

server: proxy: specify which modules to load in config

cmdline: export CommandLineParseCommaSeparatedValues

Merge pull request #5851 from akallabeth/multi_disconnect_crash_fix

Fixed missing NULL set of pointer after free

Fixed missing NULL set of pointer after free

Due to this double free was possible if disconnect_and_clear was
called multiple times.

server: proxy: add an indicative log listener->Open fails. fixes #5850

server: proxy: pf_client.c: remove unused includes

server: proxy: disconnect all active sessions on shutdown

Revert "core: rdp: reset rdp->nla in rdp_reset"

This reverts commit 205b0fba7fb92b4db7bb0fbdac7550564e48cd0e.

server: proxy: modules_init: create modules dir if not exists

server: proxy: update config.ini

rdpsnd: Add synchronization to winmm backend

Merge pull request #5844 from akallabeth/gfx_fix

Gfx fix disconnect reorder

Fixed #5843 wrong header length for pdu

@boom1 reported encomsp_send_change_participant_control_level_pdu
fails due to wrong length.

Fixed accidental reorder of surface delete and disconnect

rdpsnd: Fix possible crash and deadlock in winmm backend

audin: Fix possible crash in winmm backend

Merge pull request #5841 from akallabeth/vaapi_fixes

Vaapi fixes

Merge pull request #5836 from kubistika/proxy/fix_rail_and_update_issues

fixes: rail and update issues

Merge pull request #5783 from akallabeth/gfx_mask_arg

Added /gfx and /gfx-h264 option mask=<value>

Fixed #5839: Use correct pointer to cast.

Add CMake config option for VAAPI

core: rdp: reset rdp->nla in rdp_reset

client/X11: Avoid crash in XGetDeviceButtonMapping()

XOpenDevice() may fail and return NULL, so try to find the first
pointer device that can be opened, and ensure that ptr_dev argument
is not NULL before passing it to XGetDeviceButtonMapping().

Merge pull request #5750 from akallabeth/encomsp_auto

Encomsp automatic input control && cleanups

rdpsnd: server: fix leak in rdpsnd_server_context_free

server: update: made functions static

update: update_send_new_or_existing_window: fix signed/unsigned according to spec

server: update: call update_check_flush whenever using the reused update buffer

Merge pull request #5831 from akallabeth/drdynvc_error_tolerance

Ignore dynamic virtual channel errors

Merge pull request #5835 from akallabeth/planar_chroma_subsample

Implemented chroma subsampling for planar non RLE

server: proxy: fix race condition when between client thread and server update thread

Fixed remaining issues with chroma subsampling.

Fixed inverted color formats in YCoCgRToRGB_8u_AC4R

Fixed planar size checks.

Enabled YCoCg for planar.

Added planar RLE chroma subsampling.

Implemented chroma subsampling for planar non RLE

Planar frames that are not RLE encoded can now be decoded if
chroma subsampling is enabled. Partly fixes #5822

server: proxy: fix pf_modules_init

Merge pull request #5796 from akallabeth/rdp_10_7_large_pointers

Added RDP 10.7 large pointer support

server: proxy: do not compile modules by default

server: proxy: shutdown gracefully

server: proxy: rewrite modules API

* Add support for C++ plugins.
* Seperate between a "module" and a "plugin": a module is a shared
library that exposes a function called "proxy_module_entry_point". This
function gets a parameter to `proxyPluginsManager*`, which can be used to
register plugins.
* Refine the API of the modules infrastructure:
 * use proxyData* instead of rdpContext* when firing a hook or calling a filter.
 * use clearer names.
* Instead of having to specify a path for each module to be loaded in
the configuration, the proxy now loads modules from specificed
directory, using the CMAKE variable PROXY_PLUGINDIR.
* Add an option to specify required plugins: plugins that the proxy
wouldn't start without having them fully loaded.

winpr: collections: add ArrayList_ForEach

rail/server: rename rdpContext to rdpcontext

Fixes a compilation error when mixing c++ and c code.

Added /gfx and /gfx-h264 option mask=<value>

With this new option the gfx capability set can be masked to only
announce older versions of the protocol if required.
(Fixes #5771)

Merge pull request #5832 from akallabeth/libusb_doc_misleading

Libusb doc misleading

Added fallback for libusb < 1.0.19

we use stream_id functions which were introduced with 1.0.19

Fixed LIBUSB_HOTPLUG_NO_FLAGS definition

libusb documentation is a bit misleading, this flag was
introduced with 1.0.20

rdpgfx: client: fix leak when using rdpgfx_client_context_new API

server: fix leak in WTSCloseServer

make sure every pending message that wouldn't be handled because the
server is closing, will be free()ed as expected.

fix memory leaks in disp server and gfx.c

server: proxy: fix leak in pf_context_copy_settings

winpr: don't close file descriptor if we set it again to the same file

Merge pull request #5752 from akallabeth/manpage_no_err

Allow to generate docbook with 0 length.

Ignore dynamic virtual channel errors

If a dynamic virtual channel can not process a message, ignore it.

Implement missing functions for new smartcard

Newer smartcard channel behaviour expects a lot more callbacks
being implemented. This change provides them.

Added command line option client-build-number

The smartcard channel behaviour is influenced by what build number
is announced, expose this setting to modify by interested users.

Fix smartcard structs, conform to [MS-RDPESC]

Merge pull request #5827 from akallabeth/nightly_fix

Extend LIBUSB_API_VERSION check

Extend LIBUSB_API_VERSION check

The define exists since 1.0.13, so to compensate for older also
check if the variable is defined.

Merge pull request #5824 from akallabeth/ntstatus_helper

Added NtStatus2Tag function to stringify NTSTATUS

Added NtStatus2Tag function to stringify NTSTATUS

For higher level applications it is easier to translate known
strings instead of error codes. this function allows
to retrieve known string identifiers that can be translated.

Signed-off-by: Armin Novak <armin.novak@thincast.com>

Merge pull request #5823 from akallabeth/set_transport_err

Set transport error on close

Setting optional last error on transport close

Signed-off-by: Armin Novak <armin.novak@thincast.com>