AppArmor Profiles

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

pidgin-vs-gstreamer-1.6
Merge into master

Proposed by intrigeri on 2015-11-14

Status:	Merged
Merged at revision:	154
Proposed branch:	lp://qastaging/~intrigeri/apparmor-profiles/pidgin-vs-gstreamer-1.6
Merge into:	lp://qastaging/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://qastaging/~intrigeri/apparmor-profiles/pidgin-vs-gstreamer-1.6
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Christian Boltz		2015-11-14	Approve on 2015-12-30
Review via email: mp+277499@code.qastaging.launchpad.net

Description of the change

Fixes https://bugs.debian.org/802791

Revision history for this message

intrigeri (intrigeri) wrote on 2015-12-30:

6 weeks later: ping?

Revision history for this message

Christian Boltz (cboltz) wrote on 2015-12-30:

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 on 2016-02-10:

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