diff options
| author | Pengpeng Hou <pengpeng@iscas.ac.cn> | 2026-06-15 13:58:50 +0800 |
|---|---|---|
| committer | Andreas Larsson <andreas@gaisler.com> | 2026-06-15 10:06:35 +0200 |
| commit | 7eb475e8a738ee6fd1260aa59ddccb610fdd4300 (patch) | |
| tree | 49632f33ac235db1540f153c357144f8a955ade8 | |
| parent | 2416b30ec23fd23126b4747994c00d926d438d6f (diff) | |
| download | linux-7eb475e8a738ee6fd1260aa59ddccb610fdd4300.tar.gz linux-7eb475e8a738ee6fd1260aa59ddccb610fdd4300.zip | |
sparc: led: avoid trimming a newline from empty writes
led_proc_write() duplicates up to LED_MAX_LENGTH bytes with
memdup_user_nul() and then unconditionally inspects buf[count - 1] to
strip a trailing newline. A zero-length write therefore reads one byte
before the duplicated buffer.
The previous version rejected empty writes, but empty input already falls
through to the existing default case and turns the LED off like any other
unrecognized string. Preserve that behavior and only skip the newline
trim when there is no input byte to inspect.
Fixes: ee1858d3122d ("[SPARC]: Add sun4m LED driver.")
Suggested-by: Andreas Larsson <andreas@gaisler.com>
Signed-off-by: Pengpeng Hou <pengpeng@iscas.ac.cn>
Signed-off-by: Andreas Larsson <andreas@gaisler.com>
| -rw-r--r-- | arch/sparc/kernel/led.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/arch/sparc/kernel/led.c b/arch/sparc/kernel/led.c index f4fb82b019bb..9b53ac1fe533 100644 --- a/arch/sparc/kernel/led.c +++ b/arch/sparc/kernel/led.c @@ -78,7 +78,7 @@ static ssize_t led_proc_write(struct file *file, const char __user *buffer, return PTR_ERR(buf); /* work around \n when echo'ing into proc */ - if (buf[count - 1] == '\n') + if (count > 0 && buf[count - 1] == '\n') buf[count - 1] = '\0'; /* before we change anything we want to stop any running timers, |