[libvirt] [PATCH 1/6] conf: make hostdev info a separate object

Laine Stump laine at laine.org
Mon Feb 20 19:46:54 UTC 2012


On 02/20/2012 01:48 PM, Eric Blake wrote:
> Overall, the idea looks reasonable, but you'll need a v2 to fix the
> memory issues in qemu_command.c.

Actually, there were problems with freeing the hostdevdef on error paths
in all 4 places virDomainHostdevDefAlloc() was used :-( I don't know
what I was thinking...

I've updated the patch with the following diff and will repost the full
patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-fix-memory-leaks-in-make-hostdev-info-a-separate-obj.patch
Type: text/x-patch
Size: 5652 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120220/db328b2a/attachment-0001.bin>


More information about the libvir-list mailing list