On Thu, 2014-11-20 at 16:03 +0000, Wei Liu wrote:
> On Thu, Nov 20, 2014 at 03:48:47PM +0000, Ian Campbell wrote:
> > The libxc xc_dom_* infrastructure uses a very simple malloc memory pool 
> > which
> > is freed by xc_dom_release. However the various xc_try_*_decode routines 
> > (other
> > than the gzip one) just use plain malloc/realloc and therefore the buffer 
> > ends
> > up leaked.
> > 
> > The memory pool currently supports mmap'd buffers as well as a directly
> > allocated buffers, however the try decode routines make use of realloc and 
> > do
> > not fit well into this model. Introduce a concept of an external memory 
> > block
> > to the memory pool and provide an interface to register such memory.
> > 
> > The mmap_ptr and mmap_len fields of the memblock tracking struct lose their
> > mmap_ prefix since they are now also used for external memory blocks.
> > 
> > We are only seeing this now because the gzip decoder doesn't leak and it's 
> > only
> > relatively recently that kernels in the wild have switched to better
> > compression.
> > 
> > This is https://bugs.debian.org/767295
> > 
> > Reported by: Gedalya <geda...@gedalya.net>
> > Signed-off-by: Ian Campbell <ian.campb...@citrix.com>
> 
> Reviewed-by: Wei Liu <wei.l...@citrix.com>

Thanks. Konrad release-acked on IRC so I've applied.

Ian.


_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xen.org
http://lists.xen.org/xen-devel

Reply via email to