Code review comment for lp:~intrigeri/apparmor-profiles/gdm-pulseaudio-v2

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