diff options
| author | Thorsten Blum <thorsten.blum@linux.dev> | 2026-01-13 09:21:26 +0100 |
|---|---|---|
| committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2026-01-15 21:21:16 +0100 |
| commit | 75faabf6b92ad1d14fce8d45cf40f6db53cf96fd (patch) | |
| tree | caa3b08a9b1ce81d609c09dfeb5c67556cca76b0 | |
| parent | 47c9ef95a79a72653a5e5274e46f013df6b04626 (diff) | |
thermal: intel: Use sysfs_emit() in a sysfs show function
Replace sprintf() with sysfs_emit() in sysfs show functions.
sysfs_emit() is preferred to format sysfs output as it provides better
bounds checking.
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
[ rjw: Subject tweaks ]
Link: https://patch.msgid.link/20260113082130.789891-2-thorsten.blum@linux.dev
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
| -rw-r--r-- | drivers/thermal/intel/therm_throt.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/drivers/thermal/intel/therm_throt.c b/drivers/thermal/intel/therm_throt.c index debc94e2dc16..44fa4dd15dd1 100644 --- a/drivers/thermal/intel/therm_throt.c +++ b/drivers/thermal/intel/therm_throt.c @@ -23,6 +23,7 @@ #include <linux/types.h> #include <linux/init.h> #include <linux/smp.h> +#include <linux/sysfs.h> #include <linux/cpu.h> #include <asm/processor.h> @@ -144,8 +145,8 @@ static ssize_t therm_throt_device_show_##event##_##name( \ \ preempt_disable(); /* CPU hotplug */ \ if (cpu_online(cpu)) { \ - ret = sprintf(buf, "%lu\n", \ - per_cpu(thermal_state, cpu).event.name); \ + ret = sysfs_emit(buf, "%lu\n", \ + per_cpu(thermal_state, cpu).event.name); \ } else \ ret = 0; \ preempt_enable(); \ |