bugfix: segfault when calling an event which is not implemented by filter

server/proxy: Use new parse_string_array_from_str implementation (using ArrayList)

server/proxy: Refactor pf_server.c

server/proxy: Refactor pf_server_parse_target_from_routing_token

server/proxy: Add external filters support

Merge pull request #4350 from akallabeth/format_code_patch

New code formatting rules

Added a CMake option for the clangformat target support.

rail: added verbose logs (#5402)

* rail: added verbose logs

* rail: fix buffer initialisation in debug message

Merge pull request #5403 from akallabeth/keyboard_event_flags

Fixed fastpath/slowpatth pause event.

Merge pull request #5404 from m4ntis/feat/proxy-errors

Proxy: Redirect errors back to client

Merge pull request #5413 from akallabeth/rail_wdir

Added rail workdir parameter

Added rail workdir parameter

server/proxy: Proxy errors back to client

Export Error Info PDU.

First version of an RDP proxy (#5372)

* server: Add proxy dir with barebones server

* sever/proxy: Remove licensing

* server/proxy: Add client files

* server/proxy: rm binary

* server/proxy: Formatting

* server/proxy: Fixed includes and added basic client creation functionality

* server/proxy: Remove licensing and fix ifndef

* proxy/server: Fix cmake indentation

* server/proxy: Fix licensing

* server/proxy: Forward connection on peer_post_connect

* server/proxy: Fix function signature

* server/proxy: Changed function signature of proxy_client_start

* server/proxy: Now peer_post_connect calls proxy_client_start in a new thread

* pfreerdp.c: Clean up useless comments and logs

* server/proxy: Fix license

* server/proxy: Remove all non-connection related data from proxy_context

* server/proxy: Move Log Tag definition to pf_log.h

* server/proxy: Move context definition to pf_context

* server/proxy: Delete pfreerdp.h

* pfreerdp.c: Move context callbacks to pf_context.c

* server/proxy: Update CMakeLists.txt

* pf_channels: Use new proxy context API

* pf_client: Move context to pf_context

* pf_client.c: Remove unnessecary event handling

* server/proxy: Formatting

* proxy/server: Move server logic to pf_server.c

* server/proxy: Handle client disconnection

* Merge stash

* pf_server.c: Open GFX Connection to client

* server: CMakeLists: build proxy along with other servers

* server: proxy: get target server from rdpNego->RoutingToken

Iv'e omitted a check from  which im not sure is right. Should check in docs

* server/proxy: Handle remote server -> client disconnection

* server/proxy: Move common function  to pf_common.c

* server/proxy: Move common function  to pf_common.c

* rdpgfx.h: Add reference to freerdp.h for rdpContext

* pf_channels: Pipe GFX on channel connection

* server/proxy: Add pf_rdpgfx for proxy gfx callbacks

* pf_client: Declare dynvc and gfx capabilities on connection

* server/proxy: Add graphics callbacks

* server/proxy: Add graphics callbacks

* pf_server: Listen to channel events

* Pass user settings to server

* pf_server: Proxy mouse events

* fixup! server/proxy: Add graphics callbacks

* pf_client: Fix setting initialization

* Merge feat/proxy-gfx to feat/proxy

* pf_server: Fix double freed credentials

* server/proxy: Remove unnecessary call to freerdp_client_settings_parse_command_line

* server/proxy: Refactor re-activation code

* server/proxy: Run format scripts

* server/proxy: Fix segfault when post_disconnect return FALSE

* server/proxy: Refactor proxy_settings_mirror

* server/proxy: Redirect credentials

* server/proxy: move proxy_settings_mirror to pf_common.c

* server/proxy: Redirect desktop_resize event

* pf_client: Remove interactive CLI auth methods

* fixup! server/proxy: Redirect credentials

* server/proxy: Rename proxy_mirror_settings to pf_common_copy_settings

* pf_server.c: Fixed non-freed context

When the disconnection is forced by the target server, the function
`pf_server_handle_client_disconnection` isn't called. Therefore, the
context of the connection between the proxy to tagrget isn't freed.

* fixup! pf_server.c: Fixed non-freed context

* pf_client: Prefix all client methods with pf_client

* pf_context: Add init client to proxy context method

* pf_server: Confirm all GFX caps regardless of settings

* pf_server: Prefix all methods with pf_server

* pf_server: Move variable decleration to start of method

* pf_server: Fix client setting

* pf_server: Fix GFX init method

* pf_server: Move variable decleration to start of methods

* server/proxy: Formatting

* Merge feat/proxy

* pf_server: Proxy synchronize event

* pf_server: Proxy refresh rect update events

* pf_server: Proxy suppress output messages

* server/proxy: Fix licensing

* server/proxy: Move client input callbacks to pf_input

* server/proxy: Move client update callbacks to pf_update

* server/proxy: Fix non-terminated target host string

* Feat/proxy config (#2)

* server/proxy: Add config loading support

* server/proxy: Add config file

* server/proxy: Format code

* server/proxy: Code refactor, rename update_register_callbacks and input_register_callbacks

* server/proxy: Update config file

* server/proxy: Remove config.ini from root directory

* Remove comment from config file

* server/proxy: Fix leak in pf_server_load_config

* server/proxy: Add rdpServerProxy struct and embed it in proxyContext

* server/proxy: Load configuration and pass it inside every proxyContext instance

* server/proxy: Move rdpProxyServer to proxy.h

* server/proxy: Use configuration while proxying input events

* server/proxy: Update CMakeLists

* server/proxy: Refactor pf_input.c

* server/proxy: Add AllowedChannels, DeniedChannels in configuration

* server/proxy: Remove unnecessary variable from parse_channels_from_str

* server/proxy: Update config file

* server/proxy: config: Rename  to

* server/proxy: config: Add mode - blacklist/whitelist

* server/proxy: Refactor, fix NULL deref

* server/proxy: Add license to proxy.h

* server/proxy: Fix newline in pf_config.c

* server/proxy: config: Rename Mode to WhitelistMode

* Add target in config. Add checks for configuration validity (#3)

* Add target in config. Add checks for configuration validity

* Update config file

* libfreerdp: nego: revert commented out check of routingToken length

* pf_server: Fix target host info from RoutingToken

* pf_server: Remove hardcoded lenght of routing token prefix

* Feat/refactor context (#8)

* Refactor main structs

* Update CMakeLists.txt

* pf_server.c: Free pdata at the end of the connection

* Run format scripts

* Rename tf to pc

* Fix licenses

* pf_server: Refactor names of structs and functions

* proxy: gfx: sync caps (#4)

* proxy: gfx: sync caps

* proxy: gfx: sync caps, hook gfx client's OnClose() call and close server resources

* fixup! Feat/refactor context (#8)

* fixup! fixup! Feat/refactor context (#8)

* rdpgfx/client: Fix rdpgfx_recv_caps_confirm_pdu caps set length parsing

* Run format scripts

* proxy config.ini: Change default port to 3389

* pf_rdpgfx: Limit caps version to freerdp's supported versions

* Gfx OnOpen() wait for dynvc ready (#10)

* proxy/gfx: Wait for dynvc ready state before open

* pf_channels: Initialize pc->gfx

* pf_rdpgfx: Add log and fix comments

* rdpgfx: Fix GFX v10.6 PDUs parsing and naming according to the spec

* pf_rdpgfx: Proxy rdpgfx v10.6 PDUs

* gfx client: Publish FrameAck sending and add auto ack flag

* proxy/gfx: Forward frame ack messages

* pf_context: Forward domain on connection

* pf_rdpgfx: Change max supported caps to 10.6

* proxy: Update config

* server/proxy: Use configuration in pf_server_handle_client

* rdpgfx/client: Fix size of surface_to_scaled_window, surface_to_window

* pf_rdpgfx: Fix formatting

* pf_server.c: Fix comments

* Move pf_server_rdpgfx_init to pf_rdpgfx

* server/proxy/CMakeLists.txt: Fix formatting

* pf_client.c: Add comment in proxy_server_reactivate

* Fixed const correctness of gfx function pointer

Signed-off-by: Mati Shabtay <matishabtay@gmail.com>
* server: proxy: update copyright

* server: proxy: wrap rdpNego and add a getter for routing token

* Refactor routing token getter (#14)

* Refactor routing token getter

* pf_server_parse_target_from_routing_token change routing_token_length to be DWORD

* libfreerdp/core/nego.c: Run format script

* pf_server: Run format script

* server/proxy: Fix os msbuild tests

* pf_channels.c: Remove unused channels

* pf_client: Remove unused callbacks

* proxy: Remove encomsp callbacks from proxy's client

* client/rdpgfx_main.c: Fix msbuild test

* pf_config.c: Use StrSep instead of strsep for Windows builds

* Removed nego struct from direct access.

Signed-off-by: Mati Shabtay <matishabtay@gmail.com>
* proxy: Rename binary to freerdp-proxy

* rdpgfx_main.c: Revert unwanted double change to send_supported_caps

* Cleaned up proxy server code.

* All internal functions static
* Added simple command line argument to supply a config file
* Silence compiler warnings

Signed-off-by: kubistika <kmizrachi18@gmail.com>

Fixed fastpath/slowpatth pause event.

Added clang format script.

new code formatting rules are now enforced by this script.

Merge pull request #5398 from akallabeth/rail_client_sysparam_fix

Fixed mapping of new client sysparam values.

Fixed mapping of new client sysparam values.

Merge pull request #5394 from akallabeth/settings_update

Updated settings getter/setter for RemoteApplicationSupportMask

Updated settings getter/setter for RemoteApplicationSupportMask

Merge pull request #5314 from akallabeth/setting_getter_setter_generator

Added python generator for freerdp_[g|s]et_param*

Fixed remaining getter/setter return checks.

Merge pull request #5393 from akallabeth/rails_memleak_fix

Fixed window order cleanup for NOTIFY_ICON_STATE_ORDER

Fixed window order cleanup for NOTIFY_ICON_STATE_ORDER

Merge pull request #5327 from akallabeth/sign_compare_fixes_v3

Leak and warning fixes for 2.0

Merge pull request #5392 from akallabeth/gfx_avc_flag_fixes

Fixed GFX flags and CapsConfirm return checks

Merge pull request #5386 from akallabeth/rails_spec_update

Rails spec update

Fixed GFX flags and CapsConfirm return checks

Fixed rails set_caretwidth

Merge pull request #5388 from akallabeth/nightly_cairo

Nightly: activate cairo software scaling

Enable cairo support for software scaling.

Updated RAILS implementation

* Implement new messages and callbacks
* Announce most recent channel features
* Added settings to configure flags to announce

Merge pull request #5387 from mfleisz/image_from_icon

core: Move image conversion of icon data into the library

core: Move image conversion for icon data into the library

Fixed warnings and cleaned up gfx related xfreerdp code.

For smartcard trace functions use const arguments

Fixed uninitialized value

Fixed memory leak

Fixed memory leak

Fixed memory leak

Fixed possible memory leak.

Fixed index data type

Merge pull request #5266 from akallabeth/file_list_synth

Added a proper synthesizer for FileGroupDescriptorW to text/uri-list

Fixed error checks and uninitialized values in convert_filedescriptors_to_uri_list

Refactored settings getter/setter to stay compatible.

Fixed use of settings getter/setter

Added python generator for freerdp_[g|s]et_param*

The settings getters and setters are now automatically generated.
Added a test case to check if all available options can be retrieved.

Merge pull request #5385 from akallabeth/some_unused_parameter_silenced

Some unused parameter silenced

Merge pull request #5383 from akallabeth/gfx_spec_update

Updated GFX to 10.6 spec

Fixed doulbe semicolon in C files.

Silenced unused parameter warnings in smartcard trace

Silenced some unused parameter warnings.

Fixed memory leak and return values in rdpgfx_recv_caps_advertise_pdu

Added gfx capability filter option.

With this new option it is possible to selectively disable certain
GFX capability versions. This way buggy behaviour can be addressed
at runtime.

Fixed alignment requirements for surface sizes.

Merge pull request #5384 from m4ntis/fix-readme

Add .md file extention to README

Merge pull request #5339 from akallabeth/update_paint_lock

Unified update->BeginPaint and update->EndPaint

Add .md file extention to README

Fixed missing argument checks.

The arguments for rdpgfx_send_frame_acknowledge_pdu must be checked
as in some call stacks they might be NULL.

Optimized image scaling, copy if factor is 1.0

Updated GFX to 10.6 spec

Merge pull request #5381 from akallabeth/swscale

Added image scaling api for software drawing.

Added image scaling api for software drawing.

For future GFX channel functions an image scaling function is required.
This moves the implementation from wayland client to core library
and adds support for the much faster SWScale library.

Merge pull request #5376 from kubistika/bugfix/rdpgfx-server-v10.6-PDUs

rdpgfx: Fix GFX v10.6 PDUs parsing and naming according to the spec

Merge pull request #5377 from m4ntis/gfx/fix-caps-confirm-recv

client/rdgpfx_main.c: Fix rdpgfx_recv_caps_confirm_pdu caps length read

Merge pull request #5378 from m4ntis/gfx/manual-frame-ack-caps-sync

rdpgfx/client: Add callbacks for manual caps sending and frame acking

client/rdpgfx_main.h: Formatting

Fixed const correctness of gfx function pointer

Signed-off-by: Mati Shabtay <matishabtay@gmail.com>

rdpgfx/client: Add callbacks for manual caps sending and frame acking

client/rdgpfx_main.c: Fix rdpgfx_recv_caps_confirm_pdu caps length read

rdpgfx: Fix GFX v10.6 PDUs parsing and naming according to the spec

Merge pull request #5308 from akallabeth/fix_win32_overlinking

Fixed windows overlinking.

Fix smartcard segfault when no devices are present (#5364)

* Don't assume smartcard reader list is not empty.

* Check status of retrieval of smartcard device list before trying to filter it

* style fixes

android_freerdp.c: fixed wrong registered signature (#5371)

Merge pull request #5260 from akallabeth/rail_spec_update

Rail spec update

Merge pull request #5306 from akallabeth/gfx_server_caps_input_checks

Fixed input stream length checks in rdpgfx_recv_caps_advertise_pdu

Fixed typo in RAIL_ZORDER_SYNC

Fixed input stream length checks in rdpgfx_recv_caps_advertise_pdu

5360 segfault using smartcard without any readers (#5361)

* Fix: corrected test for astyle.

* Fix: added test for NULL readers returned by ConvertFromUnicode.

rail_orders.c: Fix wrong callback call on server handshake ex recv (#5345)

xf_cliprdr: detect null terminators more reliably. (#5353)

Clipboard formats containing plain text are specified to be terminated
by a \0 character in MS's documentation on standard clipboard formats:
https://docs.microsoft.com/en-us/windows/desktop/dataxchg/standard-clipboard-formats

xf_cliprdr_server_format_data_response receives pasted data from the
server to transfer to the client, in a sufficiently raw form that the
\0 terminator is still present, so it has to remove it. It does so by
checking only at the very end of the data. But I've observed that when
pasting out of at least one Windows program (namely Outlook 1903 on
Windows 10), the intended paste data arrives in this function followed
by \0 and then a spurious \n. In that situation the null-terminator
removal will fail to notice the \0, and will leave both bogus
characters on the end of the paste.

Fixed by using memchr to find the _first_ \0 in the paste data, which
should not lose any actually intentional data because it's in
accordance with the spec above.

Merge pull request #5354 from ffontaine/master

xf_floatbar.c: fix build without Xfixes

xf_floatbar.c: fix build without Xfixes

Remove unneeded include on Xfixes.h as it is not always available and
not used in xf_floatbar.c

Fixes:
 - http://autobuild.buildroot.org/results/69245e574787bada718c52c805ec137041dc233d

Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>

Unified update->BeginPaint and update->EndPaint

Since these functions were called from 2 different threads
(main thread or dynamic channel) depending on fastpath or
gfx channel use lock between these.
If not locked the invalid region may be accessed from both
threads and lead to crashes as experienced with nightly df280a7ff

FreeRDP#5329 if using OldLicenseBehaviour, don't try to save the Cal since we're not going to try to load it (#5330)

Fix segfault when IniFile_Load is called with invalid input (#5331)

* Fix segfault when IniFile_Load is called with invalid input

* Fix leak in TestIni.c third testcase

* TestIni.c: Refactor in order to avoid some old compilers errors

Merge pull request #5240 from akallabeth/sign_compare_fixes_v2

Sign compare fixes v2

Added optional PreFrameAck callback to RdpgfxClientContext  (#5303)

* RdpgfxClientContext: Add PreFrameAck callback

* client/rdpgfx_main.c: Call PreFrameAck callback before acking frame

* client/rdpgfx_main.c: Fix windows_msbuild_vs2010 CI build

* client/rdpgfx_main.c: Fix error logs log level

Remove unnecessary clipboard data copy.

clipboard: correctly handle flags during renegociatiion (#5323)

Fixed ntlm_av_pair_get_len return, no signed value required

Fixed argument pointer type cast

Fixed sign issues

Fixed unnecessary casts in freerdp_image_copy.

Disabled -Wimplicit-fallthrough for instances where its use is intended.

Fixed broken implicit fallthrough

Removed duplicate argument checks.

Fixed GetModuleFileNameW.

Fixed review remarks.