Differential D6197 Diff 16738 src/ioslaves/file/file.h

Changeset View

Standalone View

View Options

src/ioslaves/file/file.h

Context not available.
100 100​
101 void fileSystemFreeSpace(const QUrl &url); // KF6 TODO: Turn into virtual method in SlaveBase 101​ void fileSystemFreeSpace(const QUrl &url); // KF6 TODO: Turn into virtual method in SlaveBase
102 102​
103​ virtual bool execWithElevatedPrivilege(const QString &action, const QString &subAction,
104​ const QVariant &args);
103private: 105​private:
104 mutable QHash<KUserId, QString> mUsercache; 106​ mutable QHash<KUserId, QString> mUsercache;
105 mutable QHash<KGroupId, QString> mGroupcache; 107​ mutable QHash<KGroupId, QString> mGroupcache;
Context not available.
elvisangelaccioUnsubmitted
Done

endProvidiledgeOperation() is more descriptive, no?

elvisangelaccio: `endProvidiledgeOperation()` is more descriptive, no?
eliaspUnsubmitted
Done

I find execWithRoot() to be a bit misleading, as the goal shouldn't be to always elevate to root's privileges but to only what's required to execute the specific operation (e.g. browse /home/someotheruser doesn't need root's privileges but only someotheruser's privileges).

Always elevating to root is IMHO by far too permissive.

eliasp: I find `execWithRoot()` to be a bit misleading, as the goal shouldn't be to always elevate to…
chinmoyrAuthorUnsubmitted
Done

I agree with you on the part that execWithRoot() is slightly misleading. So I changed it to execWithElevatedPrivilege(). And acquiring somotheruser's privileges still requires elevated privilege does it not?

chinmoyr: I agree with you on the part that execWithRoot() is slightly misleading. So I changed it to…