RE: [PATCH] net/filter-rewriter: destroy g_hash_table in colo_rewriter_cleanup

["Zhang, Chen" <chen.zhang@intel.com>]

I already queued this patch to COLO proxy tree, then I will send a series to Jason.

Thanks
Chen

> -----Original Message-----
> From: Pan Nengyuan <pannengyuan@huawei.com>
> Sent: Saturday, October 10, 2020 6:55 PM
> To: Li Qiang <liq3ea@gmail.com>
> Cc: Qemu Developers <qemu-devel@nongnu.org>; Zhang, Chen
> <chen.zhang@intel.com>; Jason Wang <jasowang@redhat.com>;
> zhanghailiang <zhang.zhanghailiang@huawei.com>; Chen Qun
> <kuhn.chenqun@huawei.com>; qemu-trivial@nongnu.org; Laurent Vivier
> <laurent@vivier.eu>
> Subject: Re: [PATCH] net/filter-rewriter: destroy g_hash_table in
> colo_rewriter_cleanup
> 
> ping!
> 
> Maybe missed to queue?
> 
> On 2020/9/5 8:44, Li Qiang wrote:
> > Pan Nengyuan <pannengyuan@huawei.com> 于2020年9月4日周五 下
> 午3:23写道：
> >>
> >> s->connection_track_table forgot to destroy in colo_rewriter_cleanup. Fix
> it.
> >>
> >> Reported-by: Euler Robot <euler.robot@huawei.com>
> >> Signed-off-by: Pan Nengyuan <pannengyuan@huawei.com>
> >
> > Reviewed-by: Li Qiang <liq3ea@gmail.com>
> >
> >> ---
> >>  net/filter-rewriter.c | 2 ++
> >>  1 file changed, 2 insertions(+)
> >>
> >> diff --git a/net/filter-rewriter.c b/net/filter-rewriter.c index
> >> 1aaad101b6..9ff366d44f 100644
> >> --- a/net/filter-rewriter.c
> >> +++ b/net/filter-rewriter.c
> >> @@ -376,6 +376,8 @@ static void colo_rewriter_cleanup(NetFilterState
> *nf)
> >>          filter_rewriter_flush(nf);
> >>          g_free(s->incoming_queue);
> >>      }
> >> +
> >> +    g_hash_table_destroy(s->connection_track_table);
> >>  }
> >>
> >>  static void colo_rewriter_setup(NetFilterState *nf, Error **errp)
> >> --
> >> 2.18.2
> >>
> >>