forked from pool/freerdp
Accepting request 503011 from home:bruno_friedmann
- Redo history by using changelog url
- Update to version 2.0.0~git.1497095982.1dbd2d28d:
See 043243f...1dbd2d28d
for full details.
- Packaging :
+ Move to xz tarball in _service
+ Remove 3472.patch merged upstream
OBS-URL: https://build.opensuse.org/request/show/503011
OBS-URL: https://build.opensuse.org/package/show/X11:RemoteDesktop/freerdp?expand=0&rev=60
This commit is contained in:
Tomáš Chvátal
Git OBS Bridge
parent
8792fa1bbb
commit
b1206c0a69
179
3472.patch
179
3472.patch
@ -1,179 +0,0 @@
|
||||
From 198a5e76dc7d2938193574fcf5b3af61e5766b0a Mon Sep 17 00:00:00 2001
|
||||
From: ilammy <a.lozovsky@gmail.com>
|
||||
Date: Mon, 15 Aug 2016 22:45:20 +0300
|
||||
Subject: [PATCH] client/X11: cache original clipboard data for raw transfers
|
||||
|
||||
FreeRDP uses clipboard->data to cache the result of the Windows->X11
|
||||
clipboard format conversion, and xf_cliprdr_process_selection_request()
|
||||
immediately provides this result to local applications if they request
|
||||
the same clipboard format again. This saves us a possibly costly
|
||||
conversion in case where the user pastes data repeatedly.
|
||||
|
||||
However, this caching mechanism did not support raw clipboard transfers
|
||||
where the unmodified data is passed between two FreeRDP clients. We use
|
||||
the same XClipboard protocol for this, so the clipboard->data is in play.
|
||||
We clear the cached value when we receive new data from the server, so
|
||||
initially raw transfers are fine. But if some local application (e.g.,
|
||||
a clipboard manager) asks for some data format before the data is pasted
|
||||
into the second FreeRDP session then clipboard->data will contain the
|
||||
*converted* data. And this converted cached data will be provided to
|
||||
the second FreeRDP session as a part of the raw data transfer. Instead
|
||||
we should have provided the original data.
|
||||
|
||||
In order to achieve this we are now caching the original data in the
|
||||
same way as the converted one, and the original data is now correctly
|
||||
provided when the second FreeRDP session asks for a raw data transfer.
|
||||
---
|
||||
client/X11/xf_cliprdr.c | 72 ++++++++++++++++++++++++++++++++++++-------------
|
||||
1 file changed, 54 insertions(+), 18 deletions(-)
|
||||
|
||||
diff --git a/client/X11/xf_cliprdr.c b/client/X11/xf_cliprdr.c
|
||||
index 80b54c5..0d4b36c 100644
|
||||
--- a/client/X11/xf_cliprdr.c
|
||||
+++ b/client/X11/xf_cliprdr.c
|
||||
@@ -81,10 +81,12 @@ struct xf_clipboard
|
||||
int requestedFormatId;
|
||||
|
||||
BYTE* data;
|
||||
+ BYTE* data_raw;
|
||||
BOOL data_raw_format;
|
||||
UINT32 data_format_id;
|
||||
const char* data_format_name;
|
||||
int data_length;
|
||||
+ int data_raw_length;
|
||||
XEvent* respond;
|
||||
|
||||
Window owner;
|
||||
@@ -774,6 +776,23 @@ static BOOL xf_cliprdr_process_selection_notify(xfClipboard* clipboard,
|
||||
}
|
||||
}
|
||||
|
||||
+static void xf_cliprdr_clear_cached_data(xfClipboard* clipboard)
|
||||
+{
|
||||
+ if (clipboard->data)
|
||||
+ {
|
||||
+ free(clipboard->data);
|
||||
+ clipboard->data = NULL;
|
||||
+ }
|
||||
+ clipboard->data_length = 0;
|
||||
+
|
||||
+ if (clipboard->data_raw)
|
||||
+ {
|
||||
+ free(clipboard->data_raw);
|
||||
+ clipboard->data_raw = NULL;
|
||||
+ }
|
||||
+ clipboard->data_raw_length = 0;
|
||||
+}
|
||||
+
|
||||
static BOOL xf_cliprdr_process_selection_request(xfClipboard* clipboard,
|
||||
XEvent* xevent)
|
||||
{
|
||||
@@ -785,6 +804,7 @@ static BOOL xf_cliprdr_process_selection_request(xfClipboard* clipboard,
|
||||
BYTE* data = NULL;
|
||||
BOOL delayRespond;
|
||||
BOOL rawTransfer;
|
||||
+ BOOL matchingFormat;
|
||||
unsigned long length;
|
||||
unsigned long bytes_left;
|
||||
CLIPRDR_FORMAT* format;
|
||||
@@ -847,14 +867,24 @@ static BOOL xf_cliprdr_process_selection_request(xfClipboard* clipboard,
|
||||
}
|
||||
}
|
||||
|
||||
- if ((clipboard->data != 0) && (formatId == clipboard->data_format_id)
|
||||
- && (formatName == clipboard->data_format_name))
|
||||
+ /* We can compare format names by pointer value here as they are both
|
||||
+ * taken from the same clipboard->serverFormats array */
|
||||
+ matchingFormat = (formatId == clipboard->data_format_id)
|
||||
+ && (formatName == clipboard->data_format_name);
|
||||
+
|
||||
+ if (matchingFormat && (clipboard->data != 0) && !rawTransfer)
|
||||
{
|
||||
- /* Cached clipboard data available. Send it now */
|
||||
+ /* Cached converted clipboard data available. Send it now */
|
||||
respond->xselection.property = xevent->xselectionrequest.property;
|
||||
xf_cliprdr_provide_data(clipboard, respond, clipboard->data,
|
||||
clipboard->data_length);
|
||||
}
|
||||
+ else if (matchingFormat && (clipboard->data_raw != 0) && rawTransfer)
|
||||
+ {
|
||||
+ /* Cached raw clipboard data available. Send it now */
|
||||
+ respond->xselection.property = xevent->xselectionrequest.property;
|
||||
+ xf_cliprdr_provide_data(clipboard, respond, clipboard->data_raw, clipboard->data_raw_length);
|
||||
+ }
|
||||
else if (clipboard->respond)
|
||||
{
|
||||
/* duplicate request */
|
||||
@@ -865,11 +895,7 @@ static BOOL xf_cliprdr_process_selection_request(xfClipboard* clipboard,
|
||||
* Send clipboard data request to the server.
|
||||
* Response will be postponed after receiving the data
|
||||
*/
|
||||
- if (clipboard->data)
|
||||
- {
|
||||
- free(clipboard->data);
|
||||
- clipboard->data = NULL;
|
||||
- }
|
||||
+ xf_cliprdr_clear_cached_data(clipboard);
|
||||
|
||||
respond->xselection.property = xevent->xselectionrequest.property;
|
||||
clipboard->respond = respond;
|
||||
@@ -1126,11 +1152,7 @@ static UINT xf_cliprdr_server_format_list(CliprdrClientContext* context,
|
||||
xfContext* xfc = clipboard->xfc;
|
||||
UINT ret;
|
||||
|
||||
- if (clipboard->data)
|
||||
- {
|
||||
- free(clipboard->data);
|
||||
- clipboard->data = NULL;
|
||||
- }
|
||||
+ xf_cliprdr_clear_cached_data(clipboard);
|
||||
|
||||
clipboard->data_format_id = -1;
|
||||
clipboard->data_format_name = NULL;
|
||||
@@ -1274,11 +1296,7 @@ static UINT xf_cliprdr_server_format_data_response(CliprdrClientContext*
|
||||
if (!clipboard->respond)
|
||||
return CHANNEL_RC_OK;
|
||||
|
||||
- if (clipboard->data)
|
||||
- {
|
||||
- free(clipboard->data);
|
||||
- clipboard->data = NULL;
|
||||
- }
|
||||
+ xf_cliprdr_clear_cached_data(clipboard);
|
||||
|
||||
pDstData = NULL;
|
||||
DstSize = 0;
|
||||
@@ -1343,8 +1361,25 @@ static UINT xf_cliprdr_server_format_data_response(CliprdrClientContext*
|
||||
}
|
||||
}
|
||||
|
||||
+ /* Cache converted and original data to avoid doing a possibly costly
|
||||
+ * conversion again on subsequent requests */
|
||||
clipboard->data = pDstData;
|
||||
clipboard->data_length = DstSize;
|
||||
+
|
||||
+ /* We have to copy the original data again, as pSrcData is now owned
|
||||
+ * by clipboard->system. Memory allocation failure is not fatal here
|
||||
+ * as this is only a cached value. */
|
||||
+ clipboard->data_raw = (BYTE*) malloc(size);
|
||||
+ if (clipboard->data_raw)
|
||||
+ {
|
||||
+ CopyMemory(clipboard->data_raw, data, size);
|
||||
+ clipboard->data_raw_length = size;
|
||||
+ }
|
||||
+ else
|
||||
+ {
|
||||
+ WLog_WARN(TAG, "failed to allocate %zu bytes for a copy of raw clipboard data", size);
|
||||
+ }
|
||||
+
|
||||
xf_cliprdr_provide_data(clipboard, clipboard->respond, pDstData, DstSize);
|
||||
XSendEvent(xfc->display, clipboard->respond->xselection.requestor, 0, 0,
|
||||
clipboard->respond);
|
||||
@@ -1489,6 +1524,7 @@ void xf_clipboard_free(xfClipboard* clipboard)
|
||||
|
||||
ClipboardDestroy(clipboard->system);
|
||||
free(clipboard->data);
|
||||
+ free(clipboard->data_raw);
|
||||
free(clipboard->respond);
|
||||
free(clipboard->incr_data);
|
||||
free(clipboard);
|
||||
2
_service
2
_service
@ -9,7 +9,7 @@
|
||||
</service>
|
||||
<service name="recompress" mode="disabled">
|
||||
<param name="file">*.tar</param>
|
||||
<param name="compression">gz</param>
|
||||
<param name="compression">xz</param>
|
||||
</service>
|
||||
<service name="set_version" mode="disabled"/>
|
||||
</services>
|
||||
|
||||
@ -1,6 +1,6 @@
|
||||
<servicedata>
|
||||
<service name="tar_scm">
|
||||
<param name="url">https://github.com/FreeRDP/FreeRDP.git</param>
|
||||
<param name="changesrevision">043243f43c4dc889148a39c34eb24d2531c770bd</param>
|
||||
<param name="changesrevision">1dbd2d28d94f0ac21b3b000eaaa8fa9d1fb638e4</param>
|
||||
</service>
|
||||
</servicedata>
|
||||
</servicedata>
|
||||
@ -1,3 +0,0 @@
|
||||
version https://git-lfs.github.com/spec/v1
|
||||
oid sha256:ee0119b7b5cbbef2bfc00d760b84d5d478ff5e0012622d0e964b868cf9109b5c
|
||||
size 6487981
|
||||
3
freerdp-2.0.0~git.1497095982.1dbd2d28d.tar.xz
Normal file
3
freerdp-2.0.0~git.1497095982.1dbd2d28d.tar.xz
Normal file
@ -0,0 +1,3 @@
|
||||
version https://git-lfs.github.com/spec/v1
|
||||
oid sha256:47d16a25c8451bc9069fe3c223119f4f4517e3dfe8a95a10ca7750036d214f8a
|
||||
size 4298988
|
||||
@ -1,3 +1,17 @@
|
||||
-------------------------------------------------------------------
|
||||
Sun Jun 11 16:46:11 UTC 2017 - bruno@ioda-net.ch
|
||||
|
||||
- Redo history by using changelog url
|
||||
- Update to version 2.0.0~git.1497095982.1dbd2d28d:
|
||||
See https://github.com/freerdp/freeRDP/compare/043243f...1dbd2d28d
|
||||
for full details.
|
||||
- Packaging :
|
||||
+ Move to xz tarball in _service
|
||||
+ Remove 3472.patch merged upstream
|
||||
- Update to version 2.0.0~git.1496304402.c42ccb508:
|
||||
See https://github.com/freerdp/freeRDP/compare/043243f...c42ccb508
|
||||
for changes.
|
||||
|
||||
-------------------------------------------------------------------
|
||||
Sat Jun 10 12:27:53 UTC 2017 - meissner@suse.com
|
||||
|
||||
|
||||
28
freerdp.spec
28
freerdp.spec
@ -19,16 +19,13 @@
|
||||
%define sover 2
|
||||
|
||||
Name: freerdp
|
||||
Version: 2.0.0~git.1477034991.043243f
|
||||
Version: 2.0.0~git.1497095982.1dbd2d28d
|
||||
Release: 0
|
||||
Summary: Remote Desktop Viewer Client
|
||||
License: Apache-2.0
|
||||
Group: Productivity/Networking/Other
|
||||
Url: http://www.freerdp.com/
|
||||
Source0: %{name}-%{version}.tar.gz
|
||||
# Upstream issue #3472 - boo#1004108
|
||||
# Fix copy/paste encoding between windows host
|
||||
Patch0: https://patch-diff.githubusercontent.com/raw/FreeRDP/FreeRDP/pull/3472.patch
|
||||
Source0: %{name}-%{version}.tar.xz
|
||||
BuildRequires: alsa-devel
|
||||
BuildRequires: cmake
|
||||
BuildRequires: cups-devel
|
||||
@ -75,7 +72,6 @@ based on libfreerdp.
|
||||
|
||||
%prep
|
||||
%setup -q -n %{name}-%{version}
|
||||
%patch0 -p1
|
||||
|
||||
%build
|
||||
find . -type f -name "*.c" -exec perl -i -pe 's{__(DATE|TIME)__}{""}g' "{}" "+"
|
||||
@ -113,15 +109,17 @@ find %{buildroot} -name "*.a" -delete
|
||||
%{_bindir}/winpr-makecert
|
||||
%{_bindir}/xfreerdp
|
||||
%{_mandir}/man1/xfreerdp.1%{ext_man}
|
||||
%{_mandir}/man1/wlog.1%{?ext_man}
|
||||
%{_mandir}/man1/winpr-hash.1%{ext_man}
|
||||
%{_mandir}/man1/winpr-makecert.1%{ext_man}
|
||||
%{_mandir}/man7/wlog.7%{?ext_man}
|
||||
|
||||
%files -n %{libname}
|
||||
%defattr(-,root,root)
|
||||
%doc LICENSE
|
||||
%{_libdir}/libfreerdp-client.so.*
|
||||
%{_libdir}/libfreerdp.so.*
|
||||
%{_libdir}/libwinpr.so.*
|
||||
%{_libdir}/libwinpr-tools.so.*
|
||||
%{_libdir}/libfreerdp-client2.so.*
|
||||
%{_libdir}/libfreerdp2.so.*
|
||||
%{_libdir}/libwinpr2.so.*
|
||||
%{_libdir}/libwinpr-tools2.so.*
|
||||
|
||||
%files devel
|
||||
%defattr(-,root,root)
|
||||
@ -131,10 +129,10 @@ find %{buildroot} -name "*.a" -delete
|
||||
%{_libdir}/cmake/FreeRDP2
|
||||
%{_libdir}/cmake/FreeRDP-Client2
|
||||
%{_libdir}/cmake/WinPR2
|
||||
%{_libdir}/libfreerdp-client.so
|
||||
%{_libdir}/libfreerdp.so
|
||||
%{_libdir}/libwinpr.so
|
||||
%{_libdir}/libwinpr-tools.so
|
||||
%{_libdir}/libfreerdp-client2.so
|
||||
%{_libdir}/libfreerdp2.so
|
||||
%{_libdir}/libwinpr2.so
|
||||
%{_libdir}/libwinpr-tools2.so
|
||||
%{_libdir}/pkgconfig/freerdp2.pc
|
||||
%{_libdir}/pkgconfig/freerdp-client2.pc
|
||||
%{_libdir}/pkgconfig/winpr2.pc
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user