Merge lp:~mardy/signon-apparmor-extension/dash-1589444 into lp:signon-apparmor-extension

Proposed by Alberto Mardegan
Status: Merged
Approved by: Paweł Stołowski
Approved revision: 22
Merged at revision: 23
Proposed branch: lp:~mardy/signon-apparmor-extension/dash-1589444
Merge into: lp:signon-apparmor-extension
Diff against target: 15 lines (+4/-1)
1 file modified
src/access-control-manager.cpp (+4/-1)
To merge this branch: bzr merge lp:~mardy/signon-apparmor-extension/dash-1589444
Reviewer Review Type Date Requested Status
Paweł Stołowski (community) Approve
PS Jenkins bot continuous-integration Pending
Review via email: mp+296640@code.launchpad.net

Commit message

Allow the Dash to access any account

Description of the change

Allow the Dash to access any account

To post a comment you must log in.
Revision history for this message
Paweł Stołowski (stolowski) wrote :

Yes, that does it, thanks!

review: Approve
23. By Alberto Mardegan

From trunk

No-change test rebuild for g++5 ABI transition

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/access-control-manager.cpp'
--- src/access-control-manager.cpp 2015-01-28 15:30:34 +0000
+++ src/access-control-manager.cpp 2016-06-07 09:54:43 +0000
@@ -62,7 +62,10 @@
62 securityContext == QLatin1String("*") ||62 securityContext == QLatin1String("*") ||
63 /* Temporary workaround for63 /* Temporary workaround for
64 * https://bugs.launchpad.net/bugs/1376445 */64 * https://bugs.launchpad.net/bugs/1376445 */
65 appId == QStringLiteral("unconfined"));65 appId == QStringLiteral("unconfined") ||
66 /* Let the dash access any account:
67 * https://bugs.launchpad.net/bugs/1589444 */
68 appId == QStringLiteral("unity8-dash"));
66 qDebug() << "Process" << appId << "access to" << securityContext <<69 qDebug() << "Process" << appId << "access to" << securityContext <<
67 (allowed ? "ALLOWED" : "DENIED");70 (allowed ? "ALLOWED" : "DENIED");
68 return allowed;71 return allowed;

Subscribers

People subscribed via source and target branches