Fix rdp file writer

The RDP file writer was based on previously parsed lines from an
existing rdp file. If you created a new rdpFile, populated it from
settings and tried to write it to a file you just got an error. This PR
fixes this issue by creating the data from the rdpFile properties rather
than the parsed lines.

Reformatting

Merge pull request #5484 from akallabeth/alternate_full_address

Added alternate full address to rdp file parser.

Added alternate full address to rdp file parser.

libfreerdp: core: fix const correctness of freerdp_settings_clone

Merge pull request #5481 from akallabeth/pem_intermediate_fix

Extract whole certificate chain to PEM format.

Extract whole certificate chain to PEM format.

server: proxy: use freerdp_settings_copy

libfreerdp: settings: Fix formatting in freerdp_settings_copy

libfreerdp: settings: call key_clone in freerdp_settings_copy

libfreerdp: certificate: Add key_clone

Added freerdp_settings_copy

Merge pull request #5475 from akallabeth/tls_fail_status

Pass on cert validation failure, set freerdp error in all use cases.

Pass on cert validation failure, set freerdp error in all use cases.

Merge pull request #5472 from akallabeth/rdp_parser_fixes

Rdp parser fixes

Fixed freerdp_client_rdp_file_set_integer standard flag set.

Fixed rdp file parser tests.

Implemented rdp file parser functions with a callback for custom settings.

Added PromptForCredentials setting.

server: proxy: implement NLA fallback

winpr: ObjectPool_Return: fix error handling

winpr: ObjectPool_Return: fix wrong type for new_cap

Merge pull request #5469 from akallabeth/access_denied

Don't set ACCESS_DENIED on first HTTP gateway response.

Don't set ACCESS_DENIED on first HTTP gateway response.

The first response is alwasy ACCESS_DENIED to initiate authentication.
Only the second response must not have that status.

Merge pull request #5468 from akallabeth/gw_status_denied

Set freerdp last error on gateway access denied.

Set freerdp last error on gateway access denied.

Merge pull request #5396 from widgetii/master

Artur Zaprzala's rdp2tcp channel driver

Merge pull request #5458 from kubistika/bugfix/proxy-multimon

Declare support for multimon in proxy server.

client: populate settings didn't update NetworkAutoDetect

rail: Fixes incorrect setting of TS_RAIL_EXEC_FLAG_FILE

The TS_RAIL_EXEC_FLAG should only be set if ExeOrFile refers to a file
path. If we don't supply a published app name (using ||) but the name of
an application (i.e. notepad.exe) this flag was incorrectly set causing
an remote app execution error (error 5 - file not found).

Declare support for multimon in proxy server.
It was broken in the DISP PR.

Merge pull request #5453 from akallabeth/load_balance_info_checks

Only try to set nego token if load balance info length is >0

Only try to set nego token if load balance info length is >0

Merge pull request #5447 from akallabeth/mac_bundle

Fix plugin paths when building for a mac bundle.

server/proxy: Add check after calling p_client_context_create

server/proxy: Logging and comments improvements

server/proxy: pf_update: Update ps->settings in DesktopResize

server/proxy: set ps->thread to NULL on disconnect

freerdp_abort_connect calls the proxy's client stop function,
that closes the thread handle. if it already closed by the server,
a segfault will occur because CloseHandle will try to access free()'ed
memory.

server/proxy: Refactor pdata_set_connection_info

Connection info must be set after the proxy's client connection
established, because if the client didn't provide credentials in .rdp
file, we don't know its username yet.

server/proxy: pf_filters: Update filter_free to accept NULL

server/proxy: Fix race-condition

The client's `ClientStop` callback wasn't set, therefore calling
freerdp_client_stop wasn't really blocking until it has stopped, and a
race condition occured. Fixed by registring to `ClientStop` callback and
waiting for the client thread to finish.

server/proxy: Fix rdpgfx and events leak

server/proxy: Fix double-call to disconnect

Fix plugin paths when building for a mac bundle.

Merge pull request #5445 from akallabeth/some_small_patches

Fix various small issues

PR fixes

Fixed type mismatch.

Fix #5427: Dispatch updates to main thread.

thank you @Harrywang55666

Fixed #5406: Ensure mouse wheel rotation step is never zero

thank you @victorking528

Fixed #5441, thank you @ekse

Merge pull request #5438 from akallabeth/tsmf_patch

Patch from #5436: fix input handling with gstreamer and vaapisink

Patch from #5436: fix input handling with gstreamer and vaapisink

Merge pull request #5424 from kubistika/rdpgfx

rdpgfx: CacheImportOffer implementation

 codec: Set alpha channel to FF if not used (#5431)

* Formatting

* codec: Set alpha channel to FF if not used

This PR initializes the alpha channel to FF. Before the alpha channel
contained uninitialized memory potentially causing problems when the
image data was used by an alpha channel aware frontend.

Merge pull request #5423 from kubistika/feat/bitmap-auto-flag-calculate

rdpUpdate: add autoBitmapDataCalculate

RDPEDISP server side implementation (#5414)

* libfreerdp: Add RDPEDISP server implementation

* server/proxy: Add RDPEDISP support

rdpgfx/client: Update gfx->UnacknowlegedFrame correctly

server/proxy: Proxy CacheImportOffer & QoeFrameAck

rdpgfx/server: Fix formatting: rdpgfx_recv_cache_import_offer_pdu

rdpgfx/client: Expose QoeFrameAck

rdpgfx/client: Implement CacheImportOffer PDU

server/proxy: Fix bitmap update crash when target is win7

rdpUpdate: add autoCalculateBitmapData flag

Merge pull request #5422 from kubistika/feat/ifdef-rdpgfx-dbg

rdpgfx: Add WITH_DEBUG_RDPGFX cmake option

rdpgfx: Add WITH_DEBUG_RDPGFX option

Merge pull request #5415 from akallabeth/dump_append_fix

Fixed format security issues.

Merge pull request #5405 from m4ntis/proxy/filters-api

Proxy: Filters API

Merge pull request #5416 from m4ntis/proxy/multimon

Add multimon support to proxy server

server/proxy: Add multimon support

libfreerdp/core: Export monitor layout PDU

* Move update notification related functions to new display.c: Not
related to `rdp.c`.
    * Rename `rdp_write_monitor_layout_pdu` to `display_write_monitor_layout_pdu`.
* Add internal `display_convert_rdp_monitor_to_monitor_def` instead of
doing the conversion inside `display_write_monitor_layout_pdu`.

Merge pull request #5418 from kubistika/feat/proxy/cursor

server/proxy: Proxy pointer updates

channels: ssh-agnet: cmake: turn off server side channel by default in `ChannelsOptions.cmake` (#5420)

CMake v3.5.1 throws the following error if this is not set:
```
CMake Error at channels/CMakeLists.txt:48 (if):
  if given arguments:

    "OFF" "OR"

  Unknown arguments specified
Call Stack (most recent call first):
  channels/sshagent/ChannelOptions.cmake:6 (define_channel_options)
  channels/CMakeLists.txt:273 (include)
```

RAIL: Fix parsing of WINDOW_ORDER_FIELD_ICON_OVERLAY_NULL (#5421)

server/proxy: Proxy pointer updates

* Moved update callbacks of client to pf_update.h

fix vs2010 errors

server/proxy: Code refactor

server/proxy: Use winpr library in proxy/filters

server/proxy: Fix free invalid memory when pf_server_parse_target_from_routing_token fails

server/proxy: Fix mem leak in pf_server_parse_target_from_routing_token

server/proxy: Remove unnecessary wrap for connection info

server/proxy: Add docs in filters_api.h

server/proxy: Add return statement in demo_filter.c

Add README for filters API

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

Fixed format security issues.

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>

yet another one revertion