From: Greg Kroah-Hartman Date: Sat, 17 May 2008 00:55:12 +0000 (-0700) Subject: LEDS: fix race in device_create X-Git-Tag: v2.6.26-rc4~94^2~7 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=0b00fc5851551781e8a30153af2c94cee9fa84af;p=profile%2Fivi%2Fkernel-adaptation-intel-automotive.git LEDS: fix race in device_create There is a race from when a device is created with device_create() and then the drvdata is set with a call to dev_set_drvdata() in which a sysfs file could be open, yet the drvdata will be NULL, causing all sorts of bad things to happen. This patch fixes the problem by using the new function, device_create_drvdata(). Cc: Kay Sievers Cc: Richard Purdie Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/leds/led-class.c b/drivers/leds/led-class.c index b3c54be..559a408 100644 --- a/drivers/leds/led-class.c +++ b/drivers/leds/led-class.c @@ -103,13 +103,11 @@ int led_classdev_register(struct device *parent, struct led_classdev *led_cdev) { int rc; - led_cdev->dev = device_create(leds_class, parent, 0, "%s", - led_cdev->name); + led_cdev->dev = device_create_drvdata(leds_class, parent, 0, led_cdev, + "%s", led_cdev->name); if (IS_ERR(led_cdev->dev)) return PTR_ERR(led_cdev->dev); - dev_set_drvdata(led_cdev->dev, led_cdev); - /* register the attributes */ rc = device_create_file(led_cdev->dev, &dev_attr_brightness); if (rc)