diff options
author | Daniel De Graaf <dgdegra@tycho.nsa.gov> | 2012-09-26 11:56:07 +0200 |
---|---|---|
committer | Daniel De Graaf <dgdegra@tycho.nsa.gov> | 2012-09-26 11:56:07 +0200 |
commit | 8e804aec451a072022eece0980481ab545b5cd75 (patch) | |
tree | 513cb4af4154c824c8cfed80b9e86605533b67bb /xen/arch/x86/mm.c | |
parent | 4047479dfa0ae519ecac19528e01c2816f0c34bc (diff) | |
download | xen-8e804aec451a072022eece0980481ab545b5cd75.tar.gz xen-8e804aec451a072022eece0980481ab545b5cd75.tar.bz2 xen-8e804aec451a072022eece0980481ab545b5cd75.zip |
x86: check remote MMIO remap permissions
When a domain is mapping pages from a different pg_owner domain, the
iomem_access checks are currently only applied to the pg_owner domain,
potentially allowing a domain with a more restrictive iomem_access
policy to have the pages mapped into its page tables. To catch this,
also check the owner of the page tables. The current domain does not
need to be checked because the ability to manipulate a domain's page
tables implies full access to the target domain, so checking that
domain's permission is sufficient.
Signed-off-by: Daniel De Graaf <dgdegra@tycho.nsa.gov>
Committed-by: Jan Beulich <jbeulich@suse.com>
Diffstat (limited to 'xen/arch/x86/mm.c')
-rw-r--r-- | xen/arch/x86/mm.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/xen/arch/x86/mm.c b/xen/arch/x86/mm.c index efcca5dbee..8ab92c9ee1 100644 --- a/xen/arch/x86/mm.c +++ b/xen/arch/x86/mm.c @@ -754,6 +754,19 @@ get_page_from_l1e( return -EINVAL; } + if ( pg_owner != l1e_owner && + !iomem_access_permitted(l1e_owner, mfn, mfn) ) + { + if ( mfn != (PADDR_MASK >> PAGE_SHIFT) ) /* INVALID_MFN? */ + { + MEM_LOG("Dom%u attempted to map I/O space %08lx in dom%u to dom%u", + curr->domain->domain_id, mfn, pg_owner->domain_id, + l1e_owner->domain_id); + return -EPERM; + } + return -EINVAL; + } + if ( !(l1f & _PAGE_RW) || !rangeset_contains_singleton(mmio_ro_ranges, mfn) ) return 0; |