cedrozor [Fri, 27 Oct 2017 12:30:00 +0000 (14:30 +0200)]
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)
David Fort [Wed, 25 Oct 2017 22:17:03 +0000 (00:17 +0200)]
Merge pull request #4198 from akallabeth/nla_error_fix
Added better error mapping for NEGO results.
Armin Novak [Wed, 25 Oct 2017 07:56:39 +0000 (09:56 +0200)]
Added better error mapping for NEGO results.
David Fort [Thu, 19 Oct 2017 11:15:04 +0000 (13:15 +0200)]
Merge pull request #4195 from nfedera/fix-sse3-yuv444pixel
primitives/yuv: fix endianess issue
Norbert Federa [Thu, 19 Oct 2017 09:55:13 +0000 (11:55 +0200)]
primitives/yuv: fix endianess issue
Norbert Federa [Wed, 18 Oct 2017 08:24:35 +0000 (10:24 +0200)]
Merge pull request #4190 from akallabeth/media_foundation_fix_win7
Removed unused function pointer
Armin Novak [Wed, 18 Oct 2017 07:52:41 +0000 (09:52 +0200)]
Removed unused function pointer
The function was breaking windows 7 compatibility.
Norbert Federa [Tue, 17 Oct 2017 19:33:16 +0000 (21:33 +0200)]
Merge pull request #4189 from nfedera/fix-h264-init
codec/h264: fix potential segfault and mf_init
Norbert Federa [Tue, 17 Oct 2017 17:28:54 +0000 (19:28 +0200)]
codec/h264: fix potential segfault and mf_init
akallabeth [Mon, 16 Oct 2017 15:21:35 +0000 (17:21 +0200)]
Merge pull request #4177 from rjcorrig/tabfix
Android: Fix tab key mapping for hardware keyboard
Robert Corrigan [Thu, 12 Oct 2017 16:56:34 +0000 (12:56 -0400)]
Fix tab key mapping for hardware keyboard
Martin Fleisz [Thu, 12 Oct 2017 10:12:43 +0000 (12:12 +0200)]
Merge pull request #4175 from akallabeth/time_fix
Fixed file timestamps.
Martin Fleisz [Thu, 12 Oct 2017 10:12:08 +0000 (12:12 +0200)]
Merge pull request #4172 from hardening/egfx_cache
egfx: fix disconnection caused by invalid cache entries due to wrong announced cache size
Armin Novak [Thu, 12 Oct 2017 09:27:32 +0000 (11:27 +0200)]
Fixed file timestamps.
David Fort [Tue, 10 Oct 2017 15:12:16 +0000 (17:12 +0200)]
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.
Martin Fleisz [Mon, 9 Oct 2017 11:25:41 +0000 (13:25 +0200)]
Merge pull request #4164 from akallabeth/ak/mac_audio_refactor
Pause playback on queue empty, free buffers on close.
akallabeth [Mon, 9 Oct 2017 07:19:19 +0000 (09:19 +0200)]
Merge pull request #4167 from bmiklautz/buildversion
feat: add support for .source_version
Bernhard Miklautz [Fri, 6 Oct 2017 13:02:46 +0000 (15:02 +0200)]
feat freerdp-nighlty/deb: create .source_version
Bernhard Miklautz [Fri, 6 Oct 2017 11:52:10 +0000 (13:52 +0200)]
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.
Bernhard Miklautz [Thu, 5 Oct 2017 13:13:30 +0000 (15:13 +0200)]
Merge pull request #4162 from hardening/misc_fix2
Misc fix and improvements for the DR channel
Armin Novak [Thu, 5 Oct 2017 10:05:09 +0000 (12:05 +0200)]
Pause playback on queue empty, free buffers on close.
David Fort [Mon, 2 Oct 2017 19:28:02 +0000 (21:28 +0200)]
Add checks for DR channel
David Fort [Mon, 2 Oct 2017 18:57:27 +0000 (20:57 +0200)]
Fix for #4161
The checks were wrong, fix them.
akallabeth [Wed, 27 Sep 2017 12:56:21 +0000 (14:56 +0200)]
Merge pull request #4154 from hardening/misc_fixes
Fix raw surfaces displaying + misc other changes
akallabeth [Wed, 27 Sep 2017 09:14:39 +0000 (11:14 +0200)]
Merge pull request #4144 from bmiklautz/clipboardon
Enable clipboard channel per default
Bernhard Miklautz [Sat, 23 Sep 2017 13:16:17 +0000 (15:16 +0200)]
Enable clipboard channel per default
David Fort [Tue, 26 Sep 2017 07:21:28 +0000 (09:21 +0200)]
Fix displaying of raw surfaces
David Fort [Tue, 26 Sep 2017 07:20:38 +0000 (09:20 +0200)]
Check return value of malloc
David Fort [Tue, 26 Sep 2017 11:52:37 +0000 (13:52 +0200)]
Merge pull request #4156 from akallabeth/test_connect_extend_timeout
Added delay for connect abort
Armin Novak [Tue, 26 Sep 2017 10:03:57 +0000 (12:03 +0200)]
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.
David Fort [Tue, 26 Sep 2017 09:26:35 +0000 (11:26 +0200)]
Merge pull request #4155 from akallabeth/test_connect_extend_timeout
TestConnect: Extend timeout, only listen locally
Armin Novak [Tue, 26 Sep 2017 08:52:13 +0000 (10:52 +0200)]
TestConnect: Extend timeout, only listen locally
David Fort [Tue, 26 Sep 2017 07:40:30 +0000 (09:40 +0200)]
Merge pull request #4152 from akallabeth/dynamic_channel_test_fix
Dynamic channel test fix
Armin Novak [Mon, 25 Sep 2017 11:30:05 +0000 (13:30 +0200)]
Fixed TestConnect with dynamic channels.
Armin Novak [Mon, 25 Sep 2017 11:29:23 +0000 (13:29 +0200)]
Fixed client context setup with default initializer.
Armin Novak [Mon, 25 Sep 2017 11:28:16 +0000 (13:28 +0200)]
Fixed resizing of PubSub
David Fort [Mon, 25 Sep 2017 11:17:32 +0000 (13:17 +0200)]
Merge pull request #4150 from akallabeth/server_file_handle_leak
Unlink file after binding to it.
Armin Novak [Mon, 25 Sep 2017 08:35:24 +0000 (10:35 +0200)]
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.
Bernhard Miklautz [Mon, 25 Sep 2017 07:35:49 +0000 (09:35 +0200)]
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.
David Fort [Mon, 25 Sep 2017 07:33:58 +0000 (09:33 +0200)]
Merge pull request #4060 from akallabeth/icu_support
Using ICU instead of custom unicode conversion.
David Fort [Mon, 25 Sep 2017 07:12:35 +0000 (09:12 +0200)]
Merge pull request #4142 from bmiklautz/macOS
Fix build on macOS
David Fort [Mon, 25 Sep 2017 07:09:55 +0000 (09:09 +0200)]
Merge pull request #4143 from bmiklautz/pass_is_sc_pin
Add support for "password is pin"
Jukka-Pekka Virtanen [Sat, 23 Sep 2017 12:26:38 +0000 (14:26 +0200)]
Expose PasswordIsSmartcardPin settings from cmdline via password-is-pin arg
Jukka-Pekka Virtanen [Thu, 14 Sep 2017 10:34:09 +0000 (13:34 +0300)]
Using PasswordIsSmartcardPin option when sending TS_INFO_PACKET
Bernhard Miklautz [Sat, 23 Sep 2017 12:08:05 +0000 (14:08 +0200)]
macOS: add a README file
Add a README file containing some notes about building FreeRDP on macOS
related to openssl.
Bernhard Miklautz [Sat, 23 Sep 2017 11:57:57 +0000 (13:57 +0200)]
winpr/makecert: fix linking against libcrypto
Use OPENSSL_CRYPTO_LIBRARIES instead of linking against crypto directly.
Bernhard Miklautz [Sat, 23 Sep 2017 11:31:34 +0000 (13:31 +0200)]
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
Bernhard Miklautz [Fri, 22 Sep 2017 15:22:00 +0000 (17:22 +0200)]
Merge pull request #4045 from akallabeth/drive_fixes_overlayfs
Drive fixes overlayfs
David Fort [Fri, 22 Sep 2017 07:52:27 +0000 (09:52 +0200)]
Merge pull request #4136 from tditsch/master
Fixed endless loop when RDP Server sends SERVER_DENIED_CONNECTION
Bernhard Miklautz [Thu, 21 Sep 2017 12:50:34 +0000 (14:50 +0200)]
Merge pull request #4116 from ChickenPeanut/patch-1
Fixed BadValue with XCreateWindow
Martin Fleisz [Tue, 19 Sep 2017 11:51:09 +0000 (13:51 +0200)]
Merge pull request #4138 from akallabeth/ak/nla_free_fix
Check and invalidate handles on free.
Armin Novak [Tue, 19 Sep 2017 10:33:33 +0000 (12:33 +0200)]
Check and invalidate handles on free.
tditsch [Tue, 19 Sep 2017 08:18:41 +0000 (10:18 +0200)]
refactored Bugfix
tditsch [Mon, 18 Sep 2017 15:29:16 +0000 (17:29 +0200)]
Fixed endless loop when RDP Server sends SERVER_DENIED_CONNECTION
Ondrej Holy [Mon, 18 Sep 2017 08:47:56 +0000 (10:47 +0200)]
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.
David Fort [Mon, 18 Sep 2017 08:17:43 +0000 (10:17 +0200)]
Merge pull request #4121 from ondrejholy/setsurface-segfault-fix
Various fixes from #4117
David Fort [Tue, 12 Sep 2017 21:16:39 +0000 (23:16 +0200)]
Merge pull request #4052 from akallabeth/mac_fixes
Mac drive fixes
MartinHaimberger [Mon, 11 Sep 2017 14:34:24 +0000 (16:34 +0200)]
Merge pull request #4127 from mfleisz/avc420
channels: Don't send gfx capversion 10 if AVC420 is requested
Martin Fleisz [Mon, 11 Sep 2017 10:05:03 +0000 (12:05 +0200)]
channels: Don't send gfx capversion 10 if AVC420 is requested
Ondrej Holy [Tue, 5 Sep 2017 08:25:15 +0000 (10:25 +0200)]
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
Ondrej Holy [Tue, 5 Sep 2017 07:56:00 +0000 (09:56 +0200)]
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
Ondrej Holy [Tue, 5 Sep 2017 06:58:37 +0000 (08:58 +0200)]
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
Ondrej Holy [Mon, 4 Sep 2017 16:48:25 +0000 (18:48 +0200)]
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
Ondrej Holy [Mon, 4 Sep 2017 16:22:49 +0000 (18:22 +0200)]
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
ChickenPeanut [Sun, 3 Sep 2017 22:52:34 +0000 (18:52 -0400)]
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
David Fort [Thu, 31 Aug 2017 08:22:43 +0000 (10:22 +0200)]
Merge pull request #4079 from akallabeth/channel_pubsub_fix
Dynamic channel signal disconnect after the fact.
Armin Novak [Thu, 31 Aug 2017 07:57:49 +0000 (09:57 +0200)]
Removed unused variable.
Armin Novak [Thu, 31 Aug 2017 07:06:59 +0000 (09:06 +0200)]
Unified dynamic channel cleanup code.
David Fort [Wed, 30 Aug 2017 08:19:12 +0000 (10:19 +0200)]
Merge pull request #4063 from akallabeth/auth_fixes
Fixed leaks, certificate comparison and channel context cleanup
David Fort [Tue, 29 Aug 2017 09:06:13 +0000 (11:06 +0200)]
Merge pull request #4104 from abma/master
Revert "Possible fix for #4081 and #2707 (#4087)" (Part of #4096)
Bernhard Miklautz [Tue, 29 Aug 2017 08:35:45 +0000 (10:35 +0200)]
Merge pull request #4050 from akallabeth/rdpefs_server_client_name_fix
Mask out ignored bits from UnicodeFlag.
Bernhard Miklautz [Tue, 29 Aug 2017 07:09:38 +0000 (09:09 +0200)]
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]
David Fort [Sat, 26 Aug 2017 08:17:02 +0000 (10:17 +0200)]
Merge pull request #4100 from mmeyer724/wayland-opaque
wlfreerdp: improve performance by making the window opaque
David Fort [Sat, 26 Aug 2017 08:13:42 +0000 (10:13 +0200)]
Merge pull request #4099 from mmeyer724/wayland-fullscreen
wlfreerdp: mirror display resolution in fullscreen mode
abma [Tue, 22 Aug 2017 20:42:58 +0000 (22:42 +0200)]
Revert "Possible fix for #4081 and #2707 (#4087)"
This reverts most of commit
6b7b60237042a891221d71a596fc683d850bcb4e.
Part of #4096
Michael Meyer [Thu, 17 Aug 2017 21:32:26 +0000 (17:32 -0400)]
wlfreerdp: improve performance by making the window opaque
Michael Meyer [Thu, 17 Aug 2017 17:33:44 +0000 (13:33 -0400)]
wlfreerdp: mirror display resolution in fullscreen mode
Bernhard Miklautz [Thu, 17 Aug 2017 13:18:04 +0000 (15:18 +0200)]
Merge pull request #4098 from akallabeth/fix_rsa_generation
Fixed RSA generation for OpenSSL >= 1.1
Armin Novak [Thu, 17 Aug 2017 11:16:32 +0000 (13:16 +0200)]
Fixed RSA generation for OpenSSL >= 1.1
Bernhard Miklautz [Thu, 17 Aug 2017 08:46:16 +0000 (10:46 +0200)]
Merge pull request #4095 from mmeyer724/wayland-scrolling
Fix scrolling crash in wayland client
Michael Meyer [Wed, 16 Aug 2017 17:54:45 +0000 (17:54 +0000)]
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"
Armin Novak [Wed, 16 Aug 2017 13:40:45 +0000 (15:40 +0200)]
Fixed data type, eliminate warning.
Armin Novak [Wed, 16 Aug 2017 13:09:33 +0000 (15:09 +0200)]
Added FindICU from CMake 3.7 for backward compatibility
Bernhard Miklautz [Wed, 16 Aug 2017 11:28:11 +0000 (13:28 +0200)]
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.
Bernhard Miklautz [Wed, 16 Aug 2017 11:26:47 +0000 (13:26 +0200)]
Merge pull request #4092 from akallabeth/afreerdp_no_client_name_escape
Do not add quotes to client hostname.
Armin Novak [Wed, 16 Aug 2017 09:31:33 +0000 (11:31 +0200)]
Do not add quotes to client hostname.
Armin Novak [Wed, 16 Aug 2017 08:25:23 +0000 (10:25 +0200)]
FindNextFileAx, continue if a inalid file is found.
Bernhard Miklautz [Wed, 16 Aug 2017 08:04:13 +0000 (10:04 +0200)]
Merge pull request #4088 from akallabeth/file_api_64bit_fixes
fseeko and ftello for 64bit file support.
Bernhard Miklautz [Wed, 16 Aug 2017 06:09:31 +0000 (08:09 +0200)]
Merge pull request #4085 from akallabeth/file_warn_fix
Fixed FileSetFilePointer warnings
Jiang, Zihao [Mon, 14 Aug 2017 08:16:14 +0000 (16:16 +0800)]
Client/X11: Fix h264 context leak when DeleteSurface (#4089)
Kai [Mon, 14 Aug 2017 08:10:53 +0000 (10:10 +0200)]
Possible fix for #4081 and #2707 (#4087)
* detect openbox automatically
* format code
* copyright update
* free prop at the end of the function
Armin Novak [Fri, 11 Aug 2017 08:07:46 +0000 (10:07 +0200)]
fseeko and ftello for 64bit file support.
Bernhard Miklautz [Thu, 10 Aug 2017 15:39:34 +0000 (17:39 +0200)]
Merge pull request #4084 from psumbera/master
On Solaris use standard conforming getpwnam_r().
Armin Novak [Thu, 10 Aug 2017 14:51:55 +0000 (16:51 +0200)]
Fixed FileSetFilePointer warnings
Petr Sumbera [Thu, 10 Aug 2017 14:36:25 +0000 (07:36 -0700)]
On Solaris use standard conforming getpwnam_r().
MartinHaimberger [Wed, 9 Aug 2017 08:35:37 +0000 (10:35 +0200)]
Merge pull request #4076 from akallabeth/SetFilePointer_fix
Set file pointer fix
Bernhard Miklautz [Tue, 8 Aug 2017 19:17:01 +0000 (21:17 +0200)]
Merge pull request #4070 from yurashek/master
Sun keyboard - cursor keys
Armin Novak [Tue, 8 Aug 2017 08:52:11 +0000 (10:52 +0200)]
Fixed file seek.
Armin Novak [Tue, 8 Aug 2017 08:51:50 +0000 (10:51 +0200)]
Fixed SetFilePointer, added SetFilePointerEx
Jura Sasek [Wed, 2 Aug 2017 11:52:45 +0000 (04:52 -0700)]
Sun keyboard