Skip to content

fix: Restore clipboard sharing after 53441821e - #9307

Merged
nbolton merged 1 commit into
deskflow:masterfrom
chrschmidt:master
Dec 30, 2025
Merged

fix: Restore clipboard sharing after 53441821e#9307
nbolton merged 1 commit into
deskflow:masterfrom
chrschmidt:master

Conversation

@chrschmidt

@chrschmidt chrschmidt commented Dec 29, 2025

Copy link
Copy Markdown
Contributor

We wanted the value contained in the string as size_t, not the size (length) of the string. size_t is (normally) an alias for unsigned long, so toUlong() should do the trick.

This broke at least with X11 <-> X11, where it caused messages like
[2025-12-29T20:15:28.221] ERROR: corrupted clipboard data, expected size=3 actual size=378
[2025-12-29T20:15:28.221] ERROR: corrupted clipboard data, expected size=2 actual size=52

Edit:
fixes: #9279

@chrschmidt
chrschmidt requested a review from a team December 29, 2025 19:54

@sithlord48 sithlord48 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seams reasonable

Added note that this fixes: #9279

@sithlord48 sithlord48 added the 🩹 fix This PR fixes a reported bug label Dec 30, 2025
We wanted the value contained in the string as size_t, not the size (length) of
the string. size_t is (normally) an alias for unsigned long, so toUlong() should
do the trick.
@nbolton
nbolton enabled auto-merge (rebase) December 30, 2025 13:00
@nbolton
nbolton merged commit 2dd0360 into deskflow:master Dec 30, 2025
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🩹 fix This PR fixes a reported bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"Corrupted clipboard data" copying from server to client

3 participants