
3 Jul
2008
3 Jul
'08
6:27 a.m.
"Daniel P. Berrange" <berrange@redhat.com> wrote: ... cont'd
static int testNetworkUndefine(virNetworkPtr network) { ... static char *testNetworkGetBridgeName(virNetworkPtr network) { char *bridge; GET_NETWORK(network, NULL); - bridge = strdup(privnet->bridge); + bridge = privnet->def->bridge ? strdup(privnet->def->bridge) : NULL; if (!bridge) { testError(network->conn, NULL, network, VIR_ERR_NO_MEMORY, "network"); return NULL;
This change makes it so a NULL privnet->def->bridge will mistakenly provoke an out of memory error.