diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2012-04-02 14:17:35 +1000 |
---|---|---|
committer | Andreas Färber <afaerber@suse.de> | 2012-04-15 20:02:09 +0200 |
commit | 92615a5ab96cd2b4981e2aaef40bece6cd7553b6 (patch) | |
tree | d3a7437e69595b1da0e5c8bc2169d1ccd40a88fe /hw/spapr_vscsi.c | |
parent | 45e45ed2d61fb287adf03e49c74c152bfd2c975a (diff) |
pseries: Fix RTAS based config access
On the pseries platform, access to PCI config space is via RTAS calls(
which go to the hypervisor) rather than MMIO. This means we don't use
the same code path as nearly everyone else which goes through pci_host.c
and we're missing some of the parameter checking along the way.
We do have some parameter checking in the RTAS calls, but it's not enough.
It checks for overruns, but does not check for unaligned accesses,
oversized accesses (which means the guest could trigger an assertion
failure from pci_host_config_{read,write}_common(). Worse it doesn't do
the basic checking for the number of RTAS arguments and results before
accessing them.
This patch fixes these bugs.
Cc: Michael S. Tsirkin <mst@redhat.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
[AF: Fix typos spotted by mst]
Signed-off-by: Andreas Färber <afaerber@suse.de>
Diffstat (limited to 'hw/spapr_vscsi.c')
0 files changed, 0 insertions, 0 deletions