Fix possible NULL pointer access crash

Updated CMake channel detection

* Now both, dynamic and static channel entries can be defined by
  a single channel.
* Added better logging to distinguish between static and dynamic
  channel messages.

Add dynamic channel part for rdpsnd channel

Merge pull request #5920 from akallabeth/rdpsnd_crash_fix

Fix #5885: Check for division by zero

Merge pull request #5922 from akallabeth/channel_drdynvc_fi

Replaced CMake list find with loop to fix build issues.

fix FindFFmpeg.cmake: locating avcodec not using pkgconfig variable

Fix #5885: Check for division by zero

Replaced CMake list find with loop to fix build issues.

Merge pull request #5921 from akallabeth/nightly_optimise

Revert to RelWithDebInfo for debian.

Revert to RelWithDebInfo for debian.

Merge pull request #5907 from akallabeth/cmake_channel_dependency_fix

Cmake channel dependency fix

Deactivated URBDRC for mac and windows builds.

Merge pull request #5915 from akallabeth/gfx_versions

Updated RDPGFX_CAPVERSION_106 according to errata 2018-12-10

Merge pull request #5906 from akallabeth/tsmf_disable_for_good

Made CHANNEL_TSMF optional and deprecated.

Updated RDPGFX_CAPVERSION_106 according to errata 2018-12-10

egfx: the gfx reset grahics pdu is optional

Since the EGFX Reset Graphics PDU seems to be optional,
the graphicsReset variable (which is updated in that PDU's handler)
should be removed from the rdp_gdi struct with the next change
in public headers (as in freerdp v3).
There are still some clients that expect and check it and therefore
we keep it for now, initialized with TRUE.

sdas

Made CHANNEL_TSMF optional and deprecated.

Deactivate USB redirection for IOS and ANDROID by default.

Do not advertise /usb in help output if the channel is not built

The help output advertise an option and an example for usb redirection
regardless of the fact whether the urbdrc channel was built or not.
This is confusing for people. Let's do not show /usb in help if it is
not built.

server: proxy: client: change ClientFree->ContextFree

server: proxy: support static vc passthrough

core: channels: add API to get channel name/id

drdynvc: reorder filepaths when adding channels

Promoted usb, printer and smartcard dependencies to recommended

Fixed CMake channel dependencies for variables.

Merge pull request #5905 from akallabeth/pulse_audin_fix

audin pulse log states by name

audin pulse log states by name

Merge pull request #5707 from akallabeth/kbd_codepage

Added KeyboardCodePage option

Updated RDP_KEYBOARD_LAYOUT_TABLE

Regenerated with new fetch_language_identifiers.py, less manual
postprocessing required.

Fixed keyboard lang struct generation

* Fixed length hex strings
* Fill in missing elements with last valid

Added keyboard language codepages and listing option

Merge pull request #5732 from akallabeth/floatbar_button_fix

Fixed #5730: Decoupled floatbar button focus from click events.

Merge pull request #5901 from nfedera/add_connstate_active_timeout

core: fix endless loops waiting for activation

core: fix endless loops waiting for activation

There are two loops polling the transport pdu receiver in non-blocking mode
when waiting for reaching CONNECTION_STATE_ACTIVE rdp state.

In case of an invalid pdu size in the tpkt header this leaded to an endless
loop, utilizing 100% of a cpu core.

Added a sleep and limited the max loop time to the tcp ack timout value.

Merge pull request #5899 from akallabeth/respect_abort_connect

End connection before user callbacks if aborted.

End connection before user callbacks if aborted.

If somewhere in freerdp_connect freerdp_abort_connect was called
the user callbacks Authenticate, GatewayAuthenticate and
Verify[Changed|X509]Certificate[Ex] must not be called.

Merge pull request #5898 from akallabeth/timezone_update_20200219

Updated timezones 2020-02-19

Updated timezones 2020-02-19

Merge pull request #5896 from akallabeth/cast_warning_fixes

Fixed #5895: cast warnings.

Merge pull request #5897 from akallabeth/flatpak_update

Updated flatpak build manifest

Updated flatpak build manifest

* Use platform 19.08
* Remove shared-modules, no longer required (udev is now part of the
platform)
* Switch PCSC source to github to avoid rate limitations.

Fixed #5895: cast warnings.

Fixed #5889: Only add descriptors wanted to select

Fix markup

Merge pull request #5890 from akallabeth/sample_client_install

Install sample client binary.

Fix channel linking for built in channels

Added some callbacks to dynamic channel

* Allow listener removal of channel during runtime
* Allow retrieval of channel name

working progressive Original Method DWT

Implemented decoding utilizing Original Method DWT
(See [MS-RDPEGFX] 3.2.8.1.2.2 Reduce-Extrapolate Method)

Refactored progressive codec.

Install sample client binary.

client: remove unused variable

Merge pull request #5885 from llyzs/rdpsnd_overrun

rdpsnd: implement buffer overrun detection.

rdpsnd: implement buffer overrun detection.

Merge pull request #5881 from akallabeth/win_disable_console

Disable windows console attach.

Merge pull request #5882 from akallabeth/sample_auth_only

Added /auth-only option to sample client.

Added /auth-only option to samle client.

In #5814 the option to use a connection test was expressed.
Adding this option to the sample client allows doing that without
X11 or similar around.

Disable windows console attach.

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