diff options
author | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2007-07-17 10:20:46 (GMT) |
---|---|---|
committer | Jens Axboe <jens.axboe@oracle.com> | 2007-07-17 10:20:46 (GMT) |
commit | 9b9f770cef0037fd0e4d623f31ddfa8acda38205 (patch) | |
tree | a0b8937223df1e5a519375d1fa797157b4a930a2 /block | |
parent | 5cdf7f7678493996a835ef2e627976de8f9d1f40 (diff) | |
download | linux-fsl-qoriq-9b9f770cef0037fd0e4d623f31ddfa8acda38205.tar.xz |
bsg: fix initialization error handling bugs
This fixes the following bugs and cleans up the initialization code:
- cdev_del is missing.
- unregister_chrdev_region should be used instead of unregister_chrdev.
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Signed-off-by: Jens Axboe <jens.axboe@oracle.com>
Diffstat (limited to 'block')
-rw-r--r-- | block/bsg.c | 38 |
1 files changed, 18 insertions, 20 deletions
diff --git a/block/bsg.c b/block/bsg.c index cdb00e5..b253784 100644 --- a/block/bsg.c +++ b/block/bsg.c @@ -1057,39 +1057,37 @@ static int __init bsg_init(void) bsg_class = class_create(THIS_MODULE, "bsg"); if (IS_ERR(bsg_class)) { - kmem_cache_destroy(bsg_cmd_cachep); - return PTR_ERR(bsg_class); + ret = PTR_ERR(bsg_class); + goto destroy_kmemcache; } ret = alloc_chrdev_region(&devid, 0, BSG_MAX_DEVS, "bsg"); - if (ret) { - kmem_cache_destroy(bsg_cmd_cachep); - class_destroy(bsg_class); - return ret; - } + if (ret) + goto destroy_bsg_class; bsg_major = MAJOR(devid); cdev_init(&bsg_cdev, &bsg_fops); ret = cdev_add(&bsg_cdev, MKDEV(bsg_major, 0), BSG_MAX_DEVS); - if (ret) { - kmem_cache_destroy(bsg_cmd_cachep); - class_destroy(bsg_class); - unregister_chrdev_region(MKDEV(bsg_major, 0), BSG_MAX_DEVS); - return ret; - } + if (ret) + goto unregister_chrdev; ret = scsi_register_interface(&bsg_intf); - if (ret) { - printk(KERN_ERR "bsg: failed register scsi interface %d\n", ret); - kmem_cache_destroy(bsg_cmd_cachep); - class_destroy(bsg_class); - unregister_chrdev(bsg_major, "bsg"); - return ret; - } + if (ret) + goto remove_cdev; printk(KERN_INFO "%s loaded (major %d)\n", bsg_version, bsg_major); return 0; +remove_cdev: + printk(KERN_ERR "bsg: failed register scsi interface %d\n", ret); + cdev_del(&bsg_cdev); +unregister_chrdev: + unregister_chrdev_region(MKDEV(bsg_major, 0), BSG_MAX_DEVS); +destroy_bsg_class: + class_destroy(bsg_class); +destroy_kmemcache: + kmem_cache_destroy(bsg_cmd_cachep); + return ret; } MODULE_AUTHOR("Jens Axboe"); |