Bugzilla – Bug 337
Rdesktop doesn't support the INCR protocol when transferring clipboard data.
Last modified: 2019-02-07 15:51:01
You need to
before you can comment on or make changes to this bug.
Rdesktop doesn't support the INCR protocol (defined in the ICCCM), which is
needed for really large clipboard transfers.
Complicated bug, retargeting.
One simple solution could be to pop up a dialog that says the the clip text is
AFAIK, we haven't heard anyone complaining on not being able to move large
clipboard data chunks. My suggestion is that we move this to 'Future/Ondemand'.
On the other hand, we'll probably have to fix this in order to be able to fix
Supporting the INCR protocol is not trivial, but it should be doable.
Basically, when a requestor wants clipboard data from rdesktop, rdesktop should
respond with a property of type INCR containg a number that represents the size
of the clipboard data. The requestor deletes the property which signals to
rdesktop (via a PropertyNotify) that the data transfer should start. Rdesktop
should then send data in suitable chunks to the property named by the requestor
in the initial ConvertSelection call and wait for new PropertyNotify signals. To
end data transfer, a zero-sized data chunk is written to the property on the
We're already listening for PropertyChange, so in theory, we should be able to
extend xclip_handle_PropertyNotify and make it send data with INCR. At least,
that's my initial idea.
The other case, receiving data via INCR from other X clients must also be covered.
Of course, synchronization with the clipboard mechanism of Windows must also
happend, either simply by buffering all data before sending, or more
complicated, by actually sending data as it comes in. One problem might be that
the package size for intermediate Windows clipboard data might be incompatible
with the size of the data received from other clients, so we'll probably have to
As a beginning for implementation of this, we need to find other X programs that
behave good so we have something to test against.
Later, now that bug 491 has been moved.
INCR support has been implemented by Burt Holzman. I've applied the patch to
rdesktop. A new snapshot has been imported to the ctc tree.
During the 1.4.1 testing, we should verify that INCR works.
I tried to cut'n paste a really large piece of text from OOo to winword on
Doesn't work, winword hangs with a hourglass for a very long time.
I'll reopen and retarget to 1.5.0, so we can bugreport and/or find a solution.
We should checkout
The link in comment#9 is about a patch where the author specifically says
"Didn't touch INCR", so it probably wont help :-).
Got a patch from Burt Holzman, which makes things work better, although not
perfect. The last lines of the area doesn't get transferred. I've reported this
back to Burt.
Created an attachment (id=149) [details]
Patch from Burt. Makes INCR work better, but not perfect.
Got another patch from Burt that makes earlier failing test case succeed. I've
commited this to rdesktop cvs.
There are still trouble when doing large INCR using native windows clipboard
formats. Burt said he would try to look into this in the near future.
Most of the work is done by people on the rdesktop list, we're only testing and
commiting their code.
Looks like we can send INCR, but not receive it. Need to finish this.
rdesktop (and associated tools) is being removed from the ThinLinc product.