virInterfaceDefParseBond: avoid dead stores

* src/conf/interface_conf.c (virInterfaceDefParseBond): Avoid dead stores
to local, "node".  Remove declaration, too.
This commit is contained in:
Jim Meyering
2010-03-05 16:18:47 +01:00
parent fbe65e5e1c
commit c101092adf

View File

@@ -570,7 +570,6 @@ error:
static int static int
virInterfaceDefParseBond(virInterfaceDefPtr def, virInterfaceDefParseBond(virInterfaceDefPtr def,
xmlXPathContextPtr ctxt) { xmlXPathContextPtr ctxt) {
xmlNodePtr node;
int ret = -1; int ret = -1;
unsigned long tmp; unsigned long tmp;
@@ -582,8 +581,7 @@ virInterfaceDefParseBond(virInterfaceDefPtr def,
if (ret != 0) if (ret != 0)
goto error; goto error;
node = virXPathNode("./miimon[1]", ctxt); if (virXPathNode("./miimon[1]", ctxt) != NULL) {
if (node != NULL) {
def->data.bond.monit = VIR_INTERFACE_BOND_MONIT_MII; def->data.bond.monit = VIR_INTERFACE_BOND_MONIT_MII;
ret = virXPathULong("string(./miimon/@freq)", ctxt, &tmp); ret = virXPathULong("string(./miimon/@freq)", ctxt, &tmp);
@@ -618,7 +616,7 @@ virInterfaceDefParseBond(virInterfaceDefPtr def,
goto error; goto error;
} }
} else if ((node = virXPathNode("./arpmon[1]", ctxt)) != NULL) { } else if (virXPathNode("./arpmon[1]", ctxt) != NULL) {
def->data.bond.monit = VIR_INTERFACE_BOND_MONIT_ARP; def->data.bond.monit = VIR_INTERFACE_BOND_MONIT_ARP;