Re: [Qemu-devel] [PATCH V5] net/colo-compare.c: Fix memory leak and code style issue.

[Peter Maydell]

On Tue, 23 Jul 2019 at 04:24, Jason Wang <address@hidden> wrote:
>
>
> On 2019/7/23 上午2:38, Zhang, Chen wrote:
> >
> >
> >> -----Original Message-----
> >> From: Peter Maydell [mailto:address@hidden]
> >> Sent: Tuesday, July 23, 2019 1:25 AM
> >> To: Zhang, Chen <address@hidden>
> >> Cc: Li Zhijian <address@hidden>; Jason Wang <address@hidden>;
> >> qemu-dev <address@hidden>; Zhang Chen <address@hidden>
> >> Subject: Re: [PATCH V5] net/colo-compare.c: Fix memory leak and code style
> >> issue.
> >>
> >> On Mon, 22 Jul 2019 at 18:23, Zhang Chen <address@hidden> wrote:
> >>> From: Zhang Chen <address@hidden>
> >>>
> >>> This patch to fix the origin "char *data" memory leak, code style
> >>> issue and add necessary check here.
> >>> Reported-by: Coverity (CID 1402785)
> >>>
> >>> Signed-off-by: Zhang Chen <address@hidden>
> >>> ---
> >>>   net/colo-compare.c | 27 ++++++++++++++++++++-------
> >>>   1 file changed, 20 insertions(+), 7 deletions(-)
> >> Reviewed-by: Peter Maydell <address@hidden>
> >>
> >> Thanks for your patience in sending out all these versions as we worked
> >> through the code review process.
> > Thank you too~
> > Maybe you or Jason can pick up this patch?

> Applied.

Thanks. This is now our last outstanding coverity issue --
are you planning to send this in for rc3 ?

-- PMM