lightnvm: fix cleanup order of disk on init error
authorJavier González <jg@lightnvm.io>
Sat, 15 Apr 2017 18:55:43 +0000 (20:55 +0200)
committerJens Axboe <axboe@fb.com>
Sun, 16 Apr 2017 16:06:25 +0000 (10:06 -0600)
Reorder disk allocation such that the disk structure can be put
safely.

Signed-off-by: Javier González <javier@cnexlabs.com>
Signed-off-by: Matias Bjørling <matias@cnexlabs.com>
Signed-off-by: Jens Axboe <axboe@fb.com>
drivers/lightnvm/core.c

index 5eea3d5ecc30d395cd20625ffa7beb011d899263..5f84d2a418f642970065f88db1aa3bbf166fba23 100644 (file)
@@ -264,15 +264,15 @@ static int nvm_create_tgt(struct nvm_dev *dev, struct nvm_ioctl_create *create)
                goto err_t;
        }
 
+       tdisk = alloc_disk(0);
+       if (!tdisk)
+               goto err_dev;
+
        tqueue = blk_alloc_queue_node(GFP_KERNEL, dev->q->node);
        if (!tqueue)
-               goto err_dev;
+               goto err_disk;
        blk_queue_make_request(tqueue, tt->make_rq);
 
-       tdisk = alloc_disk(0);
-       if (!tdisk)
-               goto err_queue;
-
        sprintf(tdisk->disk_name, "%s", create->tgtname);
        tdisk->flags = GENHD_FL_EXT_DEVT;
        tdisk->major = 0;
@@ -308,9 +308,9 @@ err_sysfs:
        if (tt->exit)
                tt->exit(targetdata);
 err_init:
-       put_disk(tdisk);
-err_queue:
        blk_cleanup_queue(tqueue);
+err_disk:
+       put_disk(tdisk);
 err_dev:
        nvm_remove_tgt_dev(tgt_dev, 0);
 err_t: