diff options
author | FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp> | 2008-01-14 23:17:47 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-01-23 12:29:28 -0500 |
commit | a24484f281dda11e4fc0041a51907d3564da3d7f (patch) | |
tree | 4ce9a5b3f7911d417e16dddf11a9b23f863cce0b /drivers/scsi | |
parent | d814c5173c671cf2c88d4e07db01b13312467ecd (diff) |
[SCSI] sg: set class_data after success
If cdev_add fails in sg_add, sg_remove crashes since class_data is
bogus.
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Acked-by: Douglas Gilbert <dougg@torque.net>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/sg.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/scsi/sg.c b/drivers/scsi/sg.c index e65f5d432662..ebbf9b4b7e90 100644 --- a/drivers/scsi/sg.c +++ b/drivers/scsi/sg.c | |||
@@ -1422,7 +1422,6 @@ sg_add(struct class_device *cl_dev, struct class_interface *cl_intf) | |||
1422 | goto out; | 1422 | goto out; |
1423 | } | 1423 | } |
1424 | 1424 | ||
1425 | class_set_devdata(cl_dev, sdp); | ||
1426 | error = cdev_add(cdev, MKDEV(SCSI_GENERIC_MAJOR, sdp->index), 1); | 1425 | error = cdev_add(cdev, MKDEV(SCSI_GENERIC_MAJOR, sdp->index), 1); |
1427 | if (error) | 1426 | if (error) |
1428 | goto cdev_add_err; | 1427 | goto cdev_add_err; |
@@ -1451,6 +1450,8 @@ sg_add(struct class_device *cl_dev, struct class_interface *cl_intf) | |||
1451 | "Attached scsi generic sg%d type %d\n", sdp->index, | 1450 | "Attached scsi generic sg%d type %d\n", sdp->index, |
1452 | scsidp->type); | 1451 | scsidp->type); |
1453 | 1452 | ||
1453 | class_set_devdata(cl_dev, sdp); | ||
1454 | |||
1454 | return 0; | 1455 | return 0; |
1455 | 1456 | ||
1456 | cdev_add_err: | 1457 | cdev_add_err: |