This matches the behavior of domains, storage, etc. Also, fix up some whitespace issues in the area. Signed-off-by: Cole Robinson <crobinso(a)redhat.com&gt; --- src/test/test_driver.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/src/test/test_driver.c b/src/test/test_driver.c index 0b866d2..0541a73 100644 --- a/src/test/test_driver.c +++ b/src/test/test_driver.c @@ -779,13 +779,16 @@ static int testOpenFromFile(virConnectPtr conn, if (!def) goto error; } else { - if ((def = virInterfaceDefParseNode(conn, xml, ifaces[i])) == NULL) + if ((def = virInterfaceDefParseNode(conn, xml, ifaces[i])) == NULL) goto error; } + if (!(iface = virInterfaceAssignDef(conn, &privconn->ifaces, def))) { virInterfaceDefFree(def); goto error; } + + iface->active = 1; virInterfaceObjUnlock(iface); } VIR_FREE(ifaces);

On Mon, Oct 12, 2009 at 10:32:13AM -0400, Cole Robinson wrote: > This matches the behavior of domains, storage, etc. Also, fix up some > whitespace issues in the area. > > Signed-off-by: Cole Robinson <crobinso(a)redhat.com&gt; > --- > src/test/test_driver.c | 5 ++++- > 1 files changed, 4 insertions(+), 1 deletions(-) > > diff --git a/src/test/test_driver.c b/src/test/test_driver.c > index 0b866d2..0541a73 100644 > --- a/src/test/test_driver.c > +++ b/src/test/test_driver.c > @@ -779,13 +779,16 @@ static int testOpenFromFile(virConnectPtr conn, > if (!def) > goto error; > } else { > - if ((def = virInterfaceDefParseNode(conn, xml, ifaces[i])) == NULL) > + if ((def = virInterfaceDefParseNode(conn, xml, ifaces[i])) == NULL) > goto error; > } > + > if (!(iface = virInterfaceAssignDef(conn, &privconn->ifaces, def))) { > virInterfaceDefFree(def); > goto error; > } > + > + iface->active = 1; > virInterfaceObjUnlock(iface); > } > VIR_FREE(ifaces); ACK, please push that one too :-) Daniel