mirror of
https://github.com/torvalds/linux.git
synced 2026-07-27 17:47:41 +02:00
rtc: ab8500: replace sprintf() with sysfs_emit()
This patch replaces sprintf() with sysfs_emit() to ensure proper bounds checking. It also simplifies the return logic by directly returning the error after logging, instead of logging, calling sprintf(), then returning. Reviewed-by: Linus Walleij <linusw@kernel.org> Signed-off-by: Maxwell Doose <m32285159@gmail.com> Link: https://patch.msgid.link/20260503201236.29685-1-m32285159@gmail.com Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
This commit is contained in:
parent
254f49634e
commit
b723868644
|
|
@ -284,11 +284,10 @@ static ssize_t ab8500_sysfs_show_rtc_calibration(struct device *dev,
|
|||
retval = ab8500_rtc_get_calibration(dev, &calibration);
|
||||
if (retval < 0) {
|
||||
dev_err(dev, "Failed to read RTC calibration attribute\n");
|
||||
sprintf(buf, "0\n");
|
||||
return retval;
|
||||
}
|
||||
|
||||
return sprintf(buf, "%d\n", calibration);
|
||||
return sysfs_emit(buf, "%d\n", calibration);
|
||||
}
|
||||
|
||||
static DEVICE_ATTR(rtc_calibration, S_IRUGO | S_IWUSR,
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user