Merge pull request #4210 from akallabeth/nla_errors_extended

Added additional NLA error mappings.

Merge pull request #4053 from akallabeth/ffmpeg_encoder

Implemented FFMPEG based encoder.

Merge pull request #4200 from cedrozor/upstream-master

Windows client: refresh only updated region(s) instead of whole screen

Merge pull request #4187 from hardening/multimon_fix

Multimonitor fix

Added additional NLA error mappings.

Merge pull request #4206 from akallabeth/x_window_title

X11 window title fix

X11 window title fix

Merge pull request #4205 from akallabeth/WIN32_FIND_DATA

Fixed #4199: ConvertFindDataAToW string length

Fixed #4199: ConvertFindDataAToW string length

Merge pull request #4159 from aleksei-klv/dex

Optimized aFreeRDP for Samsung DeX

Merge pull request #4179 from akallabeth/android_cleanup_fix

[android] Ensure native session is disconnected before resource free.

Merge pull request #4204 from lunixoid/master

Fixed twice assigned values

Fixed twice assigned values

Merge pull request #4203 from floppym/mkdir-error

winpr: _IoCreateDeviceEx: fix mkdir error check

winpr: _IoCreateDeviceEx: fix mkdir error check

The mkdir(2) function returns 0 on success, and -1 on error.

This resolves an error in TestIoDevice when /tmp/.device/ does not
exist.

Bug: https://bugs.gentoo.org/635838

Merge pull request #4201 from kakaroto/listener-unlink-bugfix

Do not delete the listener socket right after creating it.

Merge pull request #4202 from lunixoid/master

Static analyser issues

Update semaphore.c

Fixed Expression 'pComm != NULL' is always true

pComm was checked earlier in 1363 string:
`if (pComm == NULL)`

Fixed duplicated assigne

Do not delete the listener socket right after creating it.

The listener server socket file needs to be deleted before we bind it
otherwise it's an "address already in use" error. But it was getting
deleted after the bind, causing the file to disappear, and preventing
anyone from connecting to the listener socket since the socket stops
existing.

This is caused by commit 884e87fde4ece132c3b7bdf947de044ecb3d4b14

fixed hdc initialization into wf_begin_paint so the region invalidation into wf_end_paint applies to the same hdc
the objective is that only the updated region(s) are refreshed instead of the whole screen
it's not much a problem for normal FreeRDP usage but becomes a serious issue if the display is processed, forwarded and rendered remotely, as done by web gateways such as Guacamole, FreeRDP-WebConnect and Myrtille (I'm the author of the latter)

remove temporary hack

Merge pull request #4198 from akallabeth/nla_error_fix

Added better error mapping for NEGO results.

Added better error mapping for NEGO results.

Fixed possible race condition on resource cleanup.

Merge pull request #4195 from nfedera/fix-sse3-yuv444pixel

primitives/yuv: fix endianess issue

primitives/yuv: fix endianess issue

Merge pull request #4190 from akallabeth/media_foundation_fix_win7

Removed unused function pointer

Removed unused function pointer

The function was breaking windows 7 compatibility.

Merge pull request #4189 from nfedera/fix-h264-init

codec/h264: fix potential segfault and mf_init

codec/h264: fix potential segfault and mf_init

multimon: flag the primary monitor according to the /monitors option

The first monitor of the list is considered like the primary monitor.

multimon: correctly set the primary monitor

According to the spec the primary monitor is supposed to be in (0,0) and other monitors
to be given relative to this one.

Added default return if instance not in list.

Merge pull request #4177 from rjcorrig/tabfix

Android: Fix tab key mapping for hardware keyboard

Ensure native session is disconnected before resource free.

Fix tab key mapping for hardware keyboard

Merge pull request #4175 from akallabeth/time_fix

Fixed file timestamps.

Merge pull request #4172 from hardening/egfx_cache

egfx: fix disconnection caused by invalid cache entries due to wrong announced cache size

Fixed file timestamps.

egfx: fix disconnection caused by invalid cache entries due to wrong announced cache size

Added some checks so that when setting a cache entry fails, we close connection (or
we fail later when trying to use that empty entry).
The small cache egfx capability has also been fixed.

Merge pull request #4164 from akallabeth/ak/mac_audio_refactor

Pause playback on queue empty, free buffers on close.

Merge pull request #4167 from bmiklautz/buildversion

 feat: add support for .source_version

feat freerdp-nighlty/deb: create .source_version

feat: add support for .source_version

When building packages, especially when source packages are used, git is
not necessarily available or the source isn't provided in git. In those
cases it wasn't possible to set the GIT_REVISION and --version shows
"n/a" for the git revision.

If the file .source_version is available now the content of it is used
as GIT_REVISION. Packagers might want to add a .source_version file
when they don't build the packages from git.

Possible breaking change:

The variable PRODUCT_VERSION isn't available anymore. Use GIT_REVISION
instead.

Merge pull request #4162 from hardening/misc_fix2

Misc fix and improvements for the DR channel

Pause playback on queue empty, free buffers on close.

Add checks for DR channel

Fix for #4161

The checks were wrong, fix them.

fix crashing app in dex mode

fix right click

revert gradle

Add Samsung DeX support

Merge pull request #4154 from hardening/misc_fixes

Fix raw surfaces displaying + misc other changes

Merge pull request #4144 from bmiklautz/clipboardon

Enable clipboard channel per default

Enable clipboard channel per default

Fix displaying of raw surfaces

Check return value of malloc

Merge pull request #4156 from akallabeth/test_connect_extend_timeout

Added delay for connect abort

Added delay for connect abort

The connection abort must be called after freerdp_connect.
Ensure that this function is already running by waiting
a second.

Merge pull request #4155 from akallabeth/test_connect_extend_timeout

TestConnect: Extend timeout, only listen locally

TestConnect: Extend timeout, only listen locally

Merge pull request #4152 from akallabeth/dynamic_channel_test_fix

Dynamic channel test fix

Fixed TestConnect with dynamic channels.

Fixed client context setup with default initializer.

Fixed resizing of PubSub

Merge pull request #4150 from akallabeth/server_file_handle_leak

Unlink file after binding to it.

Unlink file after binding to it.

When unlinking the file before binding, a new entry is created
in the file system after binding. This is not desireable, so
unlink it after binding to remove the temporary file after the process
closes.

extend /size to allow width or height percentages (#4146)

If the size parameter is used with a percentages like /size:50% now
an additional 'w' or 'h' can be appended (like /size:50%w) to specify
where the percentage should be applied. If both or none are set the
behavior is like it was before and the percentage is applied to width
and height.

Merge pull request #4060 from akallabeth/icu_support

Using ICU instead of custom unicode conversion.

Merge pull request #4142 from bmiklautz/macOS

Fix build on macOS

Merge pull request #4143 from bmiklautz/pass_is_sc_pin

Add support for "password is pin"

Expose PasswordIsSmartcardPin settings from cmdline via password-is-pin arg

Using PasswordIsSmartcardPin option when sending TS_INFO_PACKET

macOS: add a README file

Add a README file containing some notes about building FreeRDP on macOS
related to openssl.

winpr/makecert: fix linking against libcrypto

Use OPENSSL_CRYPTO_LIBRARIES instead of linking against crypto directly.

macOS: add a workaround to build with newer SDKs

SDK 10.13 (and possibly olders as well) need HAVE_UNISTD_H to be set to
a value if available otherwise the following error is thrown:

/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.13.sdk/usr/include/dispatch/dispatch.h:38:45:
 error: expected value in expression #if !defined(HAVE_UNISTD_H) || HAVE_UNISTD_H

Merge pull request #4045 from akallabeth/drive_fixes_overlayfs

Drive fixes overlayfs

Merge pull request #4136 from tditsch/master

Fixed endless loop when RDP Server sends SERVER_DENIED_CONNECTION

Merge pull request #4116 from ChickenPeanut/patch-1

Fixed BadValue with XCreateWindow

Merge pull request #4138 from akallabeth/ak/nla_free_fix

Check and invalidate handles on free.

Check and invalidate handles on free.

refactored Bugfix

Fixed endless loop when RDP Server sends SERVER_DENIED_CONNECTION

Fix colors on big endian (#4135)

* client/x11: Fix colors on big endian

The bitmaps are recieved in little endian byte order (LSBFirst in terms
of X11). This is a problem on systems, where bitmaps are expected in big
endian byte order (MSBFirst). X11 client tries to handle this situation
by different color formats (e.g. RGBX vs. BGRX), but this doesn't work.
BGRX is not MSBFirst variant of RGBX, it should be XRGB. But this would
work only for 32/24 depths, where color channels matches bytes. Let's
say to the XServer that all the bitmaps are in LSBFirst encoding instead
of changing the color formats on big endian systems.

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

* client/x11: Fix cursor color on big endian

The cursor color is wrong on big endian systems. It is not probably
possible to force bitmap_order for XcursorImage as it is possible for
XImage. Fortunately, cursors are always 32 bit so we can use ABGR
instead of BGRA to deal with.

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

* client/x11: Fix comment indentation

The comment has wrong indentation for some reason, let's fix it.

* client/x11: Fix BGR vs. RGB detection

The BGR vs. RGB detection code is leftover from history and I am conviced
that it is wrong with the current color handling, where invert flag is TRUE
by default. However, I suppose that the detection still makes sense and
XServer may use the inverted formats in some cases. Maybe we can force XServer
to use our masks somehow, but let's just fix the value to FALSE now.

* client/x11: Remove unused color shifts

The color shifts are lefover from history and are not used in current
code. Let's remove them.

Merge pull request #4121 from ondrejholy/setsurface-segfault-fix

Various fixes from #4117

Merge pull request #4052 from akallabeth/mac_fixes

Mac drive fixes

Merge pull request #4127 from mfleisz/avc420

channels: Don't send gfx capversion 10 if AVC420 is requested

channels: Don't send gfx capversion 10 if AVC420 is requested

cache: Use Bitmap_Free instead of free

Bitmap_Alloc is used to allocate memory of bitmap, Bitmap_Free should
be used to release that memory. Let's change free to Bitmap_Free.

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

orders: Fix OFFSCREEN_DELETE_LIST allocation size

The size of OFFSCREEN_DELETE_LIST list allocation was incorrectly changed
by commit 99b1481 and consequently fixed incorrectly by commit 8a0fe086.
Let's count the allocation size based on new size and not based on current
size in order to prevent some memory issues.

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

orders: Fix ORDER_TYPE_GDIPLUS_END check

Commit 6fd03ab introduced security checks for orders, but
ORDER_TYPE_GDIPLUS_END check fails in case of success and vice versa.
Let's add the missing question mark.

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

cache: Check return value from offscreen_cache_get

Currently, SetSurface is called regardless of return value from
offscreen_cache_get. It may happen that bitmap is not found in a
cache from some reason, which may lead to segfaults. Let's check
the returned value before use.

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

client/X11: Check arguments in xf_Bitmap_SetSurface

Arguments of xf_Bitmap_SetSurface are not checked before use currently
which may lead to segfaults. It may happen if bitmap is not found in a
cache from some reason. Let's check the arguments before use similarly
as it is done in client/Windows/wf_graphics.c in order to avoid segfaults.

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

Fixed BadValue with XCreateWindow

xf_FixWindowCoordinates occasionally set the dimensions of the window to invalid values (0) because the minimum value check was done at the beginning of the method rather than at the end

Merge pull request #4079 from akallabeth/channel_pubsub_fix

Dynamic channel signal disconnect after the fact.

Removed unused variable.

Unified dynamic channel cleanup code.

Merge pull request #4063 from akallabeth/auth_fixes

Fixed leaks, certificate comparison and channel context cleanup