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

Proposed by Alberto Mardegan on 2016-06-07
Status: Merged
Approved by: Paweł Stołowski on 2016-06-07
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 2016-06-07 Approve on 2016-06-07
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.
Paweł Stołowski (stolowski) wrote :

Yes, that does it, thanks!

review: Approve
23. By Alberto Mardegan on 2016-06-07

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
1=== modified file 'src/access-control-manager.cpp'
2--- src/access-control-manager.cpp 2015-01-28 15:30:34 +0000
3+++ src/access-control-manager.cpp 2016-06-07 09:54:43 +0000
4@@ -62,7 +62,10 @@
5 securityContext == QLatin1String("*") ||
6 /* Temporary workaround for
7 * https://bugs.launchpad.net/bugs/1376445 */
8- appId == QStringLiteral("unconfined"));
9+ appId == QStringLiteral("unconfined") ||
10+ /* Let the dash access any account:
11+ * https://bugs.launchpad.net/bugs/1589444 */
12+ appId == QStringLiteral("unity8-dash"));
13 qDebug() << "Process" << appId << "access to" << securityContext <<
14 (allowed ? "ALLOWED" : "DENIED");
15 return allowed;

Subscribers

People subscribed via source and target branches