[apparmor] [PATCH 1/3] Allow Totem to read /etc/wildmidi/wildmidi.cfg.

intrigeri intrigeri at debian.org
Wed Oct 9 14:33:47 UTC 2013


Hi again,

Steve Beattie wrote (09 Oct 2013 13:39:17 GMT) :
> Hrm. Actually, looking more closely at all three permissions in this
> patch series, I think they're all caused by gstreamer, at least from
> looking at the reverse dependencies on ubuntu 13.10. All three should
> probably move to the audio abstraction or a gstreamer specific
> abstraction, I think.

Now that I think of it, I do agree. Will do, stay tuned.

In the current state of the profiles we have (e.g. Totem already uses
the audio abstraction), I see little gain in creating a dedicated
gstreamer abstraction. So, unless anyone raises a reason that
I missed, I guess I'll just move these few lines to the audio
abstraction. Thoughts?

(I had totally overlooked the existence of an audio abstraction: when
I grepped for "gst" in my system's apparmor.d, I found other profiles
that allowed running gst-plugin-scanner, so I didn't look any further,
and did just the same for Totem. Sorry I didn't put this all into
perspective. This mistake should hopefully teach me and help improve
my future contributions.)

Cheers,
-- 
  intrigeri
  | GnuPG key @ https://gaffer.ptitcanardnoir.org/intrigeri/intrigeri.asc
  | OTR fingerprint @ https://gaffer.ptitcanardnoir.org/intrigeri/otr.asc



More information about the AppArmor mailing list