From: Isaku Yamahata <yamahata@valinux.co.jp>
Date: Thu, 9 Sep 2010 02:48:56 +0000 (+0900)
Subject: pci: don't ignore invalid parameter for pci_register_bar().
X-Git-Tag: TizenStudio_2.0_p2.3~3958^2^2~23
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=5ef28afbdb83e74c3433ee02d55ea2f506a2c29f;p=sdk%2Femulator%2Fqemu.git

pci: don't ignore invalid parameter for pci_register_bar().

Abort when invalid value for region_num is passed to pci_register_bar.
That is caller's bug. Abort instead of silently ignoring invalid value.

Signed-off-by: Isaku Yamahata <yamahata@valinux.co.jp>
Signed-off-by: Michael S. Tsirkin <mst@redhat.com>
---

diff --git a/hw/pci.c b/hw/pci.c
index c28b8a1..8f48d9b 100644
--- a/hw/pci.c
+++ b/hw/pci.c
@@ -765,9 +765,8 @@ void pci_register_bar(PCIDevice *pci_dev, int region_num,
     uint32_t addr;
     uint64_t wmask;
 
-    if ((unsigned int)region_num >= PCI_NUM_REGIONS)
-        return;
-
+    assert(region_num >= 0);
+    assert(region_num < PCI_NUM_REGIONS);
     if (size & (size-1)) {
         fprintf(stderr, "ERROR: PCI region size must be pow2 "
                     "type=0x%x, size=0x%"FMT_PCIBUS"\n", type, size);