diff mbox

pc: don't access fw cfg if NULL

Message ID 20130711115240.GA24956@redhat.com
State New
Headers show

Commit Message

Michael S. Tsirkin July 11, 2013, 11:52 a.m. UTC
commit f8c457b88d72a48989f190bc3d7b79f4f3b7d11c
     "pc: pass PCI hole ranges to Guests"
broke Xen as it has no fw_cfg.
Check for this configuration and boil out.

Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
Tested-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>

---

Anthony, could you fast-track this into master please?
Thanks!

 hw/i386/pc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Igor Mammedov July 11, 2013, 12:18 p.m. UTC | #1
On Thu, 11 Jul 2013 14:52:40 +0300
"Michael S. Tsirkin" <mst@redhat.com> wrote:

> commit f8c457b88d72a48989f190bc3d7b79f4f3b7d11c
>      "pc: pass PCI hole ranges to Guests"
> broke Xen as it has no fw_cfg.
> Check for this configuration and boil out.
> 
> Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
> Tested-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com>
> 
> ---
> 
> Anthony, could you fast-track this into master please?
> Thanks!
> 
>  hw/i386/pc.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/hw/i386/pc.c b/hw/i386/pc.c
> index 9f9207d..c7fc475 100644
> --- a/hw/i386/pc.c
> +++ b/hw/i386/pc.c
> @@ -999,7 +999,7 @@ typedef struct PcRomPciInfo {
>  static void pc_fw_cfg_guest_info(PcGuestInfo *guest_info)
>  {
>      PcRomPciInfo *info;
> -    if (!guest_info->has_pci_info) {
> +    if (!guest_info->has_pci_info || !guest_info->fw_cfg) {
>          return;
>      }
>  

Reviewed-By: Igor Mammedov <imammedo@redhat.com>
diff mbox

Patch

diff --git a/hw/i386/pc.c b/hw/i386/pc.c
index 9f9207d..c7fc475 100644
--- a/hw/i386/pc.c
+++ b/hw/i386/pc.c
@@ -999,7 +999,7 @@  typedef struct PcRomPciInfo {
 static void pc_fw_cfg_guest_info(PcGuestInfo *guest_info)
 {
     PcRomPciInfo *info;
-    if (!guest_info->has_pci_info) {
+    if (!guest_info->has_pci_info || !guest_info->fw_cfg) {
         return;
     }