Merge pull request #4198 from akallabeth/nla_error_fix

Added better error mapping for NEGO results.

Added better error mapping for NEGO results.

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

Merge pull request #4177 from rjcorrig/tabfix

Android: Fix tab key mapping for hardware keyboard

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.

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

Merge pull request #4104 from abma/master

Revert "Possible fix for #4081 and #2707 (#4087)" (Part of #4096)

Merge pull request #4050 from akallabeth/rdpefs_server_client_name_fix

Mask out ignored bits from UnicodeFlag.

fix clang warnings, directly include wtypes.h (#4097)

* build: clang use -Wno-unused-command-line-argument

With clang 5.0 builds are quite noisy otherwise.

* Directly include wtypes.h

Directly include winpr/wtypes.h where _fseeki64 or _ftelli64 is used.

* fix build warnings with clang 5

clang version: 5.0.0-svn310678-1~exp1 (branches/release_50)
Warning: parentheses-equality

* fix build warnings with clang 5

clang version: 5.0.0-svn310678-1~exp1 (branches/release_50)
Warning: tautological-compare

* fix build warnings with clang 5

clang version: 5.0.0-svn310678-1~exp1 (branches/release_50)
Warning:
incompatible pointer types passing 'size_t *' (aka 'unsigned
long *') to parameter of type 'UINT32 *' (aka 'unsigned int *')
[-Wincompatible-pointer-types]

Merge pull request #4100 from mmeyer724/wayland-opaque

wlfreerdp: improve performance by making the window opaque

Merge pull request #4099 from mmeyer724/wayland-fullscreen

wlfreerdp: mirror display resolution in fullscreen mode

Revert "Possible fix for #4081 and #2707 (#4087)"

This reverts most of commit 6b7b60237042a891221d71a596fc683d850bcb4e.

Part of #4096

wlfreerdp: improve performance by making the window opaque

wlfreerdp: mirror display resolution in fullscreen mode

Merge pull request #4098 from akallabeth/fix_rsa_generation

Fixed RSA generation for OpenSSL >= 1.1

Fixed RSA generation for OpenSSL >= 1.1

Merge pull request #4095 from mmeyer724/wayland-scrolling

Fix scrolling crash in wayland client

Fixes scrolling crash in wlfreerdp

An "axis discrete" handler is required since protocol version 5:
https://code.woboq.org/qt5/include/wayland-client-protocol.h.html
Crash output: "listener function for opcode 8 of wl_pointer is NULL"

Fixed data type, eliminate warning.

Added FindICU from CMake 3.7 for backward compatibility

android: fix a regression with remote program (#4091)

Commit 54a7d799d1e9dc7698425b7172a8bf81a4147168 introduced a problem
where /app was used instead of /shell causing a black screen when remote
programs were used. The Android client doesn't support remote apps
therefore /app can't work. "Remote program" just sets the selected
program as shell to run.

Merge pull request #4092 from akallabeth/afreerdp_no_client_name_escape

Do not add quotes to client hostname.

Do not add quotes to client hostname.

FindNextFileAx, continue if a inalid file is found.

Merge pull request #4088 from akallabeth/file_api_64bit_fixes

fseeko and ftello for 64bit file support.

Merge pull request #4085 from akallabeth/file_warn_fix

Fixed FileSetFilePointer warnings

Client/X11: Fix h264 context leak when DeleteSurface (#4089)

Possible fix for #4081 and #2707 (#4087)

* detect openbox automatically

* format code

* copyright update

* free prop at the end of the function

fseeko and ftello for 64bit file support.

Merge pull request #4084 from psumbera/master

On Solaris use standard conforming getpwnam_r().

Fixed FileSetFilePointer warnings

On Solaris use standard conforming getpwnam_r().

Merge pull request #4076 from akallabeth/SetFilePointer_fix

Set file pointer fix

Merge pull request #4070 from yurashek/master

Sun keyboard - cursor keys

Fixed file seek.

Fixed SetFilePointer, added SetFilePointerEx

Sun keyboard

Skip FIFO files in listings.