On Sat, May 17, 2014 at 11:44:48PM +0400, Roman Bogorodskiy wrote: > In a number of places in the bhyve driver, virObjectUnlock() > is called with an arg without check if the arg is non-NULL, which > could result in passing NULL value and a warning like: > > virObjectUnlock:340 : Object 0x0 ((unknown)) is not a virObjectLockable instance > > * src/bhyve/bhyve_driver.c (bhyveDomainGetInfo) > (bhyveDomainGetState, bhyveDomainGetAutostart) > (bhyveDomainSetAutostart, bhyveDomainIsActive) > (bhyveDomainIsPersistent, bhyveDomainGetXMLDesc) > (bhyveDomainUndefine, bhyveDomainLookupByUUID) > (bhyveDomainLookupByName, bhyveDomainLookupByID) > (bhyveDomainCreateWithFlags, bhyveDomainOpenConsole): > Check if arg is not NULL before calling virObjectUnlock on it. > --- > src/bhyve/bhyve_driver.c | 39 ++++++++++++++++++++++++++------------- > 1 file changed, 26 insertions(+), 13 deletions(-) ACK Regards, 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 :| -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list