x86: PHYSDEVOP_{prepare,release}_msix are privileged
authorJan Beulich <jbeulich@suse.com>
Fri, 24 Jan 2014 12:41:36 +0000 (13:41 +0100)
committerJan Beulich <jbeulich@suse.com>
Fri, 24 Jan 2014 12:41:36 +0000 (13:41 +0100)
Yet this wasn't being enforced.

This is XSA-87.

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
xen/arch/x86/physdev.c

index 1040b2cc86b5502bc1e8f8560936c74e230d28da..bc0634c7127e28bec6ac3510b2496b56b9e06397 100644 (file)
@@ -640,7 +640,10 @@ ret_t do_physdev_op(int cmd, XEN_GUEST_HANDLE_PARAM(void) arg)
         if ( copy_from_guest(&dev, arg, 1) )
             ret = -EFAULT;
         else
-            ret = pci_prepare_msix(dev.seg, dev.bus, dev.devfn,
+            ret = xsm_resource_setup_pci(XSM_PRIV,
+                                         (dev.seg << 16) | (dev.bus << 8) |
+                                         dev.devfn) ?:
+                  pci_prepare_msix(dev.seg, dev.bus, dev.devfn,
                                    cmd != PHYSDEVOP_prepare_msix);
         break;
     }