fix channel/smartcard: remove SCardAddReaderName

SCardAddReaderName isn't part of the SCard API.

Note: removing this also removes the possibility to redirect single
smartcard readers with /smartcard:READERNAME. However this features
wasn't implemented in a general way and will be re-added as part of
the smart card channel directly.

fix channel/smartcard: simplify channel variables

Path was not really used and name was duplicated. Use the device->Name
directly.

fix channel/smartcard: remove status mappings

The state tracking/modifications (presumably thought as optimization?!) in
PCSC_SCardGetStatusChange_Internal cause a lot of applications to behave
incorrectly and/or hang. Ideally no modifications of the states should
be necessary as PCSC implements the same API as passed over the channel.

fix channel/smartcard: GetStatusChange return code

In case SCardGetStatusChange returned an error the call didn't return
any data but STATUS_NO_MEMORY as the calloc failed. This caused problems
with multiple applications server side (hangs and incorrect behavior).

Now the case when no readers are returned is handed correctly and the
data is also filled and send if the call fails.

fix channel/smartcard: async request handling

The smart card channel tried to mimic mstsc's behavior on if an IRP was
processed synchronously or asynchronously. As the channel uses one thread per
context it could happen, especially with PCSC, that the main
channel thread was blocked waiting for an smart card operation to
complete. To prevent that behavior only call known safe functions in the
main thread (like CreateContext) and call the rest asynchronously.

For example the channel would block if a ListReaders is invoked on
the same context where a GetStatusChange (infinite timeout)
was already pending. Only when a status change happened the channel
would continue.

Note: Due to the one context per thread design it's important that
cancel isn't queued an alway run synchronously. Otherwise a specific
context might lock.

fix channel/smartcard: return value handling

Device control calls always returned SCARD_S_SUCCESS even if an error
occurred. This caused server side software (including the card manager)
to behave incorrectly.

fix channel/smartcard: error handling

According to MS-RDPESC the smart card channel must set the IoStatus to
an NTSTATUS in case a encoding or decoding error happens. The smart card
channel did this correctly but the output stream was modified
incorrectly causing the smart card remote manager to stop in error
cases.

Merge pull request #4276 from akallabeth/big_endian_more

Big endian fixes

Merge pull request #4301 from hardening/geometry

geometry: a skeleton for the MS-RDPEGT channel

Merge pull request #4277 from akallabeth/mac_server

Mac fixes

Fixed uninitialized variable.

geometry: a skeleton for the MS-RDPEGT channel

The base for implementing the MS-RDPEGT client channel that allows to track window
geometry.

Merge pull request #4297 from akallabeth/cmd_parser_hardening

Fix #4296: Hardened command line post filter.

Merge pull request #4294 from akallabeth/shadow_fix

Shadow server crash fixes

Merge pull request #4288 from hardening/disp_channel

Disp channel changes and improvements

Merge pull request #4298 from myfreeweb/libressl-fix-fips

Fix LibreSSL build

Fix LibreSSL build

LibreSSL does not support FIPS mode.

drdynvc: fix invalid check

There were a copy'n paste error in the check of a malloc.

display control channel: add a callback called when we receive capabilities

We need a signal to know when the channel is ready and it's safe to send the
monitor layouts.

Fix #4296: Hardened command line post filter.

Merge pull request #4275 from ondrejholy/big-endian-fixes

Big endian fixes

Merge pull request #4292 from daixj-shterm/master

Fix #4291: xfreerdp segfault when connecting freerdp-shadow-cli

Reset errno in X11 monitor enumeration

XOpenDisplay may succeed but set errno,
reset at the end of the function.

Fixed /monitors argument checks.

Fix #4291: xfreerdp segfault when connecting freerdp-shadow-cli

xfreerdp: when available use xrandr to get display infos

Merge pull request #4287 from bmiklautz/dev2

2.0.0-dev2 - start rc2 development cycle

client/android: increase version

2.0.0-dev2 - start rc2 development cycle

Merge pull request #4286 from bmiklautz/rc1

Version 2.0.0-rc1

Version 2.0.0-rc1

Merge pull request #4282 from akallabeth/force_ipv6

Fix #4281: Added option to prefer IPv6 over IPv4

Merge pull request #4284 from akallabeth/channel_count_fix_v2

Fixed channel count exceed checks

Fixed channel count exceed checks

The check aborted on CHANNEL_MAX_COUNT - 1 instead
of correctly at CHANNEL_MAX_COUNT

Merge pull request #4283 from bmiklautz/kfreebsd

fix build: GNU/kFreeBSD

Merge pull request #4272 from akallabeth/static_channel_checks

Fix #3378: 31 static channels are supported.

fix/build: handle GNU/kFreeBSD like other BSDs

Based on a patch for Debian from
Mike Gabriel <mike.gabriel@das-netzwerkteam.de>

fix/build: GNU/kFreeBSD is not FreeBSD

Merge pull request #3991 from akallabeth/stream_string_helper

Stream convenience functions to read/write strings.

Merge pull request #4261 from akallabeth/doc_ssl_store

Fix #3890: Point to OpenSSL doc for private CA

Fix #4281: Added option to prefer IPv6 over IPv4

Merge pull request #4220 from akallabeth/version_from_file_or_tag

Extract build version from file or git tag.

Merge pull request #4273 from akallabeth/issue_template

Added issue template.

Merge pull request #4270 from akallabeth/mac_clipboard_fix

Fixed #1536: Clipboard data size.

Merge pull request #4280 from akallabeth/snd_parser_fix

Fix #4278: Fix nasty range check bug

Added mailing list address.

Added reminder to delete template before submit

Added pull request template.

Added issue template.

Fix #4278: Fix nasty range check bug

Merge pull request #4229 from akallabeth/from_stdin_args

Option to force password prompt before connection

Merge pull request #4268 from akallabeth/big_endian_fixes

Fix ZGfx decoder on big endian

Fixed #3810: Renamed color functions

Added missing channel defines.

Fixed mac server compilation.

Fixed big endian issues with bitmap read.

Fixed endianess issues with ncrush.

Merge pull request #4271 from akallabeth/surface_c

Fixed return value check for callbacks.

Merge pull request #4274 from hardening/clipboard_checks

clipboard: add some checks for the message

client/common: Fix two memory leaks in RDP file processing

FALSE is returned immediately in case of parsing error and thus buffer
copy is leaked. Let's release the memory properly.

winpr/nt: Fix wide char string on big endian

Wide char strings are always little endian encoded and thus
Data_Write_UINT16 has to be used in _RtlAnsiStringToUnicodeString.
It fixes TestIoDevice on big endian machines among others.

https://github.com/FreeRDP/FreeRDP/issues/4231

client/common: Fix RDP file processing on big endian

TestClientRdpFile fails on big endian machines due to some bug in
unicode processing. Let's drop all the unicode functions and convert
unicode input into ascii as soon as possible. This significantly
simplify RDP file processing and also fixes TestClientRdpFile on
big endian machines.

https://github.com/FreeRDP/FreeRDP/issues/4231

clipboard: add some checks for the message

Fix #3378: 31 static channels are supported.

Merge pull request #4255 from akallabeth/rfx_fix

Fix #4253: Rfx decode tile width.

Fixed return value check for callbacks.

Fixed #1536: Clipboard data size.

Merge pull request #4267 from ondrejholy/autofips

Enable FIPS mode automatically

Remove unused variable

Unused variable was added together with FIPS mode support, let's remove it.

Enable FIPS mode automatically

FreeRDP aborts if OpenSSL operates in FIPS mode and +fipsmode is not
manually specified. Let's prevent the abortion and enable the necessary
options in that case automatically.

Fixed endianess issues of zgfx decoder.

Merge pull request #4251 from akallabeth/openssl_warn_fixes

Fix #4247: warnings introduced with #3904

Merge pull request #4244 from akallabeth/drivestoredirect

Fix parsing of drivestoredirect (#3267)

Merge pull request #4258 from akallabeth/kbd_parse_fix

Fix #4257: Proper error checks for /kbd argument

Extract version from file or git tag.

Fix #4257: Proper error checks for /kbd argument

Merge pull request #4263 from akallabeth/rdp_parser_mem_corruption_fix

Fixed #4249: Corruption due to recursive parser

Testing argument to end with file extension.

Merge pull request #4264 from cedrozor/upstream-master

Fixed NLA for Negotiate and NTLM authentication

Fixed NLA for Negotiate and NTLM authentication (regression due to the recent addition of kerberos support)

Fixed #4249: Corruption due to recursive parser

Fix #3890: Point to OpenSSL doc for private CA

Disabled ceritficate signature check.

Fixed X11 stride on screen update.

Fix #4101: Rfx decode tile width.

Merge pull request #4252 from akallabeth/mac_kbd_sync

Mac kbd sync

Merge pull request #4256 from akallabeth/15bpp_fix

Fix #4111: 15bpp color handling for brush.

Fix parsing of drivestoredirect (#3267)

Merge pull request #4254 from akallabeth/doc_update

Fix #3509: Added Ctrl+Alt+Enter description

Fix #4111: 15bpp color handling for brush.

Merge pull request #4253 from akallabeth/win_return

Win return code

Fix #3509: Added Ctrl+Alt+Enter description

Fix #3211: Return freerdp error from main.

Fixed keyboard state sync.

Removed linux specific include.

Fix #4247: warnings introduced with #3904

Merge pull request #4243 from akallabeth/man_fix

Better description for drive redirection (Fix #3513)

Better description for drive redirection (Fix #3513)

Merge pull request #3904 from bjcollins/master

FIPS Mode support for xfreerdp

Merge pull request #4236 from akallabeth/scan_fix_remastered

Scanbuild warning fixes