[Libvir] [PATCH] Delete a harmful variable in xend_parse_sexp_desc()

2007-08-27 Thread Masayuki Sunou
Hi Xend_parse_sexp_desc() of xend_internal.c declares a variable named ret to buffer SXP. However, it actually uses buf.content not ret and allocates memory to buf when size of ret became insufficient. -- virBufferAdd(buf, ...);, virBufferVSprintf(buf, ...) So freeing ret fails, because ret

Re: [Libvir] [PATCH] Delete a harmful variable in xend_parse_sexp_desc()

2007-08-27 Thread Daniel Veillard
On Mon, Aug 27, 2007 at 05:37:29PM +0900, Masayuki Sunou wrote: Hi Xend_parse_sexp_desc() of xend_internal.c declares a variable named ret to buffer SXP. However, it actually uses buf.content not ret and allocates memory to buf when size of ret became insufficient. -- virBufferAdd(buf,