On Thu, May 16, 2013 at 10:01:51AM -0400, John Ferlan wrote:
In virLXCProcessReboot, if the 'conn' value is NULL, the
eventual
call to virLXCNetworkLookupByName() via virLXCProcessStart() and
virLXCProcessSetupInterfaces() will cause a core since it references
conn->networkDriver directly.
---
src/lxc/lxc_process.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/lxc/lxc_process.c b/src/lxc/lxc_process.c
index b06d748..4410db4 100644
--- a/src/lxc/lxc_process.c
+++ b/src/lxc/lxc_process.c
@@ -181,7 +181,6 @@ virLXCProcessReboot(virLXCDriverPtr driver,
autodestroy = true;
} else {
conn = virConnectOpen("lxc:///");
- /* Ignoring NULL conn which is mostly harmless here */
I'm actually inclined to say that we should stop ignoring failures
of virConnectOpen(). I don't think there's any valid reason for us
to be doing that. Then we wouldnt have to worry about conn being
NULL later
This ignoring of errors is historical legacy of my own creation
when first writing the QEMU driver. I can't think what possessed
me to ignore the errors at the time.
Daniel
--
|:
http://berrange.com -o-
http://www.flickr.com/photos/dberrange/ :|
|:
http://libvirt.org -o-
http://virt-manager.org :|
|:
http://autobuild.org -o-
http://search.cpan.org/~danberr/ :|
|:
http://entangle-photo.org -o-
http://live.gnome.org/gtk-vnc :|