Re: [Qemu-devel] [PATCH] monitor: Plug memory leak on QMP error

2015-10-30 Thread Markus Armbruster
Luiz Capitulino writes: > On Thu, 29 Oct 2015 17:23:43 +0100 > Markus Armbruster wrote: > >> Luiz Capitulino writes: >> >> > On Thu, 29 Oct 2015 12:15:09 +0100 >> > Markus Armbruster wrote: >> > >> >> Leak introduced in commit 8a4f501..710aec9, v2.4.0. >> >> >> >> Signed-off-by: Markus Armbr

Re: [Qemu-devel] [PATCH] monitor: Plug memory leak on QMP error

2015-10-29 Thread Luiz Capitulino
On Thu, 29 Oct 2015 17:23:43 +0100 Markus Armbruster wrote: > Luiz Capitulino writes: > > > On Thu, 29 Oct 2015 12:15:09 +0100 > > Markus Armbruster wrote: > > > >> Leak introduced in commit 8a4f501..710aec9, v2.4.0. > >> > >> Signed-off-by: Markus Armbruster > > > > Reviewed-by: Luiz Capitu

Re: [Qemu-devel] [PATCH] monitor: Plug memory leak on QMP error

2015-10-29 Thread Markus Armbruster
Luiz Capitulino writes: > On Thu, 29 Oct 2015 12:15:09 +0100 > Markus Armbruster wrote: > >> Leak introduced in commit 8a4f501..710aec9, v2.4.0. >> >> Signed-off-by: Markus Armbruster > > Reviewed-by: Luiz Capitulino Thanks! > I think this can go through your tree? Yes, along with the json

Re: [Qemu-devel] [PATCH] monitor: Plug memory leak on QMP error

2015-10-29 Thread Luiz Capitulino
On Thu, 29 Oct 2015 12:15:09 +0100 Markus Armbruster wrote: > Leak introduced in commit 8a4f501..710aec9, v2.4.0. > > Signed-off-by: Markus Armbruster Reviewed-by: Luiz Capitulino I think this can go through your tree? > --- > monitor.c | 1 + > 1 file changed, 1 insertion(+) > > diff --g

Re: [Qemu-devel] [PATCH] monitor: Plug memory leak on QMP error

2015-10-29 Thread Eric Blake
On 10/29/2015 05:15 AM, Markus Armbruster wrote: > Leak introduced in commit 8a4f501..710aec9, v2.4.0. > > Signed-off-by: Markus Armbruster > --- > monitor.c | 1 + > 1 file changed, 1 insertion(+) Reviewed-by: Eric Blake > > diff --git a/monitor.c b/monitor.c > index 301a143..c59e85b 100644

[Qemu-devel] [PATCH] monitor: Plug memory leak on QMP error

2015-10-29 Thread Markus Armbruster
Leak introduced in commit 8a4f501..710aec9, v2.4.0. Signed-off-by: Markus Armbruster --- monitor.c | 1 + 1 file changed, 1 insertion(+) diff --git a/monitor.c b/monitor.c index 301a143..c59e85b 100644 --- a/monitor.c +++ b/monitor.c @@ -3862,6 +3862,7 @@ static void handle_qmp_command(JSONMess