[PATCH] block: initialize ret in bdev_disk_changed

[PATCH] block: initialize ret in bdev_disk_changed

[Christoph Hellwig]

Avoid a potentially initialized variabe in the invalidate case.

Fixes: d3c4a43d9291 ("block: refactor blk_drop_partitions")
Reported-by: kernel test robot <lkp@intel.com>
Signed-off-by: Christoph Hellwig <hch@lst.de>
---
 fs/block_dev.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/block_dev.c b/fs/block_dev.c
index 594a1bee9dd9ac..363015fcffdbba 100644
--- a/fs/block_dev.c
+++ b/fs/block_dev.c
@@ -1236,7 +1236,7 @@ static void __blkdev_put(struct block_device *bdev, fmode_t mode, int for_part);
 int bdev_disk_changed(struct block_device *bdev, bool invalidate)
 {
 	struct gendisk *disk = bdev->bd_disk;
-	int ret;
+	int ret = 0;
 
 	lockdep_assert_held(&bdev->bd_mutex);
 
-- 
2.30.1

Re: [PATCH] block: initialize ret in bdev_disk_changed

[Chaitanya Kulkarni]

On 4/8/21 12:44, Christoph Hellwig wrote:
> Avoid a potentially initialized variabe in the invalidate case.
>
> Fixes: d3c4a43d9291 ("block: refactor blk_drop_partitions")
> Reported-by: kernel test robot <lkp@intel.com>
> Signed-off-by: Christoph Hellwig <hch@lst.de>
> ---

Looks good.

Reviewed-by: Chaitanya Kulkarni <chaitanya.kulkarni@wdc.com>

Re: [PATCH] block: initialize ret in bdev_disk_changed

[Keith Busch]

On Thu, Apr 08, 2021 at 09:41:40PM +0200, Christoph Hellwig wrote:
> Avoid a potentially initialized variabe in the invalidate case.

That should say "uninitialized variable", otherwise looks good.

Reviewed-by: Keith Busch <kbusch@kernel.org>

Re: [PATCH] block: initialize ret in bdev_disk_changed

[Jens Axboe]

On 4/8/21 1:41 PM, Christoph Hellwig wrote:
> Avoid a potentially initialized variabe in the invalidate case.

Applied, thanks.

-- 
Jens Axboe