Re: [PATCH] block: fix improper kobject release inblk_integrity_unregister

From: Jens Axboe
Date: Tue Jul 28 2009 - 03:10:50 EST


On Fri, Jul 24 2009, Xiaotian Feng wrote:
> blk_integrity_unregister should use kobject_put to release the kobject,
> otherwise after bi is freed, memory of bi->kobj->name is leaked.

Applied, thanks.

>
> Signed-off-by: Xiaotian Feng <dfeng@xxxxxxxxxx>
> ---
> block/blk-integrity.c | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/block/blk-integrity.c b/block/blk-integrity.c
> index 73e28d3..15c6308 100644
> --- a/block/blk-integrity.c
> +++ b/block/blk-integrity.c
> @@ -379,6 +379,7 @@ void blk_integrity_unregister(struct gendisk *disk)
>
> kobject_uevent(&bi->kobj, KOBJ_REMOVE);
> kobject_del(&bi->kobj);
> + kobject_put(&bi->kobj);
> kmem_cache_free(integrity_cachep, bi);
> disk->integrity = NULL;
> }
> --
> 1.6.2.5
>

--
Jens Axboe

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/