From a5017377d2c80de65111e0d8989cbb05c00732d4 Mon Sep 17 00:00:00 2001 From: Ivailo Monev Date: Thu, 31 Aug 2023 19:18:56 +0300 Subject: [PATCH] kdesudo: store the DISPLAY environment variable as QString otherwise it will be converted as if it is ASCII later on several times Signed-off-by: Ivailo Monev --- kdesudo/kdesudo.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/kdesudo/kdesudo.cpp b/kdesudo/kdesudo.cpp index 08278a23..818d3e6b 100644 --- a/kdesudo/kdesudo.cpp +++ b/kdesudo/kdesudo.cpp @@ -137,7 +137,7 @@ KdeSudo::KdeSudo(const QString &icon, const QString &appname) // 'man xauth' for more info on xauth cookies. m_tmpName = KTemporaryFile::filePath("/tmp/kdesudo-XXXXXXXXXX-xauth"); - QByteArray disp = qgetenv("DISPLAY"); + const QString disp = QString::fromLocal8Bit(qgetenv("DISPLAY")); if (disp.isEmpty()) { kError() << "$DISPLAY is not set."; exit(1); @@ -151,7 +151,7 @@ KdeSudo::KdeSudo(const QString &icon, const QString &appname) xauth_ext.setStandardOutputProcess(&xauth_merge); // Start the first - xauth_ext.start("xauth", QStringList() << "extract" << "-" << QString::fromLocal8Bit(disp), QIODevice::ReadOnly); + xauth_ext.start("xauth", QStringList() << "extract" << "-" << disp, QIODevice::ReadOnly); if (!xauth_ext.waitForStarted()) { return; }