Re: [libvirt] handling qemuMonitorAddDevice failure: missing drive_del function?
On Tue, May 18, 2010 at 03:23:23PM +0200, Jim Meyering wrote:
In src/qemu/qemu_driver.c, coverity gripes (rightly) about this:
6912 qemuDomainObjEnterMonitorWithDriver(driver, vm);
6913 if (qemuCmdFlags & QEMUD_CMD_FLAG_DEVICE) {
6914 ret = qemuMonitorAddDrive(priv->mon, drivestr);
6915 if (ret == 0)
No check of the return value of "qemuMonitorAddDevice(priv->mon, devstr)".
Calling function "qemuMonitorAddDevice" without checking return value.
6916 qemuMonitorAddDevice(priv->mon, devstr);
6917 /* XXX remove the drive upon fail */
6918 } else {
Does anyone have a preference on how to deal with it
while we wait for a drive-removal function?
I think it deserves at least a diagnostic.
Add a VIR_WARN message i guess
Daniel
--
|: Red Hat, Engineering, London -o- http://people.redhat.com/berrange/ :|
|: http://libvirt.org -o- http://virt-manager.org -o- http://deltacloud.org :|
|: http://autobuild.org -o- http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505 -o- F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|