Re: [libvirt] [PATCH] virsh: Call virDomainFree in cmdDomFSTrim

2013-04-02 Thread Peter Krempa
On 04/02/13 17:40, Martin Kletzander wrote: On 04/02/2013 05:36 PM, Michal Privoznik wrote: On 02.04.2013 17:32, Martin Kletzander wrote: On 04/02/2013 05:20 PM, Michal Privoznik wrote: The virsh domfstrim command was not freeing allocated domain, leaving leaked references behind. --- tools/

Re: [libvirt] [PATCH] virsh: Call virDomainFree in cmdDomFSTrim

2013-04-02 Thread Martin Kletzander
On 04/02/2013 05:36 PM, Michal Privoznik wrote: > On 02.04.2013 17:32, Martin Kletzander wrote: >> On 04/02/2013 05:20 PM, Michal Privoznik wrote: >>> The virsh domfstrim command was not freeing allocated domain, >>> leaving leaked references behind. >>> --- >>> tools/virsh-domain.c | 2 ++ >>> 1

Re: [libvirt] [PATCH] virsh: Call virDomainFree in cmdDomFSTrim

2013-04-02 Thread Michal Privoznik
On 02.04.2013 17:32, Martin Kletzander wrote: > On 04/02/2013 05:20 PM, Michal Privoznik wrote: >> The virsh domfstrim command was not freeing allocated domain, >> leaving leaked references behind. >> --- >> tools/virsh-domain.c | 2 ++ >> 1 file changed, 2 insertions(+) >> >> diff --git a/tools/v

Re: [libvirt] [PATCH] virsh: Call virDomainFree in cmdDomFSTrim

2013-04-02 Thread Ján Tomko
On 04/02/2013 05:20 PM, Michal Privoznik wrote: > The virsh domfstrim command was not freeing allocated domain, > leaving leaked references behind. > --- > tools/virsh-domain.c | 2 ++ > 1 file changed, 2 insertions(+) > You could add the public bugzilla link to the commit message too: https://b

Re: [libvirt] [PATCH] virsh: Call virDomainFree in cmdDomFSTrim

2013-04-02 Thread Martin Kletzander
On 04/02/2013 05:20 PM, Michal Privoznik wrote: > The virsh domfstrim command was not freeing allocated domain, > leaving leaked references behind. > --- > tools/virsh-domain.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c > index 5ddcedc..

Re: [libvirt] [PATCH] virsh: Call virDomainFree in cmdDomFSTrim

2013-04-02 Thread Eric Blake
On 04/02/2013 09:20 AM, Michal Privoznik wrote: > The virsh domfstrim command was not freeing allocated domain, > leaving leaked references behind. > --- > tools/virsh-domain.c | 2 ++ > 1 file changed, 2 insertions(+) ACK. > > diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c > index 5d

[libvirt] [PATCH] virsh: Call virDomainFree in cmdDomFSTrim

2013-04-02 Thread Michal Privoznik
The virsh domfstrim command was not freeing allocated domain, leaving leaked references behind. --- tools/virsh-domain.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/tools/virsh-domain.c b/tools/virsh-domain.c index 5ddcedc..5fbfeee 100644 --- a/tools/virsh-domain.c +++ b/tools/virsh-doma