<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jan 30, 2020 at 8:06 AM Michal Privoznik <<a href="mailto:mprivozn@redhat.com" target="_blank">mprivozn@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Even though we construct a domain specific profile for each<br>
domain we start (which should cover domain specific paths), there<br>
is also another file that is included from the profile and which<br>
contains domain agnostic paths (e.g. to cover libraries that qemu<br>
links with). The paths in the file are split into blocks divided<br>
by comments. Sort the paths in each block individually (ignoring<br>
case sensitivity).<br>
<br>
Signed-off-by: Michal Privoznik <<a href="mailto:mprivozn@redhat.com" target="_blank">mprivozn@redhat.com</a>><br></blockquote><div><br></div><div>Acked-by: Christian Ehrhardt <<a href="mailto:christian.ehrhardt@canonical.com">christian.ehrhardt@canonical.com</a>><br></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
---<br>
 src/security/apparmor/libvirt-qemu | 76 +++++++++++++++---------------<br>
 1 file changed, 38 insertions(+), 38 deletions(-)<br>
<br>
diff --git a/src/security/apparmor/libvirt-qemu b/src/security/apparmor/libvirt-qemu<br>
index d33348aa05..2291829270 100644<br>
--- a/src/security/apparmor/libvirt-qemu<br>
+++ b/src/security/apparmor/libvirt-qemu<br>
@@ -22,8 +22,8 @@<br>
   signal (receive) peer=libvirtd,<br>
   signal (receive) peer=/usr/sbin/libvirtd,<br>
<br>
-  /dev/net/tun rw,<br>
   /dev/kvm rw,<br>
+  /dev/net/tun rw,<br>
   /dev/ptmx rw,<br>
   @{PROC}/*/status r,<br>
   # When qemu is signaled to terminate, it will read cmdline of signaling<br>
@@ -39,19 +39,19 @@<br>
   /sys/bus/usb/devices/ r,<br>
   /sys/devices/**/usb[0-9]*/** r,<br>
   # libusb needs udev data about usb devices (~equal to content of lsusb -v)<br>
+  /run/udev/data/+usb* r,<br>
   /run/udev/data/c16[6,7]* r,<br>
   /run/udev/data/c18[0,8,9]* r,<br>
-  /run/udev/data/+usb* r,<br>
<br>
   # WARNING: this gives the guest direct access to host hardware and specific<br>
   # portions of shared memory. This is required for sound using ALSA with kvm,<br>
   # but may constitute a security risk. If your environment does not require<br>
   # the use of sound in your VMs, feel free to comment out or prepend 'deny' to<br>
   # the rules for files in /dev.<br>
+  /dev/snd/* rw,<br>
   /{dev,run}/shm r,<br>
   /{dev,run}/shmpulse-shm* r,<br>
   /{dev,run}/shmpulse-shm* rwk,<br>
-  /dev/snd/* rw,<br>
   capability ipc_lock,<br>
   # spice<br>
   owner /{dev,run}/shm/spice.* rw,<br>
@@ -73,21 +73,21 @@<br>
   /var/lib/dbus/machine-id r,<br>
<br>
   # access to firmware's etc<br>
-  /usr/share/kvm/** r,<br>
-  /usr/share/qemu/** r,<br>
-  /usr/share/qemu-kvm/** r,<br>
+  /usr/share/AAVMF/** r,<br>
   /usr/share/bochs/** r,<br>
+  /usr/share/kvm/** r,<br>
+  /usr/share/misc/sgabios.bin r,<br>
   /usr/share/openbios/** r,<br>
   /usr/share/openhackware/** r,<br>
-  /usr/share/proll/** r,<br>
-  /usr/share/vgabios/** r,<br>
-  /usr/share/seabios/** r,<br>
-  /usr/share/misc/sgabios.bin r,<br>
-  /usr/share/ovmf/** r,<br>
   /usr/share/OVMF/** r,<br>
-  /usr/share/AAVMF/** r,<br>
+  /usr/share/ovmf/** r,<br>
+  /usr/share/proll/** r,<br>
   /usr/share/qemu-efi/** r,<br>
+  /usr/share/qemu-kvm/** r,<br>
+  /usr/share/qemu/** r,<br>
+  /usr/share/seabios/** r,<br>
   /usr/share/slof/** r,<br>
+  /usr/share/vgabios/** r,<br>
<br>
   # pki for libvirt-vnc and libvirt-spice (LP: #901272, #1690140)<br>
   /etc/pki/CA/ r,<br>
@@ -98,7 +98,33 @@<br>
   # the various binaries<br>
   /usr/bin/kvm rmix,<br>
   /usr/bin/qemu rmix,<br>
+  /usr/bin/qemu-aarch64 rmix,<br>
+  /usr/bin/qemu-alpha rmix,<br>
+  /usr/bin/qemu-arm rmix,<br>
+  /usr/bin/qemu-armeb rmix,<br>
+  /usr/bin/qemu-cris rmix,<br>
+  /usr/bin/qemu-i386 rmix,<br>
   /usr/bin/qemu-kvm rmix,<br>
+  /usr/bin/qemu-m68k rmix,<br>
+  /usr/bin/qemu-microblaze rmix,<br>
+  /usr/bin/qemu-microblazeel rmix,<br>
+  /usr/bin/qemu-mips rmix,<br>
+  /usr/bin/qemu-mips64 rmix,<br>
+  /usr/bin/qemu-mips64el rmix,<br>
+  /usr/bin/qemu-mipsel rmix,<br>
+  /usr/bin/qemu-mipsn32 rmix,<br>
+  /usr/bin/qemu-mipsn32el rmix,<br>
+  /usr/bin/qemu-or32 rmix,<br>
+  /usr/bin/qemu-ppc rmix,<br>
+  /usr/bin/qemu-ppc64 rmix,<br>
+  /usr/bin/qemu-ppc64abi32 rmix,<br>
+  /usr/bin/qemu-ppc64le rmix,<br>
+  /usr/bin/qemu-s390x rmix,<br>
+  /usr/bin/qemu-sh4 rmix,<br>
+  /usr/bin/qemu-sh4eb rmix,<br>
+  /usr/bin/qemu-sparc rmix,<br>
+  /usr/bin/qemu-sparc32plus rmix,<br>
+  /usr/bin/qemu-sparc64 rmix,<br>
   /usr/bin/qemu-system-aarch64 rmix,<br>
   /usr/bin/qemu-system-alpha rmix,<br>
   /usr/bin/qemu-system-arm rmix,<br>
@@ -132,32 +158,6 @@<br>
   /usr/bin/qemu-system-x86_64 rmix,<br>
   /usr/bin/qemu-system-xtensa rmix,<br>
   /usr/bin/qemu-system-xtensaeb rmix,<br>
-  /usr/bin/qemu-aarch64 rmix,<br>
-  /usr/bin/qemu-alpha rmix,<br>
-  /usr/bin/qemu-arm rmix,<br>
-  /usr/bin/qemu-armeb rmix,<br>
-  /usr/bin/qemu-cris rmix,<br>
-  /usr/bin/qemu-i386 rmix,<br>
-  /usr/bin/qemu-m68k rmix,<br>
-  /usr/bin/qemu-microblaze rmix,<br>
-  /usr/bin/qemu-microblazeel rmix,<br>
-  /usr/bin/qemu-mips rmix,<br>
-  /usr/bin/qemu-mips64 rmix,<br>
-  /usr/bin/qemu-mips64el rmix,<br>
-  /usr/bin/qemu-mipsel rmix,<br>
-  /usr/bin/qemu-mipsn32 rmix,<br>
-  /usr/bin/qemu-mipsn32el rmix,<br>
-  /usr/bin/qemu-or32 rmix,<br>
-  /usr/bin/qemu-ppc rmix,<br>
-  /usr/bin/qemu-ppc64 rmix,<br>
-  /usr/bin/qemu-ppc64abi32 rmix,<br>
-  /usr/bin/qemu-ppc64le rmix,<br>
-  /usr/bin/qemu-s390x rmix,<br>
-  /usr/bin/qemu-sh4 rmix,<br>
-  /usr/bin/qemu-sh4eb rmix,<br>
-  /usr/bin/qemu-sparc rmix,<br>
-  /usr/bin/qemu-sparc32plus rmix,<br>
-  /usr/bin/qemu-sparc64 rmix,<br>
   /usr/bin/qemu-unicore32 rmix,<br>
   /usr/bin/qemu-x86_64 rmix,<br>
   # for Debian/Ubuntu qemu-block-extra / RPMs qemu-block-* (LP: #1554761)<br>
-- <br>
2.24.1<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr">Christian Ehrhardt<br>Staff Engineer, Ubuntu Server<br>Canonical Ltd</div></div>