Differential D22889 Diff 63974 src/util/externalcommand_polkitbackend.cpp

Changeset View

Standalone View

View Options

src/util/externalcommand_polkitbackend.cpp

  • This file was added.
1​/*************************************************************************
2​ * Copyright (C) 2019 by Shubham <aryan100jangid@gmail.com> *
3​ * *
4​ * This program is free software; you can redistribute it and/or *
5​ * modify it under the terms of the GNU General Public License as *
6​ * published by the Free Software Foundation; either version 3 of *
7​ * the License, or (at your option) any later version. *
8​ * *
9​ * This program is distributed in the hope that it will be useful, *
10​ * but WITHOUT ANY WARRANTY; without even the implied warranty of *
11​ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the *
12​ * GNU General Public License for more details. *
13​ * *
14​ * You should have received a copy of the GNU General Public License *
15​ * along with this program. If not, see <http://www.gnu.org/licenses/>.*
16​ *************************************************************************/
17​
18​#include "util/externalcommand_polkitbackend.h"
19​
20​#include <QApplication>
21​#include <QDebug>
22​#include <QDBusConnection>
23​#include <QDBusConnectionInterface>
24​#include <QEventLoop>
25​#include <QString>
26​#include <QTimer>
27​#include <QWidget>
28​
29​#include <PolkitQt1/Authority>
30​#include <PolkitQt1/Subject>
31​
32​using namespace PolkitQt1;
33​
34​Authority::Result Auth::PolkitEventLoop::m_result = Authority::No;
35​
36​namespace Auth
37​{
38​
39​PolkitEventLoop::PolkitEventLoop(QObject *parent)
40​ : QEventLoop(qobject_cast<QEventLoop *>(parent))
41​{
42​
43​}
44​
45​PolkitEventLoop::~PolkitEventLoop()
46​{
47​
48​}
49​
50​Authority::Result PolkitEventLoop::result() const
51​{
52​ return m_result;
53​}
54​
55​PolkitQt1Backend::PolkitQt1Backend()
56​ : m_flyingActions(false)
57​{
58​ // Connect various useful Polkit signals
59​ connect(Authority::instance(), &Authority::configChanged, this, &Auth::PolkitQt1Backend::authStatusChanged);
60​ connect(Authority::instance(), &Authority::consoleKitDBChanged, this, &Auth::PolkitQt1Backend::authStatusChanged);
61​
62​ m_flyingActions = true;
63​}
64​
65​PolkitQt1Backend::~PolkitQt1Backend()
66​{
67​
68​}
69​
70​void PolkitQt1Backend::initPolkitAgent(const QString &action, QWidget *parent /*= nullptr*/) const
71​{
72​ if (!parent) {
73​ qWarning() << "Parent widget does not exists, can not proceed further";
74​ return;
75​ }
76​
77​ // Check if we are running terminal session or GUI session
78​ if (!qApp) {
79​ qWarning() << "We are running a TTY (Terminal) session";
80​ qDebug() << "Can not proceed further since we do not support Text based Polkit Authentication Agent";
81​ return;
82​ }
83​
84​ // Get the dialog parent window Id
85​ quint64 parentWindowID = parent->effectiveWinId();
86​
87​ // Make a call to the KDE polkit Authentication Agent asking for it's services
88​ QDBusMessage callAgent = QDBusMessage::createMethodCall(QLatin1String("org.kde.polkit-kde-authentication-agent-1"), QLatin1String("/org/kde/Polkit1AuthAgent"), QLatin1String("org.kde.Polkit1AuthAgent"),
89​ QLatin1String("setWIdForAction"));
90​
91​ callAgent << action;
92​ callAgent << parentWindowID;
93​
94​ QDBusPendingCall call = QDBusConnection::sessionBus().asyncCall(callAgent);
95​
96​ auto watcher = new QDBusPendingCallWatcher(call);
97​
98​ connect(watcher, &QDBusPendingCallWatcher::finished, this, [this, action, watcher](){
99​ watcher->deleteLater();
100​ const QDBusMessage reply = watcher->reply();
101​
102​ if (reply.type() == QDBusMessage::ErrorMessage) {
103​ qWarning() << "Could not call the Authentication Agent, Error:" << reply.errorMessage();
104​ }
105​ });
106​}
107​
108​Authority::Result PolkitQt1Backend::actionStatus(const QString &action, const QByteArray &callerID) const
stikonasUnsubmitted
Done

Why do we need connect to finished signal if in the line above you wait for call to finish with call.waitForFinished() that function blocks until call is done.

stikonas: Why do we need connect to finished signal if in the line above you wait for call to finish with…
109​{
110​ SystemBusNameSubject subject(QString::fromUtf8(callerID));
111​
112​ auto authority = Authority::instance();
113​
114​ PolkitEventLoop::m_result = authority->checkAuthorizationSync(action, subject, Authority::AllowUserInteraction);
115​
116​ if (authority->hasError()) {
117​ qDebug() << "Encountered error while checking action status, Error code:" << authority->lastError() << "\n";
118​ qDebug() << "Error Details:" << authority->errorDetails();
119​ authority->clearError();
120​ }
121​
122​ return PolkitEventLoop::m_result;
123​}
124​
125​QByteArray PolkitQt1Backend::callerID() const
126​{
127​ return QDBusConnection::systemBus().baseService().toUtf8();
128​}
129​
130​bool PolkitQt1Backend::authorizeAction(const QString &action, const QByteArray &callerID)
131​{
132​ // Set m_result here, otherwise there will be wrong log message displayed inside externalcommand.cpp line 383
133​
134​ SystemBusNameSubject subject(QString::fromUtf8(callerID));
135​
136​ auto authority = Authority::instance();
137​
138​ PolkitEventLoop event;
139​ event.processEvents();
140​
141​ connect(authority, &Authority::checkAuthorizationFinished, &event, &PolkitEventLoop::quit);
142​ authority->checkAuthorization(action, subject, Authority::AllowUserInteraction);
143​
144​ event.exec();
145​
146​ if (authority->hasError()) {
147​ qWarning() << "Encountered error while checking authorization, Error code:" << authority->lastError() << "\n";
148​ qDebug() << "Error details:" << authority->errorDetails();
149​
150​ // Clear all the errors from the buffer so that hasError() does not give previous error as a result when called later
151​ authority->clearError();
152​ }
153​
154​ if (/*event.result() PolkitEventLoop::m_result*/ actionStatus(action, callerID) == Authority::Yes) {
155​ return true;
156​ } else {
157​ return false;
stikonasUnsubmitted
Done

What sets m_result? in the connect call above you simply pass result to QEventLoop's function quit without processing it.

stikonas: What sets `m_result`? in the connect call above you simply pass result to QEventLoop's function…
shubhamAuthorUnsubmitted
Not Done

Actually yes, it is currently broke, however it does not matter, because I am just using this function to show the auth agent and if you notice line 383 externalcommand.cpp, to display some log message. Oh, now I get why it was reporting ""Unable to obtain Administrative privileges, the action can not be executed!!";"

shubham: Actually yes, it is currently broke, however it does not matter, because I am just using this…
158​ }
159​}
160​
161​bool PolkitQt1Backend::revokeAuthorization(const QString &action, const QByteArray &callerID)
162​{
163​ Q_UNUSED(action)
164​
165​ SystemBusNameSubject subject(QString::fromUtf8(callerID));
166​
167​ auto authority = Authority::instance();
168​
169​ return authority->revokeTemporaryAuthorizationsSync(subject);
170​}
171​
172​void PolkitQt1Backend::authStatusChanged()
173​{
174​ for (auto it = m_cachedResults.begin(); it != m_cachedResults.end(); ++it) {
175​ const QString action = it.key();
176​ QByteArray pid = QDBusConnection::systemBus().baseService().toUtf8();
177​ if (it.value() != actionStatus(action, pid)) {
178​ *it = actionStatus(action, pid);
179​ }
180​ }
181​
182​ // Force updating known actions
183​ Authority::instance()->enumerateActions();
184​ m_flyingActions = true;
185​}
186​
187​} // namespace Auth