Merge lp:~intrigeri/apparmor-profiles/gdm-pulseaudio-v2 into lp:apparmor-profiles

[intrigeri]

[Seth Arnold (seth-arnold)]

On Sat, Jul 26, 2014 at 03:15:33PM -0000, intrigeri wrote:
> intrigeri has proposed merging lp:~intrigeri/apparmor-profiles/gdm-pulseaudio-v2 into lp:apparmor-profiles.

Looks good to me, thanks.

Acked-by: Seth Arnold <email address hidden>

> === modified file 'ubuntu/14.10/usr.bin.pulseaudio'
> --- ubuntu/14.10/usr.bin.pulseaudio 2014-07-17 15:33:08 +0000
> +++ ubuntu/14.10/usr.bin.pulseaudio 2014-07-26 15:14:38 +0000
> @@ -21,8 +21,9 @@
>
> @{HOME}/.esd_auth rwk,
> @{HOME}/.pulse-cookie rwk,
> - @{HOME}/.pulse/ rw,
> - @{HOME}/.pulse/* rw,
> + @{HOME}/.config/pulse/cookie rwk,
> + @{HOME}/{.config/pulse,.pulse}/ rw,
> + @{HOME}/{.config/pulse,.pulse}/* rw,
>
> /run/pulse/ rw,
> /run/pulse/.pulse-cookie rwk,
> @@ -31,6 +32,10 @@
> /run/pulse/pid rwk,
> /run/udev/data/+sound:card* r,
>
> + # logind
> + /run/systemd/users/* r,
> + /run/user/*/dconf/user k,
> +
> /sys/bus/ r,
> /sys/class/ r,
> /sys/class/sound/ r,
> @@ -50,6 +55,10 @@
> /usr/lib/pulse-[2-9].[0-9]/modules/*.so mr,
> /usr/lib/pulseaudio/pulse/gconf-helper Cx,
>
> + owner /var/lib/gdm3/.config/pulse/ rw,
> + owner /var/lib/gdm3/.config/pulse/* rw,
> + owner /var/lib/gdm3/.config/pulse/cookie rwk,
> +
> owner /var/lib/lightdm/.Xauthority r,
> owner /var/lib/lightdm/.esd_auth rwk,
> owner /var/lib/lightdm/.pulse-cookie rwk,
>

> --
> AppArmor mailing list
> <email address hidden>
> Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor

[intrigeri (intrigeri)]

> Looks good to me, thanks.

Thanks for the review.

> Acked-by: Seth Arnold <email address hidden>

Sorry, newbie question, I'm not fully familiar with the review'n'merge process yet: what's the next thing to do to get this branch merged, and who is supposed to do it?

[Seth Arnold (seth-arnold)]

> > Looks good to me, thanks.
>
> Thanks for the review.
>
> > Acked-by: Seth Arnold <email address hidden>
>
> Sorry, newbie question, I'm not fully familiar with the review'n'merge process
> yet: what's the next thing to do to get this branch merged, and who is
> supposed to do it?

Thanks for pointing this out; I assumed that you had checkin privs and would check it in yourself. :) We can fix that later I guess...

Thanks

=== modified file 'ubuntu/14.10/usr.bin.pulseaudio'
--- ubuntu/14.10/usr.bin.pulseaudio	2014-07-17 15:33:08 +0000
+++ ubuntu/14.10/usr.bin.pulseaudio	2014-07-26 15:14:38 +0000
@@ -21,8 +21,9 @@
 
   @{HOME}/.esd_auth rwk,
   @{HOME}/.pulse-cookie rwk,
-  @{HOME}/.pulse/ rw,
-  @{HOME}/.pulse/* rw,
+  @{HOME}/.config/pulse/cookie rwk,
+  @{HOME}/{.config/pulse,.pulse}/ rw,
+  @{HOME}/{.config/pulse,.pulse}/* rw,
 
   /run/pulse/ rw,
   /run/pulse/.pulse-cookie rwk,
@@ -31,6 +32,10 @@
   /run/pulse/pid rwk,
   /run/udev/data/+sound:card* r,
 
+  # logind
+  /run/systemd/users/* r,
+  /run/user/*/dconf/user k,
+
   /sys/bus/ r,
   /sys/class/ r,
   /sys/class/sound/ r,
@@ -50,6 +55,10 @@
   /usr/lib/pulse-[2-9].[0-9]/modules/*.so mr,
   /usr/lib/pulseaudio/pulse/gconf-helper Cx,
 
+  owner /var/lib/gdm3/.config/pulse/ rw,
+  owner /var/lib/gdm3/.config/pulse/* rw,
+  owner /var/lib/gdm3/.config/pulse/cookie rwk,
+
   owner /var/lib/lightdm/.Xauthority r,
   owner /var/lib/lightdm/.esd_auth rwk,
   owner /var/lib/lightdm/.pulse-cookie rwk,