Cleanup unnecessary/unintended changes from last commit

RemoteApp Support

1. Remove all uses of "localWindowOffsetCorr" variables, they added an extra layer of complexity and they are not actually needed to handle coordination of window position/size between
the local coordinate system and the remote one. This logic was causing issues in the case where the window was moved off the left side of the screen.

2. Update the xf_setWindowVisibilityRects function to offset the visibility rects as necessary when the window is hanging off the left side of the screen.

3. Stop sending mouse events when doing keyboard moves/sizes(as desired), and stop sending two mouse events for non-keyboard moves/sizes

4. Move location of new UTF8_STRING variable from previous commit

5. Refresh window and window shape for any window position/size updates, this helps keep the local and server windows in sync and works around some race conditions

Remote app support

All window state order offsets are signed according to the RDP spec, lets treat them as such.

Store Miscellaneous X11 Atoms into variables to match how we handle other X11 Atoms.

RemoteApp support

Remove use of WindowRects to affect window shape, the VisibilityRects are used for this purpose.

Ensure that app windows always get a window type assigned to them.

Continue processing all other window orders for new remote app windows instead
of existing after only looking at part of the information. For instance, window visibility
rects are part of the message with the new order and were being ignored.

Do not support fullscreen toggle keyboard sequence in remote_app mode. There is no support in interface for this
and the fullscreen code is not designed to handle remote app windows.

Fix xf_rail_paint fencepost error

The regions used to store and calculate the invalidRegion are exclusive
of the bottom and right edges, not inclusive.

Fixes "mouse droppings" in mspaint.exe when moving the mouse leftwards
across the canvas.

Merge pull request #2924 from awakecoding/master

egfx multimon fixes

libfreerdp-gdi: fix egfx multimon support

xfreerdp: fix multiple egfx context support in X11 GFX

Merge branch 'master' of github.com:FreeRDP/FreeRDP

Merge pull request #2919 from realjiangms/fix_allow_empty_password

Sec/NLA: Support passwordless (blank password) login with NLA.

Merge pull request #2920 from awakecoding/mbedtls

WinPR mbed TLS support

libwinpr-crypto: fix OpenSSL 1.0 HMAC API change

libwinpr-crypto: fix error checking

libwinpr-crypto: avoid linking to less common digests and ciphers

Merge pull request #2912 from realjiangms/shadow_fix_auth_nopasswd

Server/Shadow: Minor fix on authentication in shadow_client.c.

Merge branch 'master' of github.com:FreeRDP/FreeRDP into mbedtls

winpr: make clean non-OpenSSL build possible (without schannel, makecert)

libwinpr-crypto: add generic digest API and OpenSSL compatible key derivation

Sec/NLA: Support passwordless (blank password) login with NLA.
It was supported in freerdp 1.0.2 but not supported in lastest master.
We should take empty password if it is explicitly specified with /v option.
If a password is not specified, we could first try SAM file. If the user entry does not exist, prompt for password.

libwinpr-crypto: add generic cipher API with OpenSSL/mbedtls support

libwinpr-sspi: remove OpenSSL dependency in NTLM SSPI module

libwinpr-crypto: add rand and RC4 functions with OpenSSL/mbedtls

libwinpr-crypto: add basic hashing and HMAC support with OpenSSL/mbedTLS

winpr: isolate OpenSSL

Merge pull request #2914 from sriramster/master

Remove warning: HANDLE event variable unused

winpr: start porting to non-OpenSSL

Remove warning: HANDLE event variable unused

Server/Shadow: Fix Authentication in shadow_client.c.
Currently if username or password is not set, the authentication is always failed because the authentication callback is never tried.
Fix to always give a chance to try the subsystem authentication callback even if username / password is not set.

Merge pull request #2909 from akallabeth/clipboard_remove_copy

Speedup for clipboard

Merge pull request #2901 from volth/patch-1

Fixed crash when unused Japanese keys are pressed.

Remove unnecessary copy operation.

Merge pull request #2864 from bjcollins/smartcard_same_name_readers

Give unique aliases for smartcards with the same name, so that two re…

Merge pull request #2908 from hardening/fix_region

Fix a bug in region_union_rect()

Fix a bug in region_union_rect()

When region_union_rect() was called with a rect that was above the region without
intersecting it, the first band was created too large. This patch fixes this and
add a unitary test.

All credits go to @nfedera that:
* found the bug;
* find a dataset to reproduce;
* code the unitary test;
* spotted a suspicious line that was the faulty one.

Merge pull request #2903 from ssieb/fix-redir

Don't fail the redirect if padding wasn't sent before disconnecting

Don't fail the redirect if padding wasn't sent before disconnecting

Update xf_keyboard.c

With Japanese physical keyboard and English layout, pressing the extra keys can cause FreeRDP crash.

afreerdp: fix build warnings

Merge pull request #2899 from awakecoding/master

Standard RDP security server redirection fix

libfreerdp-core: fix redirection with standard RDP security

Merge pull request #2897 from bjcollins/rail_fix_window_title

Minor rail fixes/tweaks

Merge pull request #2898 from bjcollins/smartcard_status_buffer_size

Fix use of invalid input causing SCARD_E_INSUFFICIENT_BUFFER errors

Dont use invalid/bogus input value for cbAtrLen for Smartcard Status Calls, instead just always use the max value here to indicate we have max
buffer space available.

fix window title for rail as it was not being set on initial window creation
remove duplicate call to XStoreName when setting window title
expand WITH_XEXT #define for rail window rects as extra unecessary work was being done when WITH_XEXT was not defined

Merge pull request #2896 from awakecoding/master

Win32 Gateway Fix, iOS TCP connection infinite loop fix

libwinpr-synch: fix broken FD_ISSET check caused by hidden type cast from int to char

libfreerdp-core: fix gateway connectivity on Windows

libwinpr-sspi: don't warn about non-error security statuses

Merge pull request #2895 from aballier/ffmpeg

channels/tsmf/client/ffmpeg: Fix build with ffmpeg git master.

channels/tsmf/client/ffmpeg: Fix build with ffmpeg git 

Replace old, deprecated and now removed, APIs with their new equivalent while retaining backward compatibility with old ffmpeg versions.

Merge pull request #2890 from realjiangms/shadow_fix_memory_leak

server: Fix several memory leak found while running valgrind on freerdp-shadow

server: Fix several memory leak while running valgrind on freerdp-shadow

Merge branch 'master' of github.com:FreeRDP/FreeRDP

Merge pull request #2889 from dvincent-devolutions/gateway

Fix RDP8 gateway disconnections

Small corrections to BIO implementation.

cmake: add mbed TLS detection

libfreerdp-crypto: add locking of operations dealing with SSL*

libfreerdp-core: alternative RDG BIO fix

libfreerdp-core: fix code style

Merge pull request #2887 from FreeRDP/mh-file-api-extension

winpr/file: extend API

Merge branch 'gateway' of https://github.com/dvincent-devolutions/FreeRDP

Merge pull request #2886 from clangm/master

Fix Network Characteristics Autodetect [RDPBCGR 2.2.14]

Merge pull request #2888 from awakecoding/master

fix ARM64 data type portability

Merge pull request #2879 from peterh/shape

xfreerdp: Fix window shape

libfreerdp-core: Gateway RDP8 BIO correction. Fixes connection losses.

Merge pull request #2763 from akallabeth/nla_null_auth

Fixed server and client NLA state machine for NULL identity.

Merge pull request #2859 from akallabeth/resource_leak_fix

Fixes API misuse and logic errors

winpr/file: extend API

added windows code
fixed compiler issue

winpr/file: extend API

Add function GetFileHandleForFileDescriptor to get
a file handle for a valid file descriptor.

Merge pull request #2883 from bceverly/master

Updated OpenBSD changes

winpr: fix ARM64 data type portability

Merge remote-tracking branch 'upstream/master'

Cleaned up previous OpenBSD change per recommendation

Merge pull request #2877 from bmiklautz/filehandling

winpr/file refactoring and regular file support

Removed if statement that was causing network characteristics autodetect to not work

Updated OpenBSD changes

Merge pull request #2880 from bceverly/master

Fixes for OpenBSD port

Fixes for OpenBSD port

xfreerdp: Fix window shape

Don't abort the entire xf_rail_window_common function when the window is
already in the correct location.

To reproduce:
 - move an application off the edge of the screen
 - resize the window to cause a shape to be set
 - move the application window fully on the screen
 - resize the application window larger
 - note lack of drawing in newly enlarged portion of window

Bug introduced in abf6d4f71ea901e31ee2d1b4625fa805af6b0cc9 "xfreerdp:
prepare RAIL migration away from libfreerdp-rail" when
xf_rail_MoveWindow was copy-and-pasted into xf_rail_window_common
without noticing that the "return" would omit the rest of the combined
function, not just the portion that was pasted.

Merge pull request #2878 from akallabeth/screen_orientation_types

Added ORIENTATION_PREFERENCE from WinUser.h.

Added ORIENTATION_PREFERENCE from WinUser.h.

winpr/file: integrate pull request feedback

* simplify RemoveDirectory
* move std handle function into an extern C block

winpr/file: don't close stdin/stdout/stderr

If the handle is closed stdin/stdout/stderr should be left open.

winpr/file: fix ios build

comm is only build on linux.

winpr/file: initial regular file support

Add initial support for "regular" files. First implemented call is
GetStdHandle to get stdin/stdout/stderr.

winpr/file: refactoring

The used method for initalizing and registering the handle creators
had certain disadvantages like that it was necessary to have the creators
initialized before the first call to CreateFile. In case of comm it
wouldn't have been possible to open an comm port without calling any
other call related function first.
Now the handle creators are initialized the first time CreateFile is
called. This way it is also possible to ensure a certain order of the
handler invocations.

This commit also splits out the client side named pipe code from
the generic file handling (now called generic.c instead of file.c) and
uses the new handle creator infrastructure to register.

Merge pull request #2868 from nfedera/fix-inverted-pointer-colors

improve handling of inverted pointer colors

Merge pull request #2873 from mfleisz/makecert_change

makecert: Remove key usage attribute

Checking for allocation value > 0

makecert: Remove key usage attribute

The original makecert tool won't add this tribute to a generated
certificate. Adding this attribute might cause problems when using the
certificate with other apps (i.e. Qt SSL).

Merge pull request #2872 from akallabeth/recording_fix

Fixed error checking and recording reset.

Merge pull request #2765 from akallabeth/connect_abort

Connect abort and unified TCP connect

Fixed error checking and recording reset.

Fixed CreateProcess argument initialisation.

Enable RFX for test connection, now required by sample server.

Resetting abortEvent on disconnect to avoid race during connect.