Re: [PATCH] Fix NULL pointer dereference in bl_free_device().

2016-07-19 Thread Anna Schumaker
Hi Artem, On 07/19/2016 07:30 AM, Artem Savkov wrote: > When bl_parse_deviceid() fails in bl_alloc_deviceid_node() on > blkdev_get_by_*() step we get an pnfs_block_dev struct that is > uninitialized except for bdev field which is set to whatever error > blkdev_get_by_*() returns. bl_free_device()

Re: [PATCH] Fix NULL pointer dereference in bl_free_device().

2016-07-19 Thread Anna Schumaker
Hi Artem, On 07/19/2016 07:30 AM, Artem Savkov wrote: > When bl_parse_deviceid() fails in bl_alloc_deviceid_node() on > blkdev_get_by_*() step we get an pnfs_block_dev struct that is > uninitialized except for bdev field which is set to whatever error > blkdev_get_by_*() returns. bl_free_device()

[PATCH] Fix NULL pointer dereference in bl_free_device().

2016-07-19 Thread Artem Savkov
When bl_parse_deviceid() fails in bl_alloc_deviceid_node() on blkdev_get_by_*() step we get an pnfs_block_dev struct that is uninitialized except for bdev field which is set to whatever error blkdev_get_by_*() returns. bl_free_device() then tries to call blkdev_put() if bdev is not 0 resulting in