aboutsummaryrefslogtreecommitdiffstats
path: root/tools
diff options
context:
space:
mode:
authorAndrew Cooper <andrew.cooper3@citrix.com>2013-10-09 12:11:48 +0200
committerJan Beulich <jbeulich@suse.com>2013-10-09 12:11:48 +0200
commitc56cdf93750fd978aebd69dc113f12cc770ff8e3 (patch)
tree92104e99e9f635b4fdffe57e96d65533f57c2a04 /tools
parent839b966e3f587bbb1a0d954230fb3904330dccb6 (diff)
downloadxen-c56cdf93750fd978aebd69dc113f12cc770ff8e3.tar.gz
xen-c56cdf93750fd978aebd69dc113f12cc770ff8e3.tar.bz2
xen-c56cdf93750fd978aebd69dc113f12cc770ff8e3.zip
hvm/vidirian: Avoid printing page_to_mfn(NULL) on error paths
While working in the viridian code, I noticed that 4cb6c4f4941 "x86/hvm: Use get_page_from_gfn() instead of get_gfn()/put_gfn." introduced two error paths where page_to_mfn(NULL) would be formatted and presented as a bad MFN. This provides junk in the warning rather than something useful. These two codepaths are fixed up to match their counterpart in wrmsr_hypervisor_regs() While auditing the other changes from 4cb6c4f4941, I noticed a small optimisation which could be made by changing the order of the validity checks to remove 6 NULL pointer checks. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Acked-by: Keir Fraser <keir@xen.org>
Diffstat (limited to 'tools')
0 files changed, 0 insertions, 0 deletions