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.