[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] monitor: Plug memory leak on QMP error
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] monitor: Plug memory leak on QMP error |
Date: |
Fri, 30 Oct 2015 08:35:00 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Luiz Capitulino <address@hidden> writes:
> On Thu, 29 Oct 2015 17:23:43 +0100
> Markus Armbruster <address@hidden> wrote:
>
>> Luiz Capitulino <address@hidden> writes:
>>
>> > On Thu, 29 Oct 2015 12:15:09 +0100
>> > Markus Armbruster <address@hidden> wrote:
>> >
>> >> Leak introduced in commit 8a4f501..710aec9, v2.4.0.
>> >>
>> >> Signed-off-by: Markus Armbruster <address@hidden>
>> >
>> > Reviewed-by: Luiz Capitulino <address@hidden>
>>
>> Thanks!
>>
>> > I think this can go through your tree?
>>
>> Yes, along with the json-streamer work.
>
> Do you need my review for that one? It's been ages that I don't
> look at that code, I'm not sure it's going to be that worthwhile.
Review is always welcome, but "need" is too strong. qobject/json* isn't
your code anyway, and I got Eric's review already.