Removed unused function.

Using bsearch to get ntstatus string.

Silenced warning due to missing debug define guard.

Removed unused functions / files

Unify pReceiveChannelData and psPeerReceiveChannelData

Fix definitions of the two function pointers.
Use and definition did not match, fix that.
Will create warnings in external projects

Fixed misinterpretation of SendChannelData

SendChannelData was defined with a return value of type int, but
used as BOOL everywhere. Fix the definition to match use.

Fix tpkt header length checks for encrypted packets

If securityFlag SEC_ENCRYPT is set, remove the encryption headers from
the TPKT header length on comparison.

Merge pull request #5963 from akallabeth/fixes

Fixes some warnings and checks

Default to positive return for missing callbacks

When using +async-update, default to positive return if some
client callback is not implemented.

Fixed broken NULL check.

Fixed return check for usbdk backend.

Removed unnecessary cast.

Merge pull request #5958 from akallabeth/usb_win_fixes

Fixed missing config include.

Fixed missing config include.

Merge pull request #5957 from akallabeth/cmake_libusb

Added libusb detection for windows.

Added libusb detection for windows.

Merge pull request #5954 from akallabeth/server_config_options

Server config options

Default proxy modules to OFF

The proxy module sample is c++ 11 which some older build targets
do not like. Deactivate this by default.

Set C++ standard for proxy demo target

CMake options to compile without proxy or shadow

* Added WITH_SHADOW (default ON) to compile without shadow server
* Added WITH_PROXY (default ON) to compile without proxy server

egfx: client: add missing NULL checks

[RDPEGFX] checks for cache import, prevent 0 size allocations

Move libusb hotplug registration to polling thread.

* libusb polling thread now is responsible for hotplug registration
  and removal as well as cleanup.
* Only register hotplug callback on systems with support.

enable usbdk on windows.

Fixed compilation issues against libusb on windows

Fixed strncmp, use _strnicmp

Fixed memory leak in tls_get_channel_bindings

Added raw function wrapping X509_digest

Added winpr_md_type_from_string and winpr_md_type_to_string

Let ssl backend handle hash checks.

Added sha3 hashes for openssl.

Added new option /cert that unifies all.

* The mess with /cert-tofu, /cert-ignore et al is now unified in
  a single option.
* Added the option to add fingerprint:<hash>:<hex string> multiple
  times to /cert to build a list of accepted certificate
  fingerprints
* Added a deprecation warning to older /cert-* options

Check cert against CertificateAcceptedFingerprints

CertificateAcceptedFingerprints may contain a list of certificate
hashes and the corresponding fingerprint.
If one of the hashes matches consider the certificate accepted.

Added CertificateAcceptedFingerprints to settings

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.