https://bugs.kde.org/show_bug.cgi?id=498837

Vlad Zahorodnii <vlad.zahorod...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|https://invent.kde.org/plas |https://invent.kde.org/plas
                   |ma/kwin/-/commit/60275f4dbe |ma/kwin/-/commit/8502be80d0
                   |1a7648b855de3fa76b2f6b378d0 |c6f8b3e081efb8390b9793b43cc
                   |86b                         |a41

--- Comment #12 from Vlad Zahorodnii <vlad.zahorod...@kde.org> ---
Git commit 8502be80d0c6f8b3e081efb8390b9793b43cca41 by Vlad Zahorodnii.
Committed on 04/02/2025 at 09:35.
Pushed by vladz into branch 'Plasma/6.3'.

xwayland: Keep ready fd until Xwayland is stopped

Xwayland writes the display name in two steps:

- first write the display name
- then write "\n"

Xwayland will quit if either write() fails. If the XwaylandLauncher
closes its endpoint of the displayfd after the first write(), then the
second write() may fail.

Given that we just want to know when Xwayland is ready, the ready fd
can be kept around until Xwayland is stopped.

The issue was introduced by 8e42599149bbd6e4e76baddd04b1c403131354ba
(kind of). Prior to that commit, kwin had been leaking the read endpoint
of the displayfd to Xwayland. So, even if kwin closes its endpoint,
Xwayland has it too so both write()s will succeed.


(cherry picked from commit 60275f4dbe1a7648b855de3fa76b2f6b378d086b)

Co-authored-by: Vlad Zahorodnii <vlad.zahorod...@kde.org>

M  +7    -8    src/xwayland/xwaylandlauncher.cpp
M  +0    -2    src/xwayland/xwaylandlauncher.h

https://invent.kde.org/plasma/kwin/-/commit/8502be80d0c6f8b3e081efb8390b9793b43cca41

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to