summaryrefslogtreecommitdiff
path: root/user
diff options
context:
space:
mode:
Diffstat (limited to 'user')
-rw-r--r--user/sddm/APKBUILD4
-rw-r--r--user/sddm/qt515.patch206
-rw-r--r--user/sddm/rootless-xorg.patch2012
3 files changed, 16 insertions, 2206 deletions
diff --git a/user/sddm/APKBUILD b/user/sddm/APKBUILD
index 00071648c..73a0b7e9f 100644
--- a/user/sddm/APKBUILD
+++ b/user/sddm/APKBUILD
@@ -28,7 +28,6 @@ source="https://github.com/sddm/sddm/releases/download/v$pkgver/sddm-$pkgver.tar
allocate-vt.patch
autologin-type.patch
session-env.patch
- rootless-xorg.patch
qt515.patch
sddm.initd
"
@@ -78,6 +77,5 @@ fea260015dfaf82e97ea2a77d7810e11f85503446ce6450ed9fe3072ce2151bb2ecac56fbc917a47
829397bdb1d5fd6887b7836d503b081a8827346aa775b702d4ea9946d3ff4374446175e462732e52e33e1a53481b99cbe652c44e031c79823d5fbf67e4b48c46 allocate-vt.patch
378a735946c93dc9fb746cc4d4cf436d728ec0f1fddd3fcac8797ef90acb0d6582e41ac0a91b96d3886695b8664627ef81822f531c67deb12085626206f27136 autologin-type.patch
0997ae1ea2f0b44835a397f107d73ae2eb0d492cbf97b3f446c388eebb6ec0b9255b2352a51c9e00ebf846c0fec61998895aba865e06814ee20ab9d526b3f353 session-env.patch
-3941a27c143205f7cc5d2e4fee96bdf463319b85f412ef076eedc730f82c16f0e565c897beeddda04b678bf9040511d3900ef8f7452a6a6b8f224c597232194b rootless-xorg.patch
-a7fba93e5eb78e2afca6106559cef64885d9be8ac5b69b2618af60d857c18a8ccac4800ca9698cc808e97039a9584dd073baf0a119dc8072d796a47001c52ee6 qt515.patch
+77d1abe99a8f0c38f9bf2d1ec695d371a46842b7f64c0a53e8f5c34550de5f9fba7ae9b187ba698e6658f03b18c6bd4b975532d9f2cecdb857e42e20413e96a7 qt515.patch
d603934552bad47edda458d7a4df2310e98bde74bdb3bf8588f5171b2a5d68814192b8dc8f5599b35402f9a747d519d985d4976e7aa50dabed445f99a112594c sddm.initd"
diff --git a/user/sddm/qt515.patch b/user/sddm/qt515.patch
index df58ca9fe..51094817c 100644
--- a/user/sddm/qt515.patch
+++ b/user/sddm/qt515.patch
@@ -1,199 +1,23 @@
-From 3eaa0d6ab1c7efc0bdd91fcd789d8187c6c27678 Mon Sep 17 00:00:00 2001
-From: Aleix Pol <aleixpol@kde.org>
-Date: Thu, 24 Jun 2021 12:34:13 +0200
-Subject: [PATCH] Bump to Qt 5.15, port away from deprecated APIs
+From e93bf95c54ad8c2a1604f8d7be05339164b19308 Mon Sep 17 00:00:00 2001
+From: "aacid@kde.org" <aacid@kde.org>
+Date: Thu, 12 Nov 2020 23:42:48 +0100
+Subject: [PATCH] Fix compilation once QTBUG-88431 gets fixed
+And also comes closes to Qt6 compatibility
---
- CMakeLists.txt | 2 +-
- README.md | 2 +-
- src/auth/Auth.cpp | 6 +++---
- src/auth/AuthRequest.cpp | 2 +-
- src/common/ConfigReader.cpp | 4 ++--
- src/daemon/PowerManager.cpp | 8 ++++++--
- src/daemon/XorgDisplayServer.cpp | 17 +++++++++--------
- src/greeter/GreeterProxy.cpp | 2 +-
- 8 files changed, 24 insertions(+), 19 deletions(-)
+ src/daemon/XorgDisplayServer.cpp | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index 41aee21d4..6936b31f6 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -96,7 +96,7 @@ find_package(XCB REQUIRED)
- find_package(XKB REQUIRED)
-
- # Qt 5
--find_package(Qt5 5.8.0 CONFIG REQUIRED Core DBus Gui Qml Quick LinguistTools Test)
-+find_package(Qt5 5.15.0 CONFIG REQUIRED Core DBus Gui Qml Quick LinguistTools Test)
-
- # find qt5 imports dir
- get_target_property(QMAKE_EXECUTABLE Qt5::qmake LOCATION)
-diff --git a/README.md b/README.md
-index 7c7a55588..97cb2221c 100644
---- a/README.md
-+++ b/README.md
-@@ -39,7 +39,7 @@ money that can be used to support the features that you most desire.
-
- ## INSTALLATION
-
--Qt >= 5.8.0 is required to use SDDM.
-+Qt >= 5.15.0 is required to use SDDM.
-
- SDDM runs the greeter as a system user named "sddm" whose home directory needs
- to be set to `/var/lib/sddm`.
-diff --git a/src/auth/Auth.cpp b/src/auth/Auth.cpp
-index 4450cc606..1dc2dd186 100644
---- a/src/auth/Auth.cpp
-+++ b/src/auth/Auth.cpp
-@@ -134,7 +134,7 @@ namespace SDDM {
- env.insert(QStringLiteral("LANG"), QStringLiteral("C"));
- child->setProcessEnvironment(env);
- connect(child, QOverload<int,QProcess::ExitStatus>::of(&QProcess::finished), this, &Auth::Private::childExited);
-- connect(child, QOverload<QProcess::ProcessError>::of(&QProcess::error), this, &Auth::Private::childError);
-+ connect(child, &QProcess::errorOccurred, this, &Auth::Private::childError);
- connect(request, &AuthRequest::finished, this, &Auth::Private::requestFinished);
- connect(request, &AuthRequest::promptsChanged, parent, &Auth::requestChanged);
- }
-@@ -267,8 +267,8 @@ namespace SDDM {
- }
-
- void Auth::registerTypes() {
-- qmlRegisterType<AuthPrompt>();
-- qmlRegisterType<AuthRequest>();
-+ qmlRegisterAnonymousType<AuthPrompt>("Auth", 1);
-+ qmlRegisterAnonymousType<AuthRequest>("Auth", 1);
- qmlRegisterType<Auth>("Auth", 1, 0, "Auth");
- }
-
-diff --git a/src/auth/AuthRequest.cpp b/src/auth/AuthRequest.cpp
-index 9583b0a4d..9ccc2d259 100644
---- a/src/auth/AuthRequest.cpp
-+++ b/src/auth/AuthRequest.cpp
-@@ -73,7 +73,7 @@ namespace SDDM {
- }
-
- QQmlListProperty<AuthPrompt> AuthRequest::promptsDecl() {
-- return QQmlListProperty<AuthPrompt>(this, d->prompts);
-+ return QQmlListProperty<AuthPrompt>(this, &d->prompts);
- }
-
- void AuthRequest::done() {
-diff --git a/src/common/ConfigReader.cpp b/src/common/ConfigReader.cpp
-index 041e5ed22..503b55e69 100644
---- a/src/common/ConfigReader.cpp
-+++ b/src/common/ConfigReader.cpp
-@@ -265,7 +265,7 @@ namespace SDDM {
- QByteArray junk;
- // stores the junk to the temporary storage
- auto collectJunk = [&junk](const QString &data) {
-- junk.append(data);
-+ junk.append(data.toUtf8());
- };
-
- // a short function to assign the current junk and current line to the right section, eventually create a new one
-@@ -275,7 +275,7 @@ namespace SDDM {
- sectionData[currentSection] = QByteArray();
- }
- sectionData[currentSection].append(junk);
-- sectionData[currentSection].append(data);
-+ sectionData[currentSection].append(data.toUtf8());
- junk.clear();
- };
-
-diff --git a/src/daemon/PowerManager.cpp b/src/daemon/PowerManager.cpp
-index 6eca29f55..ab1d6ac03 100644
---- a/src/daemon/PowerManager.cpp
-+++ b/src/daemon/PowerManager.cpp
-@@ -87,11 +87,15 @@ const QString UPOWER_OBJECT = QStringLiteral("org.freedesktop.UPower");
- }
-
- void powerOff() const {
-- QProcess::execute(mainConfig.HaltCommand.get());
-+ auto command = QProcess::splitCommand(mainConfig.HaltCommand.get());
-+ const QString program = command.takeFirst();
-+ QProcess::execute(program, command);
- }
-
- void reboot() const {
-- QProcess::execute(mainConfig.RebootCommand.get());
-+ auto command = QProcess::splitCommand(mainConfig.RebootCommand.get());
-+ const QString program = command.takeFirst();
-+ QProcess::execute(program, command);
- }
-
- void suspend() const {
diff --git a/src/daemon/XorgDisplayServer.cpp b/src/daemon/XorgDisplayServer.cpp
-index fc61ee2dd..13a1691b8 100644
+index 5f93a1b37..d5f29a94a 100644
--- a/src/daemon/XorgDisplayServer.cpp
+++ b/src/daemon/XorgDisplayServer.cpp
-@@ -110,7 +110,7 @@ namespace SDDM {
- QStringList args;
- if (!daemonApp->testing()) {
- process->setProgram(mainConfig.X11.ServerPath.get());
-- args << mainConfig.X11.ServerArguments.get().split(QLatin1Char(' '), QString::SkipEmptyParts)
-+ args << mainConfig.X11.ServerArguments.get().split(QLatin1Char(' '), Qt::SkipEmptyParts)
- << QStringLiteral("-background") << QStringLiteral("none")
- << QStringLiteral("-seat") << displayPtr()->seat()->name()
- << QStringLiteral("vt%1").arg(displayPtr()->terminalId());
-@@ -221,7 +221,7 @@ namespace SDDM {
- // log message
- qDebug() << "Display server stopped.";
-
-- QString displayStopCommand = mainConfig.X11.DisplayStopCommand.get();
-+ QStringList displayStopCommand = QProcess::splitCommand(mainConfig.X11.DisplayStopCommand.get());
-
- // create display setup script process
- QProcess *displayStopScript = new QProcess();
-@@ -236,7 +236,8 @@ namespace SDDM {
-
- // start display stop script
- qDebug() << "Running display stop script " << displayStopCommand;
-- displayStopScript->start(displayStopCommand);
-+ const auto program = displayStopCommand.takeFirst();
-+ displayStopScript->start(program, displayStopCommand);
-
- // wait for finished
- if (!displayStopScript->waitForFinished(5000))
-@@ -254,8 +255,6 @@ namespace SDDM {
+@@ -65,7 +65,7 @@ namespace SDDM {
+ // create a random hexadecimal number
+ const char *digits = "0123456789abcdef";
+ for (int i = 0; i < 32; ++i)
+- m_cookie[i] = digits[dis(gen)];
++ m_cookie[i] = QLatin1Char(digits[dis(gen)]);
}
- void XorgDisplayServer::setupDisplay() {
-- QString displayCommand = mainConfig.X11.DisplayCommand.get();
--
- // create cursor setup process
- QProcess *setCursor = new QProcess();
- // create display setup script process
-@@ -273,7 +272,7 @@ namespace SDDM {
- displayScript->setProcessEnvironment(env);
-
- qDebug() << "Setting default cursor";
-- setCursor->start(QStringLiteral("xsetroot -cursor_name left_ptr"));
-+ setCursor->start(QStringLiteral("xsetroot"), { QStringLiteral("-cursor_name"), QStringLiteral("left_ptr") });
-
- // delete setCursor on finish
- connect(setCursor, QOverload<int, QProcess::ExitStatus>::of(&QProcess::finished), setCursor, &QProcess::deleteLater);
-@@ -285,8 +284,10 @@ namespace SDDM {
- }
-
- // start display setup script
-- qDebug() << "Running display setup script " << displayCommand;
-- displayScript->start(displayCommand);
-+ qDebug() << "Running display setup script " << mainConfig.X11.DisplayCommand.get();
-+ QStringList displayCommand = QProcess::splitCommand(mainConfig.X11.DisplayCommand.get());
-+ const QString program = displayCommand.takeFirst();
-+ displayScript->start(program, displayCommand);
-
- // delete displayScript on finish
- connect(displayScript, QOverload<int, QProcess::ExitStatus>::of(&QProcess::finished), displayScript, &QProcess::deleteLater);
-diff --git a/src/greeter/GreeterProxy.cpp b/src/greeter/GreeterProxy.cpp
-index 8fde3d904..d6e62cc75 100644
---- a/src/greeter/GreeterProxy.cpp
-+++ b/src/greeter/GreeterProxy.cpp
-@@ -46,7 +46,7 @@ namespace SDDM {
- connect(d->socket, &QLocalSocket::connected, this, &GreeterProxy::connected);
- connect(d->socket, &QLocalSocket::disconnected, this, &GreeterProxy::disconnected);
- connect(d->socket, &QLocalSocket::readyRead, this, &GreeterProxy::readyRead);
-- connect(d->socket, QOverload<QLocalSocket::LocalSocketError>::of(&QLocalSocket::error), this, &GreeterProxy::error);
-+ connect(d->socket, &QLocalSocket::errorOccurred, this, &GreeterProxy::error);
-
- // connect to server
- d->socket->connectToServer(socket);
+ XorgDisplayServer::~XorgDisplayServer() {
diff --git a/user/sddm/rootless-xorg.patch b/user/sddm/rootless-xorg.patch
deleted file mode 100644
index 99a2e189d..000000000
--- a/user/sddm/rootless-xorg.patch
+++ /dev/null
@@ -1,2012 +0,0 @@
-From 9881225a92ffe6cf3d4a84273192b08b2d2f6046 Mon Sep 17 00:00:00 2001
-From: Pier Luigi Fiorini <pierluigi.fiorini@liri.io>
-Date: Tue, 27 Aug 2019 22:31:00 +0200
-Subject: [PATCH] X11 display server without root privileges
-
-Introduce an alternative display server option to run the greeter with
-the X11 server as an unprivileged user.
-
-Root privileges are required by default, but this will change in the
-future.
-
-Greeter output is forwarded to the helper process, instead of saving it
-into a separate file like user sessions do.
-This means the greeter output is available in the journal with the
-daemon and helper logs.
-
-Display start and stop commands are executed as sddm user and this might
-break the workflow of our users.
-That is the reason why we still run Xorg as root for the greeter by
-default.
-
-X11 user session always spawn a new display server without root
-privileges.
-
-Closes: #246
----
- CMakeLists.txt | 1 +
- data/man/sddm.conf.rst.in | 14 +-
- src/auth/Auth.cpp | 24 ++-
- src/auth/Auth.h | 14 ++
- src/auth/AuthMessages.h | 1 +
- src/common/Configuration.h | 3 +
- src/common/Session.cpp | 10 ++
- src/common/Session.h | 4 +
- src/common/XAuth.cpp | 127 ++++++++++++++
- src/common/XAuth.h | 55 ++++++
- src/daemon/CMakeLists.txt | 6 +-
- src/daemon/Display.cpp | 64 +++++--
- src/daemon/Display.h | 11 ++
- src/daemon/Greeter.cpp | 41 +++--
- src/daemon/Greeter.h | 5 +
- src/daemon/XorgDisplayServer.cpp | 78 ++-------
- src/daemon/XorgDisplayServer.h | 10 +-
- src/daemon/XorgUserDisplayServer.cpp | 102 +++++++++++
- src/daemon/XorgUserDisplayServer.h | 53 ++++++
- src/greeter/GreeterApp.cpp | 9 +
- src/helper/Backend.cpp | 5 +
- src/helper/Backend.h | 2 +
- src/helper/CMakeLists.txt | 4 +
- src/helper/HelperApp.cpp | 30 +++-
- src/helper/HelperApp.h | 1 +
- src/helper/UserSession.cpp | 202 +++++++++++++--------
- src/helper/UserSession.h | 20 ++-
- src/helper/backend/PamBackend.cpp | 6 +-
- src/helper/xorguserhelper.cpp | 252 +++++++++++++++++++++++++++
- src/helper/xorguserhelper.h | 64 +++++++
- 30 files changed, 1036 insertions(+), 182 deletions(-)
- create mode 100644 src/common/XAuth.cpp
- create mode 100644 src/common/XAuth.h
- create mode 100644 src/daemon/XorgUserDisplayServer.cpp
- create mode 100644 src/daemon/XorgUserDisplayServer.h
- create mode 100644 src/helper/xorguserhelper.cpp
- create mode 100644 src/helper/xorguserhelper.h
-
-diff --git a/CMakeLists.txt b/CMakeLists.txt
-index 9614b4e1e..41aee21d4 100644
---- a/CMakeLists.txt
-+++ b/CMakeLists.txt
-@@ -16,6 +16,7 @@ set(SDDM_VERSION_MAJOR 0)
- set(SDDM_VERSION_MINOR 19)
- set(SDDM_VERSION_PATCH 0)
- set(SDDM_VERSION_STRING "${SDDM_VERSION_MAJOR}.${SDDM_VERSION_MINOR}.${SDDM_VERSION_PATCH}")
-+add_compile_definitions("SDDM_VERSION=\"${SDDM_VERSION_STRING}\"")
-
- # Set up packaging
- set(CPACK_PACKAGE_NAME "sddm")
-diff --git a/data/man/sddm.conf.rst.in b/data/man/sddm.conf.rst.in
-index 1061540c0..960e65c68 100644
---- a/data/man/sddm.conf.rst.in
-+++ b/data/man/sddm.conf.rst.in
-@@ -6,7 +6,7 @@
- sddm display manager configuration
- ----------------------------------
-
--:Date: August 2015
-+:Date: March 2021
- :Version: sddm @SDDM_VERSION_STRING@
- :Manual section: 5
- :Manual group: sddm
-@@ -36,6 +36,14 @@ OPTIONS
-
- [General] section:
-
-+`DisplayServer=`
-+ Select the display server to use for the greeter.
-+ Valid values are:
-+ * `x11`: X server running as root.
-+ * `x11-user`: X server running as unprivileged user.
-+ Default value is "x11".
-+ For `x11-user` you might need to configure Xorg.wrap(1).
-+
- `HaltCommand=`
- Halt command.
- Default value is "@HALT_COMMAND@".
-@@ -134,10 +142,14 @@ OPTIONS
-
- `DisplayCommand=`
- Path of script to execute when starting the display server.
-+ The script will be executed as root when General.DisplayServer
-+ is "x11", otherwise as sddm user.
- Default value is "@DATA_INSTALL_DIR@/scripts/Xsetup".
-
- `DisplayStopCommand=`
- Path of script to execute when stopping the display server.
-+ The script will be executed as root when General.DisplayServer
-+ is "x11", otherwise as sddm user.
- Default value is "@DATA_INSTALL_DIR@/scripts/Xstop".
-
- `MinimumVT=`
-diff --git a/src/auth/Auth.cpp b/src/auth/Auth.cpp
-index 403186ee1..4450cc606 100644
---- a/src/auth/Auth.cpp
-+++ b/src/auth/Auth.cpp
-@@ -62,6 +62,7 @@ namespace SDDM {
- AuthRequest *request { nullptr };
- QProcess *child { nullptr };
- QLocalSocket *socket { nullptr };
-+ QString displayServerCmd;
- QString sessionPath { };
- QString user { };
- QString cookie { };
-@@ -202,6 +203,15 @@ namespace SDDM {
- str.send();
- break;
- }
-+ case DISPLAY_SERVER_STARTED: {
-+ QString displayName;
-+ str >> displayName;
-+ Q_EMIT auth->displayServerReady(displayName);
-+ str.reset();
-+ str << DISPLAY_SERVER_STARTED;
-+ str.send();
-+ break;
-+ }
- default: {
- Q_EMIT auth->error(QStringLiteral("Auth: Unexpected value received: %1").arg(m), ERROR_INTERNAL);
- }
-@@ -210,7 +220,9 @@ namespace SDDM {
-
- void Auth::Private::childExited(int exitCode, QProcess::ExitStatus exitStatus) {
- if (exitStatus != QProcess::NormalExit) {
-- qWarning("Auth: sddm-helper crashed (exit code %d)", exitCode);
-+ qWarning("Auth: sddm-helper (%s) crashed (exit code %d)",
-+ qPrintable(child->arguments().join(QLatin1Char(' '))),
-+ HelperExitStatus(exitStatus));
- Q_EMIT qobject_cast<Auth*>(parent())->error(child->errorString(), ERROR_INTERNAL);
- }
-
-@@ -334,6 +346,14 @@ namespace SDDM {
- }
- }
-
-+ void Auth::setDisplayServerCommand(const QString &command)
-+ {
-+ if (d->displayServerCmd != command) {
-+ d->displayServerCmd = command;
-+ Q_EMIT displayServerCommandChanged();
-+ }
-+ }
-+
- void Auth::setSession(const QString& path) {
- if (path != d->sessionPath) {
- d->sessionPath = path;
-@@ -361,6 +381,8 @@ namespace SDDM {
- args << QStringLiteral("--user") << d->user;
- if (d->autologin)
- args << QStringLiteral("--autologin");
-+ if (!d->displayServerCmd.isEmpty())
-+ args << QStringLiteral("--display-server") << d->displayServerCmd;
- if (d->greeter)
- args << QStringLiteral("--greeter");
- d->child->start(QStringLiteral("%1/sddm-helper").arg(QStringLiteral(LIBEXEC_INSTALL_DIR)), args);
-diff --git a/src/auth/Auth.h b/src/auth/Auth.h
-index f7cb8acd3..dc3df24d1 100644
---- a/src/auth/Auth.h
-+++ b/src/auth/Auth.h
-@@ -141,6 +141,12 @@ namespace SDDM {
- */
- void setUser(const QString &user);
-
-+ /**
-+ * Set the display server command to be started before the greeter.
-+ * @param command Command of the display server to be started
-+ */
-+ void setDisplayServerCommand(const QString &command);
-+
- /**
- * Set the session to be started after authenticating.
- * @param path Path of the session executable to be started
-@@ -165,6 +171,7 @@ namespace SDDM {
- void verboseChanged();
- void cookieChanged();
- void userChanged();
-+ void displayServerCommandChanged();
- void sessionChanged();
- void requestChanged();
-
-@@ -186,6 +193,13 @@ namespace SDDM {
- */
- void sessionStarted(bool success, qint64 pid);
-
-+ /**
-+ * Emitted when the display server is ready.
-+ *
-+ * @param displayName display name
-+ */
-+ void displayServerReady(const QString &displayName);
-+
- /**
- * Emitted when the helper quits, either after authentication or when the session ends.
- * Or, when something goes wrong.
-diff --git a/src/auth/AuthMessages.h b/src/auth/AuthMessages.h
-index 3bc97b6ba..6aea7483f 100644
---- a/src/auth/AuthMessages.h
-+++ b/src/auth/AuthMessages.h
-@@ -97,6 +97,7 @@ namespace SDDM {
- REQUEST,
- AUTHENTICATED,
- SESSION_STATUS,
-+ DISPLAY_SERVER_STARTED,
- MSG_LAST,
- };
-
-diff --git a/src/common/Configuration.h b/src/common/Configuration.h
-index b79871988..47bfa2710 100644
---- a/src/common/Configuration.h
-+++ b/src/common/Configuration.h
-@@ -37,6 +37,9 @@ namespace SDDM {
- enum NumState { NUM_NONE, NUM_SET_ON, NUM_SET_OFF };
-
- // Name Type Default value Description
-+ // TODO: Change default to x11-user in a future release
-+ Entry(DisplayServer, QString, _S("x11"), _S("Which display server should be used.\n"
-+ "Valid values are: x11, x11-user."));
- Entry(HaltCommand, QString, _S(HALT_COMMAND), _S("Halt command"));
- Entry(RebootCommand, QString, _S(REBOOT_COMMAND), _S("Reboot command"));
- Entry(Numlock, NumState, NUM_NONE, _S("Initial NumLock state. Can be on, off or none.\n"
-diff --git a/src/common/Session.cpp b/src/common/Session.cpp
-index a026c1f87..1b932c57a 100644
---- a/src/common/Session.cpp
-+++ b/src/common/Session.cpp
-@@ -52,6 +52,16 @@ namespace SDDM {
- return m_type;
- }
-
-+ int Session::vt() const
-+ {
-+ return m_vt;
-+ }
-+
-+ void Session::setVt(int vt)
-+ {
-+ m_vt = vt;
-+ }
-+
- QString Session::xdgSessionType() const
- {
- return m_xdgSessionType;
-diff --git a/src/common/Session.h b/src/common/Session.h
-index aa196e9c6..3abc993fb 100644
---- a/src/common/Session.h
-+++ b/src/common/Session.h
-@@ -43,6 +43,9 @@ namespace SDDM {
-
- Type type() const;
-
-+ int vt() const;
-+ void setVt(int vt);
-+
- QString xdgSessionType() const;
-
- QDir directory() const;
-@@ -70,6 +73,7 @@ namespace SDDM {
- QProcessEnvironment parseEnv(const QString &list);
- bool m_valid;
- Type m_type;
-+ int m_vt = 0;
- QDir m_dir;
- QString m_name;
- QString m_fileName;
-diff --git a/src/common/XAuth.cpp b/src/common/XAuth.cpp
-new file mode 100644
-index 000000000..bc7b10caf
---- /dev/null
-+++ b/src/common/XAuth.cpp
-@@ -0,0 +1,127 @@
-+/***************************************************************************
-+* Copyright (c) 2021 Pier Luigi Fiorini <pierluigi.fiorini@gmail.com>
-+* Copyright (c) 2013 Abdurrahman AVCI <abdurrahmanavci@gmail.com>
-+*
-+* This program is free software; you can redistribute it and/or modify
-+* it under the terms of the GNU General Public License as published by
-+* the Free Software Foundation; either version 2 of the License, or
-+* (at your option) any later version.
-+*
-+* This program is distributed in the hope that it will be useful,
-+* but WITHOUT ANY WARRANTY; without even the implied warranty of
-+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-+* GNU General Public License for more details.
-+*
-+* You should have received a copy of the GNU General Public License
-+* along with this program; if not, write to the
-+* Free Software Foundation, Inc.,
-+* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-+***************************************************************************/
-+
-+#include <QDebug>
-+#include <QDir>
-+#include <QUuid>
-+
-+#include "Configuration.h"
-+#include "Constants.h"
-+#include "XAuth.h"
-+
-+#include <random>
-+
-+namespace SDDM {
-+
-+XAuth::XAuth()
-+{
-+ m_authDir = QStringLiteral(RUNTIME_DIR);
-+}
-+
-+QString XAuth::authDirectory() const
-+{
-+ return m_authDir;
-+}
-+
-+void XAuth::setAuthDirectory(const QString &path)
-+{
-+ if (m_setup) {
-+ qWarning("Unable to set xauth directory after setup");
-+ return;
-+ }
-+
-+ m_authDir = path;
-+}
-+
-+QString XAuth::authPath() const
-+{
-+ return m_authPath;
-+}
-+
-+QString XAuth::cookie() const
-+{
-+ return m_cookie;
-+}
-+
-+void XAuth::setup()
-+{
-+ if (m_setup)
-+ return;
-+
-+ m_setup = true;
-+
-+ // Create directory if not existing
-+ QDir().mkpath(m_authDir);
-+
-+ // Set path
-+ m_authPath = QStringLiteral("%1/%2").arg(m_authDir).arg(QUuid::createUuid().toString(QUuid::WithoutBraces));
-+ qDebug() << "Xauthority path:" << m_authPath;
-+
-+ // Generate cookie
-+ std::random_device rd;
-+ std::mt19937 gen(rd());
-+ std::uniform_int_distribution<> dis(0, 15);
-+
-+ // Reseve 32 bytes
-+ m_cookie.reserve(32);
-+
-+ // Create a random hexadecimal number
-+ const char *digits = "0123456789abcdef";
-+ for (int i = 0; i < 32; ++i)
-+ m_cookie[i] = QLatin1Char(digits[dis(gen)]);
-+}
-+
-+bool XAuth::addCookie(const QString &display)
-+{
-+ if (!m_setup) {
-+ qWarning("Please setup xauth before adding a cookie");
-+ return false;
-+ }
-+
-+ return XAuth::addCookieToFile(display, m_authPath, m_cookie);
-+}
-+
-+bool XAuth::addCookieToFile(const QString &display, const QString &fileName,
-+ const QString &cookie)
-+{
-+ qDebug() << "Adding cookie to" << fileName;
-+
-+ // Touch file
-+ QFile file_handler(fileName);
-+ file_handler.open(QIODevice::Append);
-+ file_handler.close();
-+
-+ QString cmd = QStringLiteral("%1 -f %2 -q").arg(mainConfig.X11.XauthPath.get()).arg(fileName);
-+
-+ // Execute xauth
-+ FILE *fp = ::popen(qPrintable(cmd), "w");
-+
-+ // Check file
-+ if (!fp)
-+ return false;
-+ fprintf(fp, "remove %s\n", qPrintable(display));
-+ fprintf(fp, "add %s . %s\n", qPrintable(display), qPrintable(cookie));
-+ fprintf(fp, "exit\n");
-+
-+ // Close pipe
-+ return pclose(fp) == 0;
-+}
-+
-+} // namespace SDDM
-diff --git a/src/common/XAuth.h b/src/common/XAuth.h
-new file mode 100644
-index 000000000..3e80f4ead
---- /dev/null
-+++ b/src/common/XAuth.h
-@@ -0,0 +1,55 @@
-+/***************************************************************************
-+* Copyright (c) 2021 Pier Luigi Fiorini <pierluigi.fiorini@gmail.com>
-+* Copyright (c) 2013 Abdurrahman AVCI <abdurrahmanavci@gmail.com>
-+*
-+* This program is free software; you can redistribute it and/or modify
-+* it under the terms of the GNU General Public License as published by
-+* the Free Software Foundation; either version 2 of the License, or
-+* (at your option) any later version.
-+*
-+* This program is distributed in the hope that it will be useful,
-+* but WITHOUT ANY WARRANTY; without even the implied warranty of
-+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-+* GNU General Public License for more details.
-+*
-+* You should have received a copy of the GNU General Public License
-+* along with this program; if not, write to the
-+* Free Software Foundation, Inc.,
-+* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-+***************************************************************************/
-+
-+#ifndef SDDM_XAUTH_H
-+#define SDDM_XAUTH_H
-+
-+#include <QString>
-+
-+namespace SDDM {
-+
-+class XAuth
-+{
-+public:
-+ XAuth();
-+
-+ QString authDirectory() const;
-+ void setAuthDirectory(const QString &path);
-+
-+ QString authPath() const;
-+ QString cookie() const;
-+
-+ void setup();
-+ bool addCookie(const QString &display);
-+
-+ static bool addCookieToFile(const QString &display,
-+ const QString &fileName,
-+ const QString &cookie);
-+
-+private:
-+ bool m_setup = false;
-+ QString m_authDir;
-+ QString m_authPath;
-+ QString m_cookie;
-+};
-+
-+} // namespace SDDM
-+
-+#endif // SDDM_XAUTH_H
-diff --git a/src/daemon/CMakeLists.txt b/src/daemon/CMakeLists.txt
-index 86d014bec..b411e42bd 100644
---- a/src/daemon/CMakeLists.txt
-+++ b/src/daemon/CMakeLists.txt
-@@ -13,6 +13,8 @@ set(DAEMON_SOURCES
- ${CMAKE_SOURCE_DIR}/src/common/ThemeMetadata.cpp
- ${CMAKE_SOURCE_DIR}/src/common/Session.cpp
- ${CMAKE_SOURCE_DIR}/src/common/SocketWriter.cpp
-+ ${CMAKE_SOURCE_DIR}/src/common/XAuth.cpp
-+ ${CMAKE_SOURCE_DIR}/src/common/XAuth.h
- ${CMAKE_SOURCE_DIR}/src/auth/Auth.cpp
- ${CMAKE_SOURCE_DIR}/src/auth/AuthPrompt.cpp
- ${CMAKE_SOURCE_DIR}/src/auth/AuthRequest.cpp
-@@ -22,13 +24,15 @@ set(DAEMON_SOURCES
- DisplayManager.cpp
- DisplayServer.cpp
- LogindDBusTypes.cpp
-- XorgDisplayServer.cpp
- Greeter.cpp
- PowerManager.cpp
- Seat.cpp
- SeatManager.cpp
- SignalHandler.cpp
- SocketServer.cpp
-+ XorgDisplayServer.cpp
-+ XorgUserDisplayServer.cpp
-+ XorgUserDisplayServer.h
- )
-
- # Different implementations of the VT switching code
-diff --git a/src/daemon/Display.cpp b/src/daemon/Display.cpp
-index 3c77454fc..26ca3bb60 100644
---- a/src/daemon/Display.cpp
-+++ b/src/daemon/Display.cpp
-@@ -25,6 +25,7 @@
- #include "DaemonApp.h"
- #include "DisplayManager.h"
- #include "XorgDisplayServer.h"
-+#include "XorgUserDisplayServer.h"
- #include "Seat.h"
- #include "SocketServer.h"
- #include "Greeter.h"
-@@ -56,7 +57,6 @@
- namespace SDDM {
- Display::Display(Seat *parent) : QObject(parent),
- m_auth(new Auth(this)),
-- m_displayServer(new XorgDisplayServer(this)),
- m_seat(parent),
- m_socketServer(new SocketServer(this)),
- m_greeter(new Greeter(this)) {
-@@ -64,6 +64,32 @@ namespace SDDM {
- // Allocate vt
- m_terminalId = VirtualTerminal::setUpNewVt();
-
-+ // Save display server type
-+ const QString &displayServerType = mainConfig.DisplayServer.get().toLower();
-+ if (displayServerType == QLatin1String("x11"))
-+ m_displayServerType = X11DisplayServerType;
-+ else if (displayServerType == QStringLiteral("x11-user"))
-+ m_displayServerType = X11UserDisplayServerType;
-+ else {
-+ qWarning("\"%s\" is an invalid value for General.DisplayServer: fall back to \"x11\"",
-+ qPrintable(displayServerType));
-+ m_displayServerType = X11DisplayServerType;
-+ }
-+
-+ // Create display server
-+ switch (m_displayServerType) {
-+ case X11DisplayServerType:
-+ m_displayServer = new XorgDisplayServer(this);
-+ break;
-+ case X11UserDisplayServerType:
-+ m_displayServer = new XorgUserDisplayServer(this);
-+ m_greeter->setDisplayServerCommand(XorgUserDisplayServer::command(this));
-+ break;
-+ }
-+
-+ // Print what VT we are using for more information
-+ qDebug("Using VT %d", m_terminalId);
-+
- // respond to authentication requests
- m_auth->setVerbose(true);
- connect(m_auth, &Auth::requestChanged, this, &Display::slotRequestChanged);
-@@ -89,6 +115,16 @@ namespace SDDM {
- stop();
- }
-
-+ Display::DisplayServerType Display::displayServerType() const
-+ {
-+ return m_displayServerType;
-+ }
-+
-+ DisplayServer *Display::displayServer() const
-+ {
-+ return m_displayServer;
-+ }
-+
- QString Display::displayId() const {
- return m_displayServer->display();
- }
-@@ -181,7 +217,8 @@ namespace SDDM {
-
- // set greeter params
- m_greeter->setDisplay(this);
-- m_greeter->setAuthPath(qobject_cast<XorgDisplayServer *>(m_displayServer)->authPath());
-+ if (qobject_cast<XorgDisplayServer *>(m_displayServer))
-+ m_greeter->setAuthPath(qobject_cast<XorgDisplayServer *>(m_displayServer)->authPath());
- m_greeter->setSocket(m_socketServer->socketAddress());
- m_greeter->setTheme(findGreeterTheme());
-
-@@ -317,20 +354,16 @@ namespace SDDM {
- // last session later, in slotAuthenticationFinished()
- m_sessionName = session.fileName();
-
-+ // New VT
-+ m_lastSession.setVt(VirtualTerminal::setUpNewVt());
-+
- // some information
- qDebug() << "Session" << m_sessionName << "selected, command:" << session.exec();
-
- QProcessEnvironment env;
- env.insert(session.additionalEnv());
-
-- if (seat()->name() == QLatin1String("seat0")) {
-- // Use the greeter VT, for wayland sessions the helper overwrites this
-- env.insert(QStringLiteral("XDG_VTNR"), QString::number(terminalId()));
-- }
--
- env.insert(QStringLiteral("PATH"), mainConfig.Users.DefaultPath.get());
-- if (session.xdgSessionType() == QLatin1String("x11"))
-- env.insert(QStringLiteral("DISPLAY"), name());
- env.insert(QStringLiteral("XDG_SEAT_PATH"), daemonApp->displayManager()->seatPath(seat()->name()));
- env.insert(QStringLiteral("XDG_SESSION_PATH"), daemonApp->displayManager()->sessionPath(QStringLiteral("Session%1").arg(daemonApp->newSessionId())));
- env.insert(QStringLiteral("DESKTOP_SESSION"), session.desktopSession());
-@@ -338,10 +371,16 @@ namespace SDDM {
- env.insert(QStringLiteral("XDG_SESSION_CLASS"), QStringLiteral("user"));
- env.insert(QStringLiteral("XDG_SESSION_TYPE"), session.xdgSessionType());
- env.insert(QStringLiteral("XDG_SEAT"), seat()->name());
-+ env.insert(QStringLiteral("XDG_VTNR"), QString::number(m_lastSession.vt()));
- env.insert(QStringLiteral("XDG_SESSION_DESKTOP"), session.desktopNames());
-
- m_auth->insertEnvironment(env);
-
-+ if (session.xdgSessionType() == QLatin1String("x11"))
-+ m_auth->setDisplayServerCommand(XorgUserDisplayServer::command(this));
-+ else
-+ m_auth->setDisplayServerCommand(QStringLiteral());
-+
- m_auth->setUser(user);
- if (m_reuseSessionId.isNull()) {
- m_auth->setSession(session.exec());
-@@ -358,7 +397,8 @@ namespace SDDM {
- manager.UnlockSession(m_reuseSessionId);
- manager.ActivateSession(m_reuseSessionId);
- } else {
-- m_auth->setCookie(qobject_cast<XorgDisplayServer *>(m_displayServer)->cookie());
-+ if (qobject_cast<XorgDisplayServer *>(m_displayServer))
-+ m_auth->setCookie(qobject_cast<XorgDisplayServer *>(m_displayServer)->cookie());
- }
-
- // save last user and last session
-@@ -411,6 +451,10 @@ namespace SDDM {
- // greeter
- if (status != Auth::HELPER_AUTH_ERROR)
- stop();
-+
-+ // Start the greeter again as soon as the user session is closed
-+ if (m_auth->user() != QLatin1String("sddm"))
-+ m_greeter->start();
- }
-
- void Display::slotRequestChanged() {
-diff --git a/src/daemon/Display.h b/src/daemon/Display.h
-index 61dd9f630..a31542cda 100644
---- a/src/daemon/Display.h
-+++ b/src/daemon/Display.h
-@@ -41,9 +41,18 @@ namespace SDDM {
- Q_OBJECT
- Q_DISABLE_COPY(Display)
- public:
-+ enum DisplayServerType {
-+ X11DisplayServerType,
-+ X11UserDisplayServerType
-+ };
-+ Q_ENUM(DisplayServerType)
-+
- explicit Display(Seat *parent);
- ~Display();
-
-+ DisplayServerType displayServerType() const;
-+ DisplayServer *displayServer() const;
-+
- QString displayId() const;
- const int terminalId() const;
-
-@@ -76,6 +85,8 @@ namespace SDDM {
- void startAuth(const QString &user, const QString &password,
- const Session &session);
-
-+ DisplayServerType m_displayServerType = X11DisplayServerType;
-+
- bool m_relogin { true };
- bool m_started { false };
-
-diff --git a/src/daemon/Greeter.cpp b/src/daemon/Greeter.cpp
-index 436ecc3d5..158d8bbbe 100644
---- a/src/daemon/Greeter.cpp
-+++ b/src/daemon/Greeter.cpp
-@@ -27,6 +27,7 @@
- #include "ThemeConfig.h"
- #include "ThemeMetadata.h"
- #include "Display.h"
-+#include "XorgUserDisplayServer.h"
-
- #include <QtCore/QDebug>
- #include <QtCore/QProcess>
-@@ -71,6 +72,16 @@ namespace SDDM {
- }
- }
-
-+ QString Greeter::displayServerCommand() const
-+ {
-+ return m_displayServerCmd;
-+ }
-+
-+ void Greeter::setDisplayServerCommand(const QString &cmd)
-+ {
-+ m_displayServerCmd = cmd;
-+ }
-+
- bool Greeter::start() {
- // check flag
- if (m_started)
-@@ -111,17 +122,16 @@ namespace SDDM {
- // log message
- qDebug() << "Greeter starting...";
-
-- // set process environment
-+ args << QStringLiteral("--test-mode");
-+
-+ // set process environment
- QProcessEnvironment env = QProcessEnvironment::systemEnvironment();
- env.insert(QStringLiteral("DISPLAY"), m_display->name());
- env.insert(QStringLiteral("XAUTHORITY"), m_authPath);
- env.insert(QStringLiteral("XCURSOR_THEME"), xcursorTheme);
-- env.insert(QStringLiteral("QT_IM_MODULE"), mainConfig.InputMethod.get());
- m_process->setProcessEnvironment(env);
-
- // start greeter
-- if (daemonApp->testing())
-- args << QStringLiteral("--test-mode");
- m_process->start(QStringLiteral("%1/sddm-greeter").arg(QStringLiteral(BIN_INSTALL_DIR)), args);
-
- //if we fail to start bail immediately, and don't block in waitForStarted
-@@ -149,11 +159,12 @@ namespace SDDM {
- m_auth->setVerbose(true);
- connect(m_auth, &Auth::requestChanged, this, &Greeter::onRequestChanged);
- connect(m_auth, &Auth::sessionStarted, this, &Greeter::onSessionStarted);
-+ connect(m_auth, &Auth::displayServerReady, this, &Greeter::onDisplayServerReady);
- connect(m_auth, &Auth::finished, this, &Greeter::onHelperFinished);
- connect(m_auth, &Auth::info, this, &Greeter::authInfo);
- connect(m_auth, &Auth::error, this, &Greeter::authError);
-
-- // greeter command
-+ // command
- QStringList cmd;
- cmd << QStringLiteral("%1/sddm-greeter").arg(QStringLiteral(BIN_INSTALL_DIR))
- << args;
-@@ -173,8 +184,6 @@ namespace SDDM {
- }, sysenv, env);
-
- env.insert(QStringLiteral("PATH"), mainConfig.Users.DefaultPath.get());
-- env.insert(QStringLiteral("DISPLAY"), m_display->name());
-- env.insert(QStringLiteral("XAUTHORITY"), m_authPath);
- env.insert(QStringLiteral("XCURSOR_THEME"), xcursorTheme);
- env.insert(QStringLiteral("XDG_SEAT"), m_display->seat()->name());
- env.insert(QStringLiteral("XDG_SEAT_PATH"), daemonApp->displayManager()->seatPath(m_display->seat()->name()));
-@@ -183,11 +192,10 @@ namespace SDDM {
- env.insert(QStringLiteral("XDG_VTNR"), QString::number(m_display->terminalId()));
- env.insert(QStringLiteral("XDG_SESSION_CLASS"), QStringLiteral("greeter"));
- env.insert(QStringLiteral("XDG_SESSION_TYPE"), m_display->sessionType());
-- env.insert(QStringLiteral("QT_IM_MODULE"), mainConfig.InputMethod.get());
--
-- //some themes may use KDE components and that will automatically load KDE's crash handler which we don't want
-- //counterintuitively setting this env disables that handler
-- env.insert(QStringLiteral("KDE_DEBUG"), QStringLiteral("1"));
-+ if (m_display->displayServerType() == Display::X11DisplayServerType) {
-+ env.insert(QStringLiteral("DISPLAY"), m_display->name());
-+ env.insert(QStringLiteral("XAUTHORITY"), m_authPath);
-+ }
- m_auth->insertEnvironment(env);
-
- // log message
-@@ -195,6 +203,7 @@ namespace SDDM {
-
- // start greeter
- m_auth->setUser(QStringLiteral("sddm"));
-+ m_auth->setDisplayServerCommand(m_displayServerCmd);
- m_auth->setGreeter(true);
- m_auth->setSession(cmd.join(QLatin1Char(' ')));
- m_auth->start();
-@@ -261,6 +270,14 @@ namespace SDDM {
- qDebug() << "Greeter session failed to start";
- }
-
-+ void Greeter::onDisplayServerReady(const QString &displayName)
-+ {
-+ auto *displayServer = m_display->displayServer();
-+ auto *xorgUser = qobject_cast<XorgUserDisplayServer *>(displayServer);
-+ if (xorgUser)
-+ xorgUser->setDisplayName(displayName);
-+ }
-+
- void Greeter::onHelperFinished(Auth::HelperExitStatus status) {
- // reset flag
- m_started = false;
-diff --git a/src/daemon/Greeter.h b/src/daemon/Greeter.h
-index 7391a3597..bf472375f 100644
---- a/src/daemon/Greeter.h
-+++ b/src/daemon/Greeter.h
-@@ -43,6 +43,9 @@ namespace SDDM {
- void setSocket(const QString &socket);
- void setTheme(const QString &theme);
-
-+ QString displayServerCommand() const;
-+ void setDisplayServerCommand(const QString &cmd);
-+
- public slots:
- bool start();
- void stop();
-@@ -51,6 +54,7 @@ namespace SDDM {
- private slots:
- void onRequestChanged();
- void onSessionStarted(bool success);
-+ void onDisplayServerReady(const QString &displayName);
- void onHelperFinished(Auth::HelperExitStatus status);
- void onReadyReadStandardOutput();
- void onReadyReadStandardError();
-@@ -64,6 +68,7 @@ namespace SDDM {
- QString m_authPath;
- QString m_socket;
- QString m_themePath;
-+ QString m_displayServerCmd;
- ThemeMetadata *m_metadata { nullptr };
- ThemeConfig *m_themeConfig { nullptr };
-
-diff --git a/src/daemon/XorgDisplayServer.cpp b/src/daemon/XorgDisplayServer.cpp
-index 331adcda7..fc61ee2dd 100644
---- a/src/daemon/XorgDisplayServer.cpp
-+++ b/src/daemon/XorgDisplayServer.cpp
-@@ -41,31 +41,9 @@
-
- namespace SDDM {
- XorgDisplayServer::XorgDisplayServer(Display *parent) : DisplayServer(parent) {
-- // get auth directory
-- QString authDir = QStringLiteral(RUNTIME_DIR);
--
-- // use "." as authdir in test mode
- if (daemonApp->testing())
-- authDir = QStringLiteral(".");
--
-- // create auth dir if not existing
-- QDir().mkpath(authDir);
--
-- // set auth path
-- m_authPath = QStringLiteral("%1/%2").arg(authDir).arg(QUuid::createUuid().toString());
--
-- // generate cookie
-- std::random_device rd;
-- std::mt19937 gen(rd());
-- std::uniform_int_distribution<> dis(0, 15);
--
-- // resever 32 bytes
-- m_cookie.reserve(32);
--
-- // create a random hexadecimal number
-- const char *digits = "0123456789abcdef";
-- for (int i = 0; i < 32; ++i)
-- m_cookie[i] = digits[dis(gen)];
-+ m_xauth.setAuthDirectory(QStringLiteral("."));
-+ m_xauth.setup();
- }
-
- XorgDisplayServer::~XorgDisplayServer() {
-@@ -76,41 +54,16 @@ namespace SDDM {
- return m_display;
- }
-
-- const QString &XorgDisplayServer::authPath() const {
-- return m_authPath;
-+ QString XorgDisplayServer::authPath() const {
-+ return m_xauth.authPath();
- }
-
- QString XorgDisplayServer::sessionType() const {
- return QStringLiteral("x11");
- }
-
-- const QString &XorgDisplayServer::cookie() const {
-- return m_cookie;
-- }
--
-- bool XorgDisplayServer::addCookie(const QString &file) {
-- // log message
-- qDebug() << "Adding cookie to" << file;
--
-- // Touch file
-- QFile file_handler(file);
-- file_handler.open(QIODevice::Append);
-- file_handler.close();
--
-- QString cmd = QStringLiteral("%1 -f %2 -q").arg(mainConfig.X11.XauthPath.get()).arg(file);
--
-- // execute xauth
-- FILE *fp = popen(qPrintable(cmd), "w");
--
-- // check file
-- if (!fp)
-- return false;
-- fprintf(fp, "remove %s\n", qPrintable(m_display));
-- fprintf(fp, "add %s . %s\n", qPrintable(m_display), qPrintable(m_cookie));
-- fprintf(fp, "exit\n");
--
-- // close pipe
-- return pclose(fp) == 0;
-+ QString XorgDisplayServer::cookie() const {
-+ return m_xauth.cookie();
- }
-
- bool XorgDisplayServer::start() {
-@@ -136,7 +89,7 @@ namespace SDDM {
- // For the X server's copy, the display number doesn't matter.
- // An empty file would result in no access control!
- m_display = QStringLiteral(":0");
-- if(!addCookie(m_authPath)) {
-+ if(!m_xauth.addCookie(m_display)) {
- qCritical() << "Failed to write xauth file";
- return false;
- }
-@@ -159,18 +112,15 @@ namespace SDDM {
- process->setProgram(mainConfig.X11.ServerPath.get());
- args << mainConfig.X11.ServerArguments.get().split(QLatin1Char(' '), QString::SkipEmptyParts)
- << QStringLiteral("-background") << QStringLiteral("none")
-- << QStringLiteral("-seat") << displayPtr()->seat()->name();
--
-- if (displayPtr()->seat()->name() == QLatin1String("seat0")) {
-- args << QStringLiteral("vt%1").arg(displayPtr()->terminalId());
-- }
-+ << QStringLiteral("-seat") << displayPtr()->seat()->name()
-+ << QStringLiteral("vt%1").arg(displayPtr()->terminalId());
- } else {
- process->setProgram(mainConfig.X11.XephyrPath.get());
- args << QStringLiteral("-br")
- << QStringLiteral("-screen") << QStringLiteral("800x600");
- }
-
-- args << QStringLiteral("-auth") << m_authPath
-+ args << QStringLiteral("-auth") << m_xauth.authPath()
- << QStringLiteral("-noreset")
- << QStringLiteral("-displayfd") << QString::number(pipeFds[1]);
-
-@@ -222,13 +172,13 @@ namespace SDDM {
- // The file is also used by the greeter, which does care about the
- // display number. Write the proper entry, if it's different.
- if(m_display != QStringLiteral(":0")) {
-- if(!addCookie(m_authPath)) {
-+ if(!m_xauth.addCookie(m_display)) {
- qCritical() << "Failed to write xauth file";
- stop();
- return false;
- }
- }
-- changeOwner(m_authPath);
-+ changeOwner(m_xauth.authPath());
-
- emit started();
-
-@@ -297,7 +247,7 @@ namespace SDDM {
- displayStopScript = nullptr;
-
- // remove authority file
-- QFile::remove(m_authPath);
-+ QFile::remove(m_xauth.authPath());
-
- // emit signal
- emit stopped();
-@@ -316,7 +266,7 @@ namespace SDDM {
- env.insert(QStringLiteral("DISPLAY"), m_display);
- env.insert(QStringLiteral("HOME"), QStringLiteral("/"));
- env.insert(QStringLiteral("PATH"), mainConfig.Users.DefaultPath.get());
-- env.insert(QStringLiteral("XAUTHORITY"), m_authPath);
-+ env.insert(QStringLiteral("XAUTHORITY"), m_xauth.authPath());
- env.insert(QStringLiteral("SHELL"), QStringLiteral("/bin/sh"));
- env.insert(QStringLiteral("XCURSOR_THEME"), mainConfig.Theme.CursorTheme.get());
- setCursor->setProcessEnvironment(env);
-diff --git a/src/daemon/XorgDisplayServer.h b/src/daemon/XorgDisplayServer.h
-index e97a0b531..ebf189920 100644
---- a/src/daemon/XorgDisplayServer.h
-+++ b/src/daemon/XorgDisplayServer.h
-@@ -22,6 +22,7 @@
- #define SDDM_XORGDISPLAYSERVER_H
-
- #include "DisplayServer.h"
-+#include "XAuth.h"
-
- class QProcess;
-
-@@ -34,13 +35,11 @@ namespace SDDM {
- ~XorgDisplayServer();
-
- const QString &display() const;
-- const QString &authPath() const;
-+ QString authPath() const;
-
- QString sessionType() const;
-
-- const QString &cookie() const;
--
-- bool addCookie(const QString &file);
-+ QString cookie() const;
-
- public slots:
- bool start();
-@@ -49,8 +48,7 @@ namespace SDDM {
- void setupDisplay();
-
- private:
-- QString m_authPath;
-- QString m_cookie;
-+ XAuth m_xauth;
-
- QProcess *process { nullptr };
-
-diff --git a/src/daemon/XorgUserDisplayServer.cpp b/src/daemon/XorgUserDisplayServer.cpp
-new file mode 100644
-index 000000000..1e7c008a4
---- /dev/null
-+++ b/src/daemon/XorgUserDisplayServer.cpp
-@@ -0,0 +1,102 @@
-+/***************************************************************************
-+* Copyright (c) 2021 Pier Luigi Fiorini <pierluigi.fiorini@gmail.com>
-+*
-+* This program is free software; you can redistribute it and/or modify
-+* it under the terms of the GNU General Public License as published by
-+* the Free Software Foundation; either version 2 of the License, or
-+* (at your option) any later version.
-+*
-+* This program is distributed in the hope that it will be useful,
-+* but WITHOUT ANY WARRANTY; without even the implied warranty of
-+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-+* GNU General Public License for more details.
-+*
-+* You should have received a copy of the GNU General Public License
-+* along with this program; if not, write to the
-+* Free Software Foundation, Inc.,
-+* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-+***************************************************************************/
-+
-+#include "Configuration.h"
-+#include "DaemonApp.h"
-+#include "Display.h"
-+#include "Seat.h"
-+#include "XorgUserDisplayServer.h"
-+
-+namespace SDDM {
-+
-+XorgUserDisplayServer::XorgUserDisplayServer(Display *parent)
-+ : DisplayServer(parent)
-+{
-+}
-+
-+XorgUserDisplayServer::~XorgUserDisplayServer()
-+{
-+ stop();
-+}
-+
-+QString XorgUserDisplayServer::sessionType() const
-+{
-+ return QStringLiteral("x11");
-+}
-+
-+void XorgUserDisplayServer::setDisplayName(const QString &displayName)
-+{
-+ m_display = displayName;
-+}
-+
-+QString XorgUserDisplayServer::command(Display *display)
-+{
-+ QStringList args;
-+
-+ if (daemonApp->testing()) {
-+ args << mainConfig.X11.XephyrPath.get()
-+ << QStringLiteral("-br")
-+ << QStringLiteral("-screen") << QStringLiteral("800x600");
-+ } else {
-+ args << mainConfig.X11.ServerPath.get()
-+ << mainConfig.X11.ServerArguments.get().split(QLatin1Char(' '), Qt::SkipEmptyParts)
-+ << QStringLiteral("-background") << QStringLiteral("none")
-+ << QStringLiteral("-seat") << display->seat()->name()
-+ << QStringLiteral("-noreset")
-+ << QStringLiteral("-keeptty")
-+ << QStringLiteral("-novtswitch")
-+ << QStringLiteral("-verbose") << QStringLiteral("3");
-+ }
-+
-+ return args.join(QLatin1Char(' '));
-+}
-+
-+bool XorgUserDisplayServer::start()
-+{
-+ // Check flag
-+ if (m_started)
-+ return false;
-+
-+ // Set flag
-+ m_started = true;
-+ emit started();
-+
-+ return true;
-+}
-+
-+void XorgUserDisplayServer::stop()
-+{
-+ // Check flag
-+ if (!m_started)
-+ return;
-+
-+ // Reset flag
-+ m_started = false;
-+ emit stopped();
-+}
-+
-+void XorgUserDisplayServer::finished()
-+{
-+}
-+
-+void XorgUserDisplayServer::setupDisplay()
-+{
-+}
-+
-+} // namespace SDDM
-diff --git a/src/daemon/XorgUserDisplayServer.h b/src/daemon/XorgUserDisplayServer.h
-new file mode 100644
-index 000000000..aa7cbe4a8
---- /dev/null
-+++ b/src/daemon/XorgUserDisplayServer.h
-@@ -0,0 +1,53 @@
-+/***************************************************************************
-+* Copyright (c) 2021 Pier Luigi Fiorini <pierluigi.fiorini@gmail.com>
-+*
-+* This program is free software; you can redistribute it and/or modify
-+* it under the terms of the GNU General Public License as published by
-+* the Free Software Foundation; either version 2 of the License, or
-+* (at your option) any later version.
-+*
-+* This program is distributed in the hope that it will be useful,
-+* but WITHOUT ANY WARRANTY; without even the implied warranty of
-+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-+* GNU General Public License for more details.
-+*
-+* You should have received a copy of the GNU General Public License
-+* along with this program; if not, write to the
-+* Free Software Foundation, Inc.,
-+* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-+***************************************************************************/
-+
-+#ifndef SDDM_XORGUSERDISPLAYSERVER_H
-+#define SDDM_XORGUSERDISPLAYSERVER_H
-+
-+#include "DisplayServer.h"
-+#include "XAuth.h"
-+
-+class QProcess;
-+
-+namespace SDDM {
-+
-+class XorgUserDisplayServer : public DisplayServer
-+{
-+ Q_OBJECT
-+ Q_DISABLE_COPY(XorgUserDisplayServer)
-+public:
-+ explicit XorgUserDisplayServer(Display *parent);
-+ ~XorgUserDisplayServer();
-+
-+ QString sessionType() const;
-+
-+ void setDisplayName(const QString &displayName);
-+
-+ static QString command(Display *display);
-+
-+public Q_SLOTS:
-+ bool start();
-+ void stop();
-+ void finished();
-+ void setupDisplay();
-+};
-+
-+} // namespace SDDM
-+
-+#endif // SDDM_XORGUSERDISPLAYSERVER_H
-diff --git a/src/greeter/GreeterApp.cpp b/src/greeter/GreeterApp.cpp
-index 01f53fafb..b7f740f41 100644
---- a/src/greeter/GreeterApp.cpp
-+++ b/src/greeter/GreeterApp.cpp
-@@ -337,6 +337,15 @@ int main(int argc, char **argv)
- QSurfaceFormat::setDefaultFormat(format);
- }
-
-+ // Some themes may use KDE components and that will automatically load KDE's
-+ // crash handler which we don't want counterintuitively setting this env
-+ // disables that handler
-+ qputenv("KDE_DEBUG", "1");
-+
-+ // Qt IM module
-+ if (!SDDM::mainConfig.InputMethod.get().isEmpty())
-+ qputenv("QT_IM_MODULE", SDDM::mainConfig.InputMethod.get().toLocal8Bit().constData());
-+
- QGuiApplication app(argc, argv);
-
- QCommandLineParser parser;
-diff --git a/src/helper/Backend.cpp b/src/helper/Backend.cpp
-index a324b39fb..9a36a62ba 100644
---- a/src/helper/Backend.cpp
-+++ b/src/helper/Backend.cpp
-@@ -54,6 +54,11 @@ namespace SDDM {
- m_autologin = on;
- }
-
-+ void Backend::setDisplayServer(bool on)
-+ {
-+ m_displayServer = on;
-+ }
-+
- void Backend::setGreeter(bool on) {
- m_greeter = on;
- }
-diff --git a/src/helper/Backend.h b/src/helper/Backend.h
-index b790e0011..915d09ca8 100644
---- a/src/helper/Backend.h
-+++ b/src/helper/Backend.h
-@@ -36,6 +36,7 @@ namespace SDDM {
- static Backend *get(HelperApp *parent);
-
- void setAutologin(bool on = true);
-+ void setDisplayServer(bool on = true);
- void setGreeter(bool on = true);
-
- public slots:
-@@ -50,6 +51,7 @@ namespace SDDM {
- Backend(HelperApp *parent);
- HelperApp *m_app;
- bool m_autologin { false };
-+ bool m_displayServer = false;
- bool m_greeter { false };
- };
- }
-diff --git a/src/helper/CMakeLists.txt b/src/helper/CMakeLists.txt
-index 8914ea757..24f36eb97 100644
---- a/src/helper/CMakeLists.txt
-+++ b/src/helper/CMakeLists.txt
-@@ -10,9 +10,13 @@ set(HELPER_SOURCES
- ${CMAKE_SOURCE_DIR}/src/common/Configuration.cpp
- ${CMAKE_SOURCE_DIR}/src/common/ConfigReader.cpp
- ${CMAKE_SOURCE_DIR}/src/common/SafeDataStream.cpp
-+ ${CMAKE_SOURCE_DIR}/src/common/XAuth.cpp
-+ ${CMAKE_SOURCE_DIR}/src/common/XAuth.h
- Backend.cpp
- HelperApp.cpp
- UserSession.cpp
-+ xorguserhelper.cpp
-+ xorguserhelper.h
- )
-
- # Different implementations of the VT switching code
-diff --git a/src/helper/HelperApp.cpp b/src/helper/HelperApp.cpp
-index 3f92f1d8f..059e9c156 100644
---- a/src/helper/HelperApp.cpp
-+++ b/src/helper/HelperApp.cpp
-@@ -88,6 +88,16 @@ namespace SDDM {
- m_user = args[pos + 1];
- }
-
-+ if ((pos = args.indexOf(QStringLiteral("--display-server"))) >= 0) {
-+ if (pos >= args.length() - 1) {
-+ qCritical() << "This application is not supposed to be executed manually";
-+ exit(Auth::HELPER_OTHER_ERROR);
-+ return;
-+ }
-+ m_session->setDisplayServerCommand(args[pos + 1]);
-+ m_backend->setDisplayServer(true);
-+ }
-+
- if ((pos = args.indexOf(QStringLiteral("--autologin"))) >= 0) {
- m_backend->setAutologin(true);
- }
-@@ -103,7 +113,7 @@ namespace SDDM {
- }
-
- connect(m_socket, &QLocalSocket::connected, this, &HelperApp::doAuth);
-- connect(m_session, QOverload<int>::of(&QProcess::finished), this, &HelperApp::sessionFinished);
-+ connect(m_session, &UserSession::finished, this, &HelperApp::sessionFinished);
- m_socket->connectToServer(server, QIODevice::ReadWrite | QIODevice::Unbuffered);
- }
-
-@@ -131,11 +141,6 @@ namespace SDDM {
-
- if (!m_session->path().isEmpty()) {
- env.insert(m_session->processEnvironment());
-- // Allocate a new VT for the wayland session
-- if(env.value(QStringLiteral("XDG_SESSION_TYPE")) == QLatin1String("wayland")) {
-- int vtNumber = VirtualTerminal::setUpNewVt();
-- env.insert(QStringLiteral("XDG_VTNR"), QString::number(vtNumber));
-- }
- m_session->setProcessEnvironment(env);
-
- if (!m_backend->openSession()) {
-@@ -216,6 +221,19 @@ namespace SDDM {
- }
- }
-
-+ void HelperApp::displayServerStarted(const QString &displayName)
-+ {
-+ Msg m = Msg::MSG_UNKNOWN;
-+ SafeDataStream str(m_socket);
-+ str << Msg::DISPLAY_SERVER_STARTED << displayName;
-+ str.send();
-+ str.receive();
-+ str >> m;
-+ if (m != DISPLAY_SERVER_STARTED) {
-+ qCritical() << "Received a wrong opcode instead of DISPLAY_SERVER_STARTED:" << m;
-+ }
-+ }
-+
- UserSession *HelperApp::session() {
- return m_session;
- }
-diff --git a/src/helper/HelperApp.h b/src/helper/HelperApp.h
-index 632435ee5..1a75c2d67 100644
---- a/src/helper/HelperApp.h
-+++ b/src/helper/HelperApp.h
-@@ -48,6 +48,7 @@ namespace SDDM {
- void error(const QString &message, Auth::Error type);
- QProcessEnvironment authenticated(const QString &user);
- void sessionOpened(bool success);
-+ void displayServerStarted(const QString &displayName);
-
- private slots:
- void setUp();
-diff --git a/src/helper/UserSession.cpp b/src/helper/UserSession.cpp
-index 8e36f2113..f02a42c21 100644
---- a/src/helper/UserSession.cpp
-+++ b/src/helper/UserSession.cpp
-@@ -19,10 +19,14 @@
- *
- */
-
-+#include <QSocketNotifier>
-+
- #include "Configuration.h"
- #include "UserSession.h"
- #include "HelperApp.h"
- #include "VirtualTerminal.h"
-+#include "XAuth.h"
-+#include "xorguserhelper.h"
-
- #include <sys/types.h>
- #include <sys/ioctl.h>
-@@ -37,11 +41,19 @@
-
- namespace SDDM {
- UserSession::UserSession(HelperApp *parent)
-- : QProcess(parent) {
-- }
--
-- UserSession::~UserSession() {
--
-+ : QObject(parent)
-+ , m_process(new QProcess(this))
-+ , m_xorgUser(new XOrgUserHelper(this))
-+ {
-+ connect(m_process, QOverload<int>::of(&QProcess::finished), this, &UserSession::finished);
-+ connect(m_xorgUser, &XOrgUserHelper::displayChanged, this, [this, parent](const QString &display) {
-+ auto env = processEnvironment();
-+ env.insert(QStringLiteral("DISPLAY"), m_xorgUser->display());
-+ env.insert(QStringLiteral("XAUTHORITY"), m_xorgUser->xauthPath());
-+ setProcessEnvironment(env);
-+
-+ parent->displayServerStarted(display);
-+ });
- }
-
- bool UserSession::start() {
-@@ -49,21 +61,68 @@ namespace SDDM {
-
- setup();
-
-- if (env.value(QStringLiteral("XDG_SESSION_CLASS")) == QLatin1String("greeter")) {
-- QProcess::start(m_path);
-- } else if (env.value(QStringLiteral("XDG_SESSION_TYPE")) == QLatin1String("x11")) {
-- const QString cmd = QStringLiteral("%1 \"%2\"").arg(mainConfig.X11.SessionCommand.get()).arg(m_path);
-- qInfo() << "Starting:" << cmd;
-- QProcess::start(cmd);
-+ if (!m_displayServerCmd.isEmpty()) {
-+ m_xorgUser->setEnvironment(env);
-+ if (!m_xorgUser->start(m_displayServerCmd))
-+ return false;
-+ }
-+
-+ if (env.value(QStringLiteral("XDG_SESSION_TYPE")) == QLatin1String("x11")) {
-+ if (env.value(QStringLiteral("XDG_SESSION_CLASS")) == QLatin1String("greeter")) {
-+ qInfo() << "Starting X11 greeter session:" << m_path;
-+ auto args = QProcess::splitCommand(m_path);
-+ const auto program = args.takeFirst();
-+ m_process->start(program, args);
-+ } else {
-+ const QString cmd = QStringLiteral("%1 \"%2\"").arg(mainConfig.X11.SessionCommand.get()).arg(m_path);
-+ qInfo() << "Starting X11 user session:" << cmd;
-+ m_process->start(mainConfig.X11.SessionCommand.get(), QStringList{m_path});
-+ }
- } else if (env.value(QStringLiteral("XDG_SESSION_TYPE")) == QLatin1String("wayland")) {
- const QString cmd = QStringLiteral("%1 %2").arg(mainConfig.Wayland.SessionCommand.get()).arg(m_path);
-- qInfo() << "Starting:" << cmd;
-- QProcess::start(cmd);
-+ qInfo() << "Starting Wayland user session:" << cmd;
-+ m_process->start(mainConfig.Wayland.SessionCommand.get(), QStringList{m_path});
- } else {
- qCritical() << "Unable to run user session: unknown session type";
- }
-
-- return waitForStarted();
-+ if (m_process->waitForStarted()) {
-+ int vtNumber = processEnvironment().value(QStringLiteral("XDG_VTNR")).toInt();
-+ VirtualTerminal::jumpToVt(vtNumber, true);
-+ return true;
-+ }
-+
-+ return false;
-+ }
-+
-+ void UserSession::stop()
-+ {
-+ m_process->terminate();
-+ if (!m_process->waitForFinished(5000))
-+ m_process->kill();
-+
-+ if (!m_displayServerCmd.isEmpty())
-+ m_xorgUser->stop();
-+ }
-+
-+ QProcessEnvironment UserSession::processEnvironment() const
-+ {
-+ return m_process->processEnvironment();
-+ }
-+
-+ void UserSession::setProcessEnvironment(const QProcessEnvironment &env)
-+ {
-+ m_process->setProcessEnvironment(env);
-+ }
-+
-+ QString UserSession::displayServerCommand() const
-+ {
-+ return m_displayServerCmd;
-+ }
-+
-+ void UserSession::setDisplayServerCommand(const QString &command)
-+ {
-+ m_displayServerCmd = command;
- }
-
- void UserSession::setPath(const QString& path) {
-@@ -74,13 +133,22 @@ namespace SDDM {
- return m_path;
- }
-
-+ qint64 UserSession::processId() const
-+ {
-+ return m_process->processId();
-+ }
-+
- void UserSession::setupChildProcess() {
- // Session type
- QString sessionType = processEnvironment().value(QStringLiteral("XDG_SESSION_TYPE"));
--
-- // For Wayland sessions we leak the VT into the session as stdin so
-- // that it stays open without races
-- if (sessionType == QLatin1String("wayland")) {
-+ QString sessionClass = processEnvironment().value(QStringLiteral("XDG_SESSION_CLASS"));
-+ const bool hasDisplayServer = !m_displayServerCmd.isEmpty();
-+ const bool x11UserSession = sessionType == QLatin1String("x11") && sessionClass == QLatin1String("user");
-+ const bool waylandUserSession = sessionType == QLatin1String("wayland") && sessionClass == QLatin1String("user");
-+
-+ // When the display server is part of the session, we leak the VT into
-+ // the session as stdin so that it stays open without races
-+ if (hasDisplayServer || waylandUserSession) {
- // open VT and get the fd
- int vtNumber = processEnvironment().value(QStringLiteral("XDG_VTNR")).toInt();
- QString ttyString = QStringLiteral("/dev/tty%1").arg(vtNumber);
-@@ -226,74 +294,56 @@ namespace SDDM {
- qCritical() << "verify directory exist and has sufficient permissions";
- exit(Auth::HELPER_OTHER_ERROR);
- }
-- const QString homeDir = QString::fromLocal8Bit(pw.pw_dir);
--
-- //we cannot use setStandardError file as this code is run in the child process
-- //we want to redirect after we setuid so that the log file is owned by the user
--
-- // determine stderr log file based on session type
-- QString sessionLog = QStringLiteral("%1/%2")
-- .arg(homeDir)
-- .arg(sessionType == QLatin1String("x11")
-- ? mainConfig.X11.SessionLogFile.get()
-- : mainConfig.Wayland.SessionLogFile.get());
--
-- // create the path
-- QFileInfo finfo(sessionLog);
-- QDir().mkpath(finfo.absolutePath());
--
-- //swap the stderr pipe of this subprcess into a file
-- int fd = ::open(qPrintable(sessionLog), O_WRONLY | O_CREAT | O_TRUNC, 0600);
-- if (fd >= 0)
-- {
-- dup2 (fd, STDERR_FILENO);
-- ::close(fd);
-- } else {
-- qWarning() << "Could not open stderr to" << sessionLog;
-- }
--
-- //redirect any stdout to /dev/null
-- fd = ::open("/dev/null", O_WRONLY);
-- if (fd >= 0)
-- {
-- dup2 (fd, STDOUT_FILENO);
-- ::close(fd);
-- } else {
-- qWarning() << "Could not redirect stdout";
-- }
-
-- // set X authority for X11 sessions only
-- if (sessionType != QLatin1String("x11"))
-- return;
-- QString cookie = qobject_cast<HelperApp*>(parent())->cookie();
-- if (!cookie.isEmpty()) {
-- QString file = processEnvironment().value(QStringLiteral("XAUTHORITY"));
-- QString display = processEnvironment().value(QStringLiteral("DISPLAY"));
-- qDebug() << "Adding cookie to" << file;
-+ if (sessionClass != QLatin1String("greeter")) {
-+ //we cannot use setStandardError file as this code is run in the child process
-+ //we want to redirect after we setuid so that the log file is owned by the user
-
-+ // determine stderr log file based on session type
-+ QString sessionLog = QStringLiteral("%1/%2")
-+ .arg(QString::fromLocal8Bit(pw.pw_dir))
-+ .arg(sessionType == QLatin1String("x11")
-+ ? mainConfig.X11.SessionLogFile.get()
-+ : mainConfig.Wayland.SessionLogFile.get());
-
- // create the path
-- QFileInfo finfo(file);
-+ QFileInfo finfo(sessionLog);
- QDir().mkpath(finfo.absolutePath());
-
-- QFile file_handler(file);
-- file_handler.open(QIODevice::Append);
-- file_handler.close();
-+ //swap the stderr pipe of this subprcess into a file
-+ int fd = ::open(qPrintable(sessionLog), O_WRONLY | O_CREAT | O_TRUNC, 0600);
-+ if (fd >= 0)
-+ {
-+ dup2 (fd, STDERR_FILENO);
-+ ::close(fd);
-+ } else {
-+ qWarning() << "Could not open stderr to" << sessionLog;
-+ }
-
-- QString cmd = QStringLiteral("%1 -f %2 -q").arg(mainConfig.X11.XauthPath.get()).arg(file);
-+ //redirect any stdout to /dev/null
-+ fd = ::open("/dev/null", O_WRONLY);
-+ if (fd >= 0)
-+ {
-+ dup2 (fd, STDOUT_FILENO);
-+ ::close(fd);
-+ } else {
-+ qWarning() << "Could not redirect stdout";
-+ }
-+ }
-
-- // execute xauth
-- FILE *fp = popen(qPrintable(cmd), "w");
-+ // set X authority for X11 sessions only
-+ if (x11UserSession) {
-+ QString cookie = qobject_cast<HelperApp*>(parent())->cookie();
-+ if (!cookie.isEmpty()) {
-+ QString file = processEnvironment().value(QStringLiteral("XAUTHORITY"));
-+ QString display = processEnvironment().value(QStringLiteral("DISPLAY"));
-
-- // check file
-- if (!fp)
-- return;
-- fprintf(fp, "remove %s\n", qPrintable(display));
-- fprintf(fp, "add %s . %s\n", qPrintable(display), qPrintable(cookie));
-- fprintf(fp, "exit\n");
-+ // Create the path
-+ QFileInfo finfo(file);
-+ QDir().mkpath(finfo.absolutePath());
-
-- // close pipe
-- pclose(fp);
-+ XAuth::addCookieToFile(display, file, cookie);
-+ }
- }
- }
- }
-diff --git a/src/helper/UserSession.h b/src/helper/UserSession.h
-index c2383f068..11bff1d1b 100644
---- a/src/helper/UserSession.h
-+++ b/src/helper/UserSession.h
-@@ -23,41 +23,55 @@
- #define SDDM_AUTH_SESSION_H
-
- #include <QtCore/QObject>
--#include <QtCore/QString>
- #include <QtCore/QProcess>
-
- namespace SDDM {
- class HelperApp;
-- class UserSession : public QProcess
-+ class XOrgUserHelper;
-+ class UserSession : public QObject
- {
- Q_OBJECT
- public:
- explicit UserSession(HelperApp *parent);
-- virtual ~UserSession();
-
- bool start();
-+ void stop();
-+
-+ QProcessEnvironment processEnvironment() const;
-+ void setProcessEnvironment(const QProcessEnvironment &env);
-+
-+ QString displayServerCommand() const;
-+ void setDisplayServerCommand(const QString &command);
-
- void setPath(const QString &path);
- QString path() const;
-
- /*!
- \brief Sets m_cachedProcessId. Needed for getting the PID of a finished UserSession
- and calling HelperApp::utmpLogout
- \param pid The process ID
- */
- void setCachedProcessId(qint64 pid);
-
- /*!
- \brief Gets m_cachedProcessId
- \return The cached process ID
- */
- qint64 cachedProcessId();
-
-+ qint64 processId() const;
-+
-+ Q_SIGNALS:
-+ void finished(int exitCode);
-+
- protected:
- void setupChildProcess();
-
- private:
- QString m_path { };
- qint64 m_cachedProcessId;
-+ QProcess *m_process = nullptr;
-+ XOrgUserHelper *m_xorgUser = nullptr;
-+ QString m_displayServerCmd;
- };
- }
-
-diff --git a/src/helper/backend/PamBackend.cpp b/src/helper/backend/PamBackend.cpp
-index f86d77d63..c97056a66 100644
---- a/src/helper/backend/PamBackend.cpp
-+++ b/src/helper/backend/PamBackend.cpp
-@@ -248,7 +248,9 @@ namespace SDDM {
- }
-
- QProcessEnvironment sessionEnv = m_app->session()->processEnvironment();
-- if (sessionEnv.value(QStringLiteral("XDG_SESSION_TYPE")) == QLatin1String("x11")) {
-+ const auto sessionType = sessionEnv.value(QStringLiteral("XDG_SESSION_TYPE"));
-+ const auto sessionClass = sessionEnv.value(QStringLiteral("XDG_SESSION_CLASS"));
-+ if (sessionType == QLatin1String("x11") && (sessionClass == QLatin1String("user") || !m_displayServer)) {
- QString display = sessionEnv.value(QStringLiteral("DISPLAY"));
- if (!display.isEmpty()) {
- #ifdef PAM_XDISPLAY
-@@ -256,7 +258,7 @@ namespace SDDM {
- #endif
- m_pam->setItem(PAM_TTY, qPrintable(display));
- }
-- } else if (sessionEnv.value(QStringLiteral("XDG_SESSION_TYPE")) == QLatin1String("wayland")) {
-+ } else {
- QString tty = QStringLiteral("/dev/tty%1").arg(sessionEnv.value(QStringLiteral("XDG_VTNR")));
- m_pam->setItem(PAM_TTY, qPrintable(tty));
- }
-diff --git a/src/helper/xorguserhelper.cpp b/src/helper/xorguserhelper.cpp
-new file mode 100644
-index 000000000..573ed101f
---- /dev/null
-+++ b/src/helper/xorguserhelper.cpp
-@@ -0,0 +1,252 @@
-+/***************************************************************************
-+* Copyright (c) 2021 Pier Luigi Fiorini <pierluigi.fiorini@gmail.com>
-+*
-+* This program is free software; you can redistribute it and/or modify
-+* it under the terms of the GNU General Public License as published by
-+* the Free Software Foundation; either version 2 of the License, or
-+* (at your option) any later version.
-+*
-+* This program is distributed in the hope that it will be useful,
-+* but WITHOUT ANY WARRANTY; without even the implied warranty of
-+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-+* GNU General Public License for more details.
-+*
-+* You should have received a copy of the GNU General Public License
-+* along with this program; if not, write to the
-+* Free Software Foundation, Inc.,
-+* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-+***************************************************************************/
-+
-+#include <QCoreApplication>
-+#include <QFile>
-+#include <QStandardPaths>
-+
-+#include "Configuration.h"
-+
-+#include "xorguserhelper.h"
-+
-+#include <fcntl.h>
-+#include <unistd.h>
-+
-+namespace SDDM {
-+
-+XOrgUserHelper::XOrgUserHelper(QObject *parent)
-+ : QObject(parent)
-+{
-+}
-+
-+QProcessEnvironment XOrgUserHelper::environment() const
-+{
-+ return m_environment;
-+}
-+
-+void XOrgUserHelper::setEnvironment(const QProcessEnvironment &env)
-+{
-+ m_environment = env;
-+}
-+
-+QString XOrgUserHelper::display() const
-+{
-+ return m_display;
-+}
-+
-+QString XOrgUserHelper::xauthPath() const
-+{
-+ return m_xauth.authPath();
-+}
-+
-+bool XOrgUserHelper::start(const QString &cmd)
-+{
-+ // Create xauthority
-+ m_xauth.setAuthDirectory(m_environment.value(QStringLiteral("XDG_RUNTIME_DIR")));
-+ m_xauth.setup();
-+
-+ // Start server process
-+ if (!startServer(cmd))
-+ return false;
-+
-+ // Setup display
-+ startDisplayCommand();
-+
-+ return true;
-+}
-+
-+void XOrgUserHelper::stop()
-+{
-+ if (m_serverProcess) {
-+ qInfo("Stopping server...");
-+ m_serverProcess->terminate();
-+ if (!m_serverProcess->waitForFinished(5000))
-+ m_serverProcess->kill();
-+ m_serverProcess->deleteLater();
-+ m_serverProcess = nullptr;
-+
-+ displayFinished();
-+ }
-+}
-+
-+bool XOrgUserHelper::startProcess(const QString &cmd,
-+ const QProcessEnvironment &env,
-+ QProcess **p)
-+{
-+ auto args = QProcess::splitCommand(cmd);
-+ const auto program = args.takeFirst();
-+
-+ // Make sure to forward the input of this process into the Xorg
-+ // server, otherwise it will complain that only console users are allowed
-+ auto *process = new QProcess(this);
-+ process->setInputChannelMode(QProcess::ForwardedInputChannel);
-+ process->setProcessChannelMode(QProcess::ForwardedChannels);
-+ process->setProcessEnvironment(env);
-+
-+ connect(process, QOverload<int, QProcess::ExitStatus>::of(&QProcess::finished),
-+ process, [](int exitCode, QProcess::ExitStatus exitStatus) {
-+ if (exitCode != 0 || exitStatus != QProcess::NormalExit)
-+ QCoreApplication::instance()->quit();
-+ });
-+
-+ process->start(program, args);
-+ if (!process->waitForStarted(10000)) {
-+ qWarning("Failed to start \"%s\": %s",
-+ qPrintable(cmd),
-+ qPrintable(process->errorString()));
-+ return false;
-+ }
-+
-+ if (p)
-+ *p = process;
-+
-+ return true;
-+}
-+
-+bool XOrgUserHelper::startServer(const QString &cmd)
-+{
-+ QString serverCmd = cmd;
-+
-+ // Create pipe for communicating with X server
-+ // 0 == read from X, 1 == write to X
-+ int pipeFds[2];
-+ if (::pipe(pipeFds) != 0) {
-+ qCritical("Could not create pipe to start X server");
-+ return false;
-+ }
-+
-+ // Do not leak the read endpoint to the X server process
-+ fcntl(pipeFds[0], F_SETFD, FD_CLOEXEC);
-+
-+ // Server environment
-+ // Not setting XORG_RUN_AS_USER_OK=1 will make Xorg require root privileges
-+ // under Fedora and all distros that use their patch.
-+ // https://src.fedoraproject.org/rpms/xorg-x11-server/blob/rawhide/f/0001-Fedora-hack-Make-the-suid-root-wrapper-always-start-.patch
-+ // https://fedoraproject.org/wiki/Changes/XorgWithoutRootRights
-+ QProcessEnvironment serverEnv = m_environment;
-+ serverEnv.insert(QStringLiteral("XORG_RUN_AS_USER_OK"), QStringLiteral("1"));
-+
-+ // Append xauth and display fd to the command
-+ auto args = QStringList()
-+ << QStringLiteral("-auth") << m_xauth.authPath()
-+ << QStringLiteral("-displayfd") << QString::number(pipeFds[1]);
-+
-+ // Append VT from environment
-+ args << QStringLiteral("vt%1").arg(serverEnv.value(QStringLiteral("XDG_VTNR")));
-+
-+ // Log to stdout
-+ args << QStringLiteral("-logfile") << QStringLiteral("/dev/null");
-+
-+ // Command string
-+ serverCmd += QLatin1Char(' ') + args.join(QLatin1Char(' '));
-+
-+ // Start the server process
-+ qInfo("Running server: %s", qPrintable(serverCmd));
-+ if (!startProcess(serverCmd, serverEnv, &m_serverProcess)) {
-+ ::close(pipeFds[0]);
-+ return false;
-+ }
-+
-+ // Close the other side of pipe in our process, otherwise reading
-+ // from it may stuck even X server exit
-+ ::close(pipeFds[1]);
-+
-+ // Read the display number from the pipe
-+ QFile readPipe;
-+ if (!readPipe.open(pipeFds[0], QIODevice::ReadOnly)) {
-+ qCritical("Failed to open pipe to start X Server");
-+ ::close(pipeFds[0]);
-+ return false;
-+ }
-+ QByteArray displayNumber = readPipe.readLine();
-+ if (displayNumber.size() < 2) {
-+ // X server gave nothing (or a whitespace)
-+ qCritical("Failed to read display number from pipe");
-+ ::close(pipeFds[0]);
-+ return false;
-+ }
-+ displayNumber.prepend(QByteArray(":"));
-+ displayNumber.remove(displayNumber.size() -1, 1); // trim trailing whitespace
-+ m_display = QString::fromLocal8Bit(displayNumber);
-+ qDebug("X11 display: %s", qPrintable(m_display));
-+ Q_EMIT displayChanged(m_display);
-+
-+ // Generate xauthority file
-+ // For the X server's copy, the display number doesn't matter.
-+ // An empty file would result in no access control!
-+ if (!m_xauth.addCookie(m_display)) {
-+ qCritical("Failed to write xauth file");
-+ return false;
-+ }
-+
-+ // Close our pipe
-+ ::close(pipeFds[0]);
-+
-+ return true;
-+}
-+
-+void XOrgUserHelper::startDisplayCommand()
-+{
-+ auto env = QProcessEnvironment::systemEnvironment();
-+ env.insert(QStringLiteral("DISPLAY"), m_display);
-+ env.insert(QStringLiteral("XAUTHORITY"), m_xauth.authPath());
-+
-+ // Set cursor
-+ qInfo("Setting default cursor...");
-+ QProcess *setCursor = nullptr;
-+ if (startProcess(QStringLiteral("xsetroot -cursor_name left_ptr"), env, &setCursor)) {
-+ if (!setCursor->waitForFinished(1000)) {
-+ qWarning() << "Could not setup default cursor";
-+ setCursor->kill();
-+ }
-+ setCursor->deleteLater();
-+ }
-+
-+ // Display setup script
-+ auto cmd = mainConfig.X11.DisplayCommand.get();
-+ qInfo("Running display setup script: %s", qPrintable(cmd));
-+ QProcess *displayScript = nullptr;
-+ if (startProcess(cmd, env, &displayScript)) {
-+ if (!displayScript->waitForFinished(30000))
-+ displayScript->kill();
-+ displayScript->deleteLater();
-+ }
-+}
-+
-+void XOrgUserHelper::displayFinished()
-+{
-+ auto env = QProcessEnvironment::systemEnvironment();
-+ env.insert(QStringLiteral("DISPLAY"), m_display);
-+ env.insert(QStringLiteral("XAUTHORITY"), m_xauth.authPath());
-+ env.insert(QStringLiteral("QT_QPA_PLATFORM"), QStringLiteral("xcb"));
-+
-+ auto cmd = mainConfig.X11.DisplayStopCommand.get();
-+ qInfo("Running display stop script: %s", qPrintable(cmd));
-+ QProcess *displayStopScript = nullptr;
-+ if (startProcess(cmd, env, &displayStopScript)) {
-+ if (!displayStopScript->waitForFinished(5000))
-+ displayStopScript->kill();
-+ displayStopScript->deleteLater();
-+ }
-+
-+ // Remove xauthority file
-+ QFile::remove(m_xauth.authPath());
-+}
-+
-+} // namespace SDDM
-diff --git a/src/helper/xorguserhelper.h b/src/helper/xorguserhelper.h
-new file mode 100644
-index 000000000..73536965a
---- /dev/null
-+++ b/src/helper/xorguserhelper.h
-@@ -0,0 +1,64 @@
-+/***************************************************************************
-+* Copyright (c) 2021 Pier Luigi Fiorini <pierluigi.fiorini@gmail.com>
-+*
-+* This program is free software; you can redistribute it and/or modify
-+* it under the terms of the GNU General Public License as published by
-+* the Free Software Foundation; either version 2 of the License, or
-+* (at your option) any later version.
-+*
-+* This program is distributed in the hope that it will be useful,
-+* but WITHOUT ANY WARRANTY; without even the implied warranty of
-+* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-+* GNU General Public License for more details.
-+*
-+* You should have received a copy of the GNU General Public License
-+* along with this program; if not, write to the
-+* Free Software Foundation, Inc.,
-+* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
-+***************************************************************************/
-+
-+#ifndef XORGUSERHELPER_H
-+#define XORGUSERHELPER_H
-+
-+#include <QProcess>
-+
-+#include "XAuth.h"
-+
-+namespace SDDM {
-+
-+class XOrgUserHelper : public QObject
-+{
-+ Q_OBJECT
-+ Q_PROPERTY(QString display READ display NOTIFY displayChanged)
-+public:
-+ explicit XOrgUserHelper(QObject *parent = nullptr);
-+
-+ QProcessEnvironment environment() const;
-+ void setEnvironment(const QProcessEnvironment &env);
-+
-+ QString display() const;
-+
-+ QString xauthPath() const;
-+
-+ bool start(const QString &cmd);
-+ void stop();
-+
-+Q_SIGNALS:
-+ void displayChanged(const QString &display);
-+
-+private:
-+ QString m_display = QStringLiteral(":0");
-+ XAuth m_xauth;
-+ QProcessEnvironment m_environment;
-+ QProcess *m_serverProcess = nullptr;
-+
-+ bool startProcess(const QString &cmd, const QProcessEnvironment &env,
-+ QProcess **p = nullptr);
-+ bool startServer(const QString &cmd);
-+ void startDisplayCommand();
-+ void displayFinished();
-+};
-+
-+} // namespace SDDM
-+
-+#endif // XORGUSERHELPER_H