summaryrefslogtreecommitdiff
path: root/hw
diff options
context:
space:
mode:
authorMichael S. Tsirkin <mst@redhat.com>2009-11-12 13:54:31 +0200
committerMichael S. Tsirkin <mst@redhat.com>2009-12-01 17:52:51 +0200
commit1ef91b31308b5ddd275be6bab66b25e0256d87c3 (patch)
tree45146584d748f54d76fed58830ac226a606cc5ee /hw
parent65f558a71786562c5d00fddc8789e1538678cd65 (diff)
downloadqemu-1ef91b31308b5ddd275be6bab66b25e0256d87c3.tar.gz
qemu-1ef91b31308b5ddd275be6bab66b25e0256d87c3.tar.bz2
qemu-1ef91b31308b5ddd275be6bab66b25e0256d87c3.zip
pci: convert goto into scope in bridge_filter
goto into scope is evil. rearrange pci_bridge_filter so that we always go to end of function on error. Signed-off-by: Michael S. Tsirkin <mst@redhat.com> Acked-by: Isaku Yamahata <yamahata@valinux.co.jp>
Diffstat (limited to 'hw')
-rw-r--r--hw/pci.c13
1 files changed, 7 insertions, 6 deletions
diff --git a/hw/pci.c b/hw/pci.c
index e26b3d0341..8cf008d31d 100644
--- a/hw/pci.c
+++ b/hw/pci.c
@@ -717,13 +717,14 @@ static void pci_bridge_filter(PCIDevice *d, pcibus_t *addr, pcibus_t *size,
}
if (base > limit) {
- no_map:
- *addr = PCI_BAR_UNMAPPED;
- *size = 0;
- } else {
- *addr = base;
- *size = limit - base + 1;
+ goto no_map;
}
+ *addr = base;
+ *size = limit - base + 1;
+ return;
+no_map:
+ *addr = PCI_BAR_UNMAPPED;
+ *size = 0;
}
static pcibus_t pci_bar_address(PCIDevice *d,