aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTobias C. Berner <tcberner@FreeBSD.org>2021-08-14 08:32:46 +0000
committerTobias C. Berner <tcberner@FreeBSD.org>2021-08-14 08:37:43 +0000
commit43f4cf91623eaecea287fdc4be9561d833f3dd7f (patch)
treeae9457fafe610d667140d285f9414072b84ca978
parent598cb706852900edb91fb7752a0d5ffc9d92eb93 (diff)
downloadports-43f4cf91623eaecea287fdc4be9561d833f3dd7f.tar.gz
ports-43f4cf91623eaecea287fdc4be9561d833f3dd7f.zip
x11-toolkits/qt5-gui: add upstream fix to clipboard
Bug description from [2]: QXcbConnection::getTimestamp() returns a timestamp from an earlier PropertyNotify event which was already in the event queue. I found this issue when I was trying to figure out why gvim (with GTK+) exits with a BadWindow error when selecting or copying a large text to the clipboard in a KDE environment. It turns out that GTK+ uses the INCR protocol to send the data and QT uses getTimestamp (in QXcbClipboard::clipboardReadProperty) to set the start time of the transfer. Since that start time is incorrect QT expects data which hasn't been sent yet and closes the window. GTK+ still tries to add the data to the window which causes a BadWindow error. From [1]: xcb: add a timeout control when reading INCR property For the first call of QXcbClipboard::clipboardReadProperty() inside of clipboardReadIncrementalProperty() in getSelection(), it will get a XCB_NONE reply before the contents arrived via property change. Then we give a chance to read more. Upstream Code Review [1]: https://codereview.qt-project.org/c/qt/qtbase/+/364040 Upstream Bug Report [2]: https://bugreports.qt.io/browse/QTBUG-56595
-rw-r--r--x11-toolkits/qt5-gui/Makefile2
-rw-r--r--x11-toolkits/qt5-gui/files/patch-git_73fc1f93-add_timeout_control_when_reading_incr_property59
2 files changed, 60 insertions, 1 deletions
diff --git a/x11-toolkits/qt5-gui/Makefile b/x11-toolkits/qt5-gui/Makefile
index a49eadb62a48..bc9525794292 100644
--- a/x11-toolkits/qt5-gui/Makefile
+++ b/x11-toolkits/qt5-gui/Makefile
@@ -1,6 +1,6 @@
PORTNAME= gui
DISTVERSION= ${QT5_VERSION}
-PORTREVISION= 5
+PORTREVISION= 6
CATEGORIES= x11-toolkits graphics
PKGNAMEPREFIX= qt5-
diff --git a/x11-toolkits/qt5-gui/files/patch-git_73fc1f93-add_timeout_control_when_reading_incr_property b/x11-toolkits/qt5-gui/files/patch-git_73fc1f93-add_timeout_control_when_reading_incr_property
new file mode 100644
index 000000000000..6f6dec7d35ec
--- /dev/null
+++ b/x11-toolkits/qt5-gui/files/patch-git_73fc1f93-add_timeout_control_when_reading_incr_property
@@ -0,0 +1,59 @@
+From 73fc1f93e8bea1c493ed16655ad6fd68ae270e38 Mon Sep 17 00:00:00 2001
+From: Liang Qi <liang.qi@qt.io>
+Date: Wed, 7 Jul 2021 13:19:14 +0200
+Subject: [PATCH] xcb: add a timeout control when reading INCR property
+MIME-Version: 1.0
+Content-Type: text/plain; charset=utf8
+Content-Transfer-Encoding: 8bit
+
+For the first call of QXcbClipboard::clipboardReadProperty()
+inside of clipboardReadIncrementalProperty() in getSelection(),
+it will get a XCB_NONE reply before the contents arrived via
+property change. Then we give a chance to read more.
+
+Manually tested with following setups:
+* examples/widgets/mainwindows/application with gvim(gtk3)
+* examples/widgets/widgets/imageviewer with GIMP 2.10.18(based on
+gtk2) and GIMP 2.99.6(based on gtk3 via flatpak)
+
+Fixes: QTBUG-56595
+Done-With: JiDe Zhang <zhangjide@uniontech.com>
+Change-Id: Ib45f08464d39ad79137b1da99808c89b7dca2d08
+Reviewed-by: JiDe Zhang <zhangjide@uniontech.com>
+Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
+(cherry picked from commit 02248eea5562c1df39ee23f195011afacc6759b0)
+Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
+---
+ src/plugins/platforms/xcb/qxcbclipboard.cpp | 8 ++++++--
+ 1 file changed, 6 insertions(+), 2 deletions(-)
+
+diff --git a/src/plugins/platforms/xcb/qxcbclipboard.cpp b/src/plugins/platforms/xcb/qxcbclipboard.cpp
+index 725c0e4d514..a4940f1c491 100644
+--- src/plugins/platforms/xcb/qxcbclipboard.cpp
++++ src/plugins/platforms/xcb/qxcbclipboard.cpp
+@@ -841,6 +841,8 @@ QByteArray QXcbClipboard::clipboardReadIncrementalProperty(xcb_window_t win, xcb
+ alloc_error = buf.size() != nbytes+1;
+ }
+
++ QElapsedTimer timer;
++ timer.start();
+ for (;;) {
+ connection()->flush();
+ xcb_generic_event_t *ge = waitForClipboardEvent(win, XCB_PROPERTY_NOTIFY);
+@@ -876,9 +878,11 @@ QByteArray QXcbClipboard::clipboardReadIncrementalProperty(xcb_window_t win, xcb
+ tmp_buf.resize(0);
+ offset += length;
+ }
+- } else {
+- break;
+ }
++
++ const auto elapsed = timer.elapsed();
++ if (elapsed > clipboard_timeout)
++ break;
+ }
+
+ // timed out ... create a new requestor window, otherwise the requestor
+--
+2.16.3
+