Re: [Qemu-devel] [v2 3/3] hmp: fix MemdevList memory leak

2014-08-17 Thread chen.fan.f...@cn.fujitsu.com
On Tue, 2014-08-12 at 12:00 +1000, Peter Crosthwaite wrote: > On Mon, Aug 4, 2014 at 2:21 PM, Chen Fan wrote: > > the memdev_list in hmp_info_memdev() is never freed. > > so we use existent method qapi_free_MemdevList() to free it. > > and also we can use qapi_free_MemdevList() to replace list lo

Re: [Qemu-devel] [v2 3/3] hmp: fix MemdevList memory leak

2014-08-11 Thread Peter Crosthwaite
On Mon, Aug 4, 2014 at 2:21 PM, Chen Fan wrote: > the memdev_list in hmp_info_memdev() is never freed. > so we use existent method qapi_free_MemdevList() to free it. > and also we can use qapi_free_MemdevList() to replace list loops > to clean up the memdev list in error path. > > Signed-off-by: C

Re: [Qemu-devel] [v2 3/3] hmp: fix MemdevList memory leak

2014-08-04 Thread Hu Tao
On Mon, Aug 04, 2014 at 12:21:19PM +0800, Chen Fan wrote: > the memdev_list in hmp_info_memdev() is never freed. > so we use existent method qapi_free_MemdevList() to free it. > and also we can use qapi_free_MemdevList() to replace list loops > to clean up the memdev list in error path. > > Signed