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"

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.

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

Merge pull request #4071 from bmiklautz/debian_fixes

Fixes from the Debian project

Fix warning in man pages

"warning: can't find macro file `www.tmac))'""`"

Fix typos in some error messages

Merge pull request #4066 from akallabeth/input_event_fixes

Fixed capability checks for mouse and unicode input.

Merge pull request #4069 from yurashek/master

Build on Solaris

Fixed capability checks for mouse and unicode input.

Merge pull request #4064 from floppym/gst-detect

Fix gstreamer-1.0 detection

Fix gstreamer-1.0 detection

Channels with a context must free it themselves.

Free credentials on exit.

Unexported internal NLA functions.

Fixed certificate check return.

Fixed uninitialized values and leaks.

Merge pull request #4061 from vakartel/master

Fix build with LibreSSL

Fix build with LibreSSL

Merge pull request #4056 from akallabeth/disable_jpeg

Disable JPEG codec support if not compiled in.

Disable JPEG codec support if not compiled in.

Merge pull request #4055 from akallabeth/talos_v3

Security fixes and code cleanups

Added return check for ConvertFromUnicode.

Fixed missing unicode conversion for mac drive redirection.

Ignore gdi_resize if not initialized.

clipboard: fix possible invalid memory access

Fix an possible issue found by Sébastien Duquette.

Fixed missing include.

Fixed resource cleanup.

Fixed uninitialized values.

Fixed double free.

Fixed wrong type for AuthenticationLevel.

Replaced type BOOL with INT32 as according to MSDN they are typedefed.
This keeps the ABI compatible and fixes the wrong data type.

Fixed use of reserved keywords for include guards.

Fixed clang malloc integer overflow warnings.

Fix TALOS issues

Fix the following issues identified by the CISCO TALOS project:
 * TALOS-2017-0336 CVE-2017-2834
 * TALOS-2017-0337 CVE-2017-2834
 * TALOS-2017-0338 CVE-2017-2836
 * TALOS-2017-0339 CVE-2017-2837
 * TALOS-2017-0340 CVE-2017-2838
 * TALOS-2017-0341 CVE-2017-2839

Merge pull request #3956 from dodo040/smartcard

redirect specific smartcard readers

fix format code

fix smartcard argument parsing

fix wrong declaration

redirect specific smartcard readers

Merge pull request #4044 from akallabeth/disable_h264_if_not_compiled

Disable h264 if no backend compiled in

Fixed variable assignment.

Merge pull request #4047 from akallabeth/win_gfx_fix

Always use software GFX for windows client.

Added warning on GFX channel load that it does not support acceleration.

Don't use an option for WITH_GFX_H264

CMAKE_DEPENDENT_OPTION the option is initialized on the first run and
then saved. Later configuration changes are not considered anymore.
Therefor use an variable instead of an option here.

Mask out ignored bits from UnicodeFlag.

Always use software GFX for windows client.

Reordered backends with regard to feature completeness.

Merge pull request #4041 from wayk/PathMakePathA

Fixed PathMakePathA (returned true even if it can't create the last f…

Readded H264 struct pointers, keep API compatible.

Add 10.3 capability check.

Fixed config option.

Merge pull request #4043 from rjcorrig/kb4020322

winpr: Updates time zone data to July 2017

Deactivated all H264 related code paths.

Disable WITH_GFX_H264 if no backend compiled in.

Removed H264 dummy, did not work (crash on use)