Added missing WINPR_*_DIGEST_LENGTH definitions.

Consume all TPKT data reading new/upgrade license

rdp_recv_tpkt_pdu verbose debug parsing issues

Print out parsing issues found in MCS Channel 1003 parsing.

tpkt_ensure_stream_consumed now more verbose.

tpkt_ensure_stream_consumed now writes the function it was called
from to the log. This should help identify areas the check failed.

Merge pull request #5944 from akallabeth/fragmented_packet_length_check_fix

Fragmented packet length check fix

Added clarifications in freerdp_channel_process

Fixed arguments for peer->ReceiveChannelData

Fixed conversion and return checks.

* Fix some missing argument checks for function pointer implementations
* Fix broken return value check for client->SendChannelData
* Updated const correctness for function pointer implementations

Fixed function pointer const correctness.

Fixed freerdp_channel_process length checks

Fragmented data was not handled properly.

Added freerdp_channels_data argument checks.

Merge pull request #5942 from akallabeth/gfx_cache_evict

Fix GFX cache eviction, use 1 based indexing.

Fix GFX cache eviction, use 1 based indexing.

X11 XEvent handling use const arguments

X11 XEvent disp refactored to use const arguments

X11 XEvent refactored input

* Use proper types and const arguments where appropriate

X11 XEvent refactored floatbar

* Use const event pointers of correct type where appropriate

X11 XEvent clipboard processing cleanup

* Use const X*Event where possible
* Helper functions use the actual type as argument

X11: Lock display during event processing

* Added a X11 lock debug messages for incorrect locking.
* Lock X11 display during XEvent processing

Merge pull request #5939 from akallabeth/scaling_patch

Reverted minimum scaling diff

Merge pull request #5941 from akallabeth/clang_fixes

Fixed clang scanbuild warnings.

Merge pull request #5932 from nfedera/fix_gfx_bitmapcache_indexing

Fix RDPEGFX bitmap cache indexing

Fixed clang scanbuild warnings.

Merge pull request #5940 from akallabeth/pretty_print_signals

Improve signal handler printout.

Improve signal handler printout.

dynvc/client: fix and improve channel closing code

- fixed and consolitate the duplicated code for sending the
  CLOSE_REQUEST_PDU to the server into dvcman_close_channel
- call dvcman_close_channel if a dynamic channel plugin fails
  to process the received channel data
- rdpegfx: don't try to remove a non-existing cache entry,
  return an error instead which now will close the channel, as
  expected by Microsoft's windows protocols test suite

Reverted minimum scaling diff

Revert to the diff value from @volth

Merge pull request #5935 from akallabeth/shadow_server_mstsc_fix

Fixed shadow server with mstsc as client.

Fixed reading suppress output pdu

Optional rectangle must be read from PDU

Removed function pointer casts for shadow.

Fixed shadow server with mstsc as client.

Merge pull request #5934 from akallabeth/urbdrc_close_fix

URBDRC: Only mark closed if control is closed.

URBDRC: Only mark closed if control is closed.

Merge pull request #5933 from akallabeth/autodetect_consume_stream

Consume stream data in autoconnect

Consume stream data in autoconnect

TPKT checks failed due to payload data not consumed in autodetect.

wf_cliprdr.c forget OleUninitialize if create_cliprdr_window failed

Fixed tpkt header length checks.

TPKT header length and consumption of data was not consistently
checked. This adds checks after each packet processed and fixes
any inconsistencies found.

Fixed warnings in capabilities.c

Fixed compiler warnings in info.c

Added tpkt header length plausibility checks.

Added fallbacks for primitives auto detection.

When using primitives auto detection it was possible that no
primitive implementation was chosen at all.
This now skips broken implementations and falls back to generic
primitives if no other implementation does work.

Check length of TPKT header for plausibility.

Implemented del_device by vid/pid

Cleaned up USB redirection

* Removed obsolete searchman
* Unified device add and virtual channel add
* Unified device id generation

Fix #5910: USB hotplug added device wrong

When a USB device was added via hotplug add_device was called
with wrong arguments.

rdpegfx: rename MaxCacheSlot to MaxCacheSlots

Although Microsoft uses 1-based numbering on the wire for indexing the
GFX bitmap cache entries, the code in FreeRDP uses 0-based numbering and
therefore the name MaxCacheSlots is less confusing.

rdpegfx: use 1-based indexing for bitmap cache

Weird but Microsoft uses 1-based indexing in the RDPGFX bitmap
cache PDU's.

This does not seem to be documented but can be deducted from the
RDP client test code in Microsoft's "Windows Protocol Test Suites"
GitHub repository and the observation that mstsc aborts with a
protocol error if the cacheSlot index value 0 is used in e.g. a
GFX surface to cache PDU.

Merge pull request #5931 from akallabeth/rpm_nightly_fix

Disable GSS/KRB5 support

Disable GSS/KRB5 support

Kerberos support in FreeRDP is not working, deactivate it. Fixes #5930

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