Re: [libvirt] [PATCH 1/4] conf: Use virDomainChrSourceDefNew for vhostuser

2018-04-09 Thread Marc Hartmayer
On Fri, Apr 06, 2018 at 06:53 PM +0200, John Ferlan wrote: > Rather than using VIR_ALLOC, use the New API since we already > use the virDomainChrSourceDefFree function when done. > > Signed-off-by: John Ferlan > --- > src/conf/domain_conf.c | 2 +- > 1

Re: [libvirt] [PATCH 1/4] conf: Use virDomainChrSourceDefNew for vhostuser

2018-04-06 Thread Eric Blake
On 04/06/2018 01:31 PM, Laine Stump wrote: > On 04/06/2018 12:53 PM, John Ferlan wrote: >> Rather than using VIR_ALLOC, use the New API since we already >> use the virDomainChrSourceDefFree function when done. >> >> -if (VIR_ALLOC(def->data.vhostuser) < 0) >> +if

Re: [libvirt] [PATCH 1/4] conf: Use virDomainChrSourceDefNew for vhostuser

2018-04-06 Thread Laine Stump
On 04/06/2018 12:53 PM, John Ferlan wrote: > Rather than using VIR_ALLOC, use the New API since we already > use the virDomainChrSourceDefFree function when done. > > Signed-off-by: John Ferlan > --- > src/conf/domain_conf.c | 2 +- > 1 file changed, 1 insertion(+), 1