aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorFUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>2008-01-14 23:17:47 -0500
committerJames Bottomley <James.Bottomley@HansenPartnership.com>2008-01-23 12:29:28 -0500
commita24484f281dda11e4fc0041a51907d3564da3d7f (patch)
tree4ce9a5b3f7911d417e16dddf11a9b23f863cce0b
parentd814c5173c671cf2c88d4e07db01b13312467ecd (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>
-rw-r--r--drivers/scsi/sg.c3
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
1456cdev_add_err: 1457cdev_add_err: