Daniel P. Berrange wrote: > 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 Pushed, thanks! Roman Bogorodskiy
Attachment:
pgppO8DdjCS2p.pgp
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list