]> Gentwo Git Trees - linux/.git/commitdiff
leds: led-class: Replace sprintf() with sysfs_emit() in sysfs show functions
authorThorsten Blum <thorsten.blum@linux.dev>
Mon, 3 Nov 2025 12:08:09 +0000 (13:08 +0100)
committerLee Jones <lee@kernel.org>
Thu, 13 Nov 2025 15:22:24 +0000 (15:22 +0000)
Replace sprintf() with sysfs_emit() in sysfs show functions.
sysfs_emit() is preferred to format sysfs output as it provides better
bounds checking.  No functional changes.

Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
Link: https://patch.msgid.link/20251103120809.32834-1-thorsten.blum@linux.dev
Signed-off-by: Lee Jones <lee@kernel.org>
drivers/leds/led-class.c

index f3faf37f9a08ac762ed87b91cb3cab5faa8eacb0..885399ed077687222b9170d9a90e9c1dd6dd9bc4 100644 (file)
@@ -38,7 +38,7 @@ static ssize_t brightness_show(struct device *dev,
        brightness = led_cdev->brightness;
        mutex_unlock(&led_cdev->led_access);
 
-       return sprintf(buf, "%u\n", brightness);
+       return sysfs_emit(buf, "%u\n", brightness);
 }
 
 static ssize_t brightness_store(struct device *dev,
@@ -80,7 +80,7 @@ static ssize_t max_brightness_show(struct device *dev,
        max_brightness = led_cdev->max_brightness;
        mutex_unlock(&led_cdev->led_access);
 
-       return sprintf(buf, "%u\n", max_brightness);
+       return sysfs_emit(buf, "%u\n", max_brightness);
 }
 static DEVICE_ATTR_RO(max_brightness);
 
@@ -122,7 +122,7 @@ static ssize_t brightness_hw_changed_show(struct device *dev,
        if (led_cdev->brightness_hw_changed == -1)
                return -ENODATA;
 
-       return sprintf(buf, "%u\n", led_cdev->brightness_hw_changed);
+       return sysfs_emit(buf, "%u\n", led_cdev->brightness_hw_changed);
 }
 
 static DEVICE_ATTR_RO(brightness_hw_changed);