Skip to content

Commit 92f4b01

Browse files
tobluxlag-linaro
authored andcommitted
leds: led-class: Replace sprintf() with sysfs_emit() in sysfs show functions
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>
1 parent c42b91c commit 92f4b01

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

drivers/leds/led-class.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ static ssize_t brightness_show(struct device *dev,
3838
brightness = led_cdev->brightness;
3939
mutex_unlock(&led_cdev->led_access);
4040

41-
return sprintf(buf, "%u\n", brightness);
41+
return sysfs_emit(buf, "%u\n", brightness);
4242
}
4343

4444
static ssize_t brightness_store(struct device *dev,
@@ -80,7 +80,7 @@ static ssize_t max_brightness_show(struct device *dev,
8080
max_brightness = led_cdev->max_brightness;
8181
mutex_unlock(&led_cdev->led_access);
8282

83-
return sprintf(buf, "%u\n", max_brightness);
83+
return sysfs_emit(buf, "%u\n", max_brightness);
8484
}
8585
static DEVICE_ATTR_RO(max_brightness);
8686

@@ -122,7 +122,7 @@ static ssize_t brightness_hw_changed_show(struct device *dev,
122122
if (led_cdev->brightness_hw_changed == -1)
123123
return -ENODATA;
124124

125-
return sprintf(buf, "%u\n", led_cdev->brightness_hw_changed);
125+
return sysfs_emit(buf, "%u\n", led_cdev->brightness_hw_changed);
126126
}
127127

128128
static DEVICE_ATTR_RO(brightness_hw_changed);

0 commit comments

Comments
 (0)