Merge lp:~intrigeri/apparmor-profiles/pidgin-vs-gstreamer-1.6 into lp:apparmor-profiles

Proposed by intrigeri
Status: Merged
Merged at revision: 154
Proposed branch: lp:~intrigeri/apparmor-profiles/pidgin-vs-gstreamer-1.6
Merge into: lp:apparmor-profiles
Diff against target: 14 lines (+4/-0)
1 file modified
ubuntu/16.04/usr.bin.pidgin (+4/-0)
To merge this branch: bzr merge lp:~intrigeri/apparmor-profiles/pidgin-vs-gstreamer-1.6
Reviewer Review Type Date Requested Status
Christian Boltz Approve
Review via email: mp+277499@code.launchpad.net

Description of the change

To post a comment you must log in.
Revision history for this message
intrigeri (intrigeri) wrote :

6 weeks later: ping?

Revision history for this message
Christian Boltz (cboltz) wrote :

I never used pidgin, but the changes look sane and valid, therefore merged into r154.

I tend to think that all gstreamer-related rules should go into abstractions/gstreamer, but
a) that's something for a separate cleanup patch and
b) I don't really know gstreamer, so I'm not sure if all applications using gstreamer also need those permissions (if yes, moving to the abstraction makes sense)

review: Approve
Revision history for this message
intrigeri (intrigeri) wrote :

Christian Boltz wrote (30 Dec 2015 21:03:35 GMT) :
> I tend to think that all gstreamer-related rules should go into abstractions/gstreamer,

Agreed, added to my personal todo list for some day I don't feel like
doing this kind of things.

Cheers,
--
intrigeri

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'ubuntu/16.04/usr.bin.pidgin'
2--- ubuntu/16.04/usr.bin.pidgin 2015-10-27 01:49:53 +0000
3+++ ubuntu/16.04/usr.bin.pidgin 2015-11-14 13:30:39 +0000
4@@ -41,6 +41,10 @@
5
6 owner @{HOME}/.gstreamer*/ rw,
7 owner @{HOME}/.gstreamer*/** rw,
8+ owner @{HOME}/.cache/gstreamer-[0-9]*.[0-9]*/ rw,
9+ owner @{HOME}/.cache/gstreamer-[0-9]*.[0-9]*/registry.*.bin rw,
10+ owner @{HOME}/.cache/gstreamer-[0-9]*.[0-9]*/registry.*.bin.tmp* rw,
11+ /usr/lib/@{multiarch}/gstreamer[0-9]*.[0-9]*/gstreamer-[0-9]*.[0-9]*/gst-plugin-scanner Pix -> gst_plugin_scanner,
12 owner @{HOME}/.purple/ rw,
13 owner @{HOME}/.purple/** rwk,
14 owner @{HOME}/.config/indicators/ rw,

Subscribers

People subscribed via source and target branches

to status/vote changes: