m68k/emu/nfblock: add error handling support for add_disk()
authorLuis Chamberlain <mcgrof@kernel.org>
Fri, 15 Oct 2021 23:30:25 +0000 (16:30 -0700)
committerJens Axboe <axboe@kernel.dk>
Thu, 21 Oct 2021 15:00:56 +0000 (09:00 -0600)
We never checked for errors on add_disk() as this function
returned void. Now that this is fixed, use the shiny new
error handling.

Reviewed-by: Geert Uytterhoeven <geert@linux-m68k.org>
Acked-by: Geert Uytterhoeven <geert@linux-m68k.org>
Signed-off-by: Luis Chamberlain <mcgrof@kernel.org>
Link: https://lore.kernel.org/r/20211015233028.2167651-7-mcgrof@kernel.org
Signed-off-by: Jens Axboe <axboe@kernel.dk>
arch/m68k/emu/nfblock.c

index 4ef457ba52209c8bdb263463ca27426dbd26dd83..9c57b245dc12aa4ad5a9eace20dcb30315163365 100644 (file)
@@ -99,6 +99,7 @@ static int __init nfhd_init_one(int id, u32 blocks, u32 bsize)
 {
        struct nfhd_device *dev;
        int dev_id = id - NFHD_DEV_OFFSET;
+       int err = -ENOMEM;
 
        pr_info("nfhd%u: found device with %u blocks (%u bytes)\n", dev_id,
                blocks, bsize);
@@ -129,16 +130,20 @@ static int __init nfhd_init_one(int id, u32 blocks, u32 bsize)
        sprintf(dev->disk->disk_name, "nfhd%u", dev_id);
        set_capacity(dev->disk, (sector_t)blocks * (bsize / 512));
        blk_queue_logical_block_size(dev->disk->queue, bsize);
-       add_disk(dev->disk);
+       err = add_disk(dev->disk);
+       if (err)
+               goto out_cleanup_disk;
 
        list_add_tail(&dev->list, &nfhd_list);
 
        return 0;
 
+out_cleanup_disk:
+       blk_cleanup_disk(dev->disk);
 free_dev:
        kfree(dev);
 out:
-       return -ENOMEM;
+       return err;
 }
 
 static int __init nfhd_init(void)