From b15be9c77de593581007de53b2bbca97d121900a Mon Sep 17 00:00:00 2001
From: Vaxry <vaxry@vaxry.net>
Date: Sun, 16 Jun 2024 21:34:17 +0200
Subject: [PATCH] xwayland: do not set a new data source if it has no MIMEs

ref #6247
---
 src/xwayland/XWM.cpp | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/xwayland/XWM.cpp b/src/xwayland/XWM.cpp
index 719adcb9..7b57de0f 100644
--- a/src/xwayland/XWM.cpp
+++ b/src/xwayland/XWM.cpp
@@ -1034,16 +1034,16 @@ void CXWM::initSelection() {
 }
 
 void CXWM::setClipboardToWayland(SXSelection& sel) {
-    sel.dataSource = makeShared<CXDataSource>(sel);
-    if (sel.dataSource->mimes().empty()) {
+    auto source = makeShared<CXDataSource>(sel);
+    if (source->mimes().empty()) {
         Debug::log(ERR, "[xwm] can't set clipboard: no MIMEs");
-        sel.dataSource.reset();
+        return;
     }
 
-    if (sel.dataSource) {
-        Debug::log(LOG, "[xwm] X clipboard at {:x} takes clipboard", (uintptr_t)sel.dataSource.get());
-        g_pSeatManager->setCurrentSelection(sel.dataSource);
-    }
+    sel.dataSource = source;
+
+    Debug::log(LOG, "[xwm] X clipboard at {:x} takes clipboard", (uintptr_t)sel.dataSource.get());
+    g_pSeatManager->setCurrentSelection(sel.dataSource);
 }
 
 void CXWM::getTransferData(SXSelection& sel) {