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

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.

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.

Fixed error handling for rdp_client_connect fail.

WSAEvent fixes ported from realjiangms pull 2870.

Removed rdp disconnect, using unified abortEvent instead.

Merge pull request #2869 from peterh/negorigin

X11 /app: Fix drawing with negative window origin

Fix drawing with negative window origin

Since REGION16 uses unsigned values, when appWindow->x or appWindow->y
is negative, the region will have a very large left or top value.

Avoid this problem by clamping to 0 before casting to an unsigned value.

improve handling of inverted pointer colors

Merge pull request #2865 from akallabeth/h264_ignore_errors

[H264] ignore decoder errors

Merge pull request #2867 from akallabeth/channel_error_reset

Resetting channel error on freerdp_connect.

Resetting channel error on connect.

Fixed uninitialized environment block.

Ignoring H264 decompression failures.

When connecting to windows 8.1 machines the remote RDP server
implementation sometimes sends invalid H264 data. To avoid client
disconnections ignore the broken updates.

Generalized rdpgfx_decode to work with all codecs.

Reverted broken malloc size fix.

Merge pull request #2861 from llyzs/winmm_volume_fix

rdpsnd/winmm: fix SetVolume when device is not opened.

Merge pull request #2852 from bmiklautz/process_env

Update CreateProcess and rfx

rdpsnd/winmm: fix SetVolume when device is not opened.

Fixed possible memory leak.

Fixed uninitialized values.

Fixed uninitialized values.

Fixed zero length allocation.

Fixed zero lenght allocation.

Fixed zero length allocation.

Fixed zero length allocation.

Fixed uninitialized memory.

Fixed array out of bound access.

Fixed broken structure comparison.

Fixed error handling of OSS audio plugin.

Fixed resource leak.

Merge pull request #2825 from akallabeth/pointer_decoding_fix

Fixed color pointer decoding.

Ignore AND mask for 32 bit pointer.

Merge pull request #2855 from akallabeth/memleak_fixes

Memleak fixes

Fixed broken cleanup code.

Merge pull request #2854 from bmiklautz/issue/2853

settings: update settings/help to match

Fixed cleanup code.

Fixed memory leak.

settings: update settings/help to match

* wallpaper was off but help said "default: on"
* compression is on but help said "default: off"

Fixes #2853

Fixed memory leak.

Fixed memory leak.

Fixed memory leak.