[sheepdog] [PATCH] sheep: fix memory leaks at start_deletion()

Hitoshi Mitake mitake.hitoshi at gmail.com
Tue Dec 17 02:54:04 CET 2013


At Mon, 16 Dec 2013 23:00:18 +0900,
Ryusuke Konishi wrote:
> 
> The start_deletion function does not free the deletion_work structure
> nor the buffer (dw->buf) that it locally allocated when it deletes
> inode of the VDI or dw->count was equal to zero.
> 
> This fixes these leak bugs.
> 
> Signed-off-by: Ryusuke Konishi <konishi.ryusuke at lab.ntt.co.jp>
> ---
>  sheep/vdi.c |   10 ++++++----
>  1 file changed, 6 insertions(+), 4 deletions(-)

Looks good to me.
Reviewed-by: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>

Thanks,
Hitoshi



More information about the sheepdog mailing list