diff options
| author | Shao Mingyin <shao.mingyin@zte.com.cn> | 2026-10-31 22:21:33 +0800 |
|---|---|---|
| committer | Martin K. Petersen (Oracle) <mkp@kernel.org> | 2026-11-08 22:17:43 -0500 |
| commit | 787fd1429b3603fdc69ebba864165478c4e73544 (patch) | |
| tree | 4a463e8d6bc0630438b539734691336e8b852ee8 /scripts/Makefile.thinlto | |
| parent | dd86dc1d86c892c75592d45e000499dfb20d2c62 (diff) | |
| download | linux-next-787fd1429b3603fdc69ebba864165478c4e73544.tar.gz linux-next-787fd1429b3603fdc69ebba864165478c4e73544.zip | |
scsi: scsi_devinfo: Return an error when the proc write fails
proc_scsi_devinfo_write() calls scsi_dev_info_list_add_str() but ignores
its return value, so a write to /proc/scsi/device_info always reports
success, even when the string is malformed and no entry was added. Users
and scripts then believe that the quirk is in effect.
scsi_init_devinfo() already checks the return value of the same function
and fails the initialization. Return the parser error, but keep returning
the number of bytes written on success: a proc write handler that returns
zero for a non-empty write would look like a short write to the caller.
Signed-off-by: Shao Mingyin <shao.mingyin@zte.com.cn>
Reviewed-by: Laurence Oberman <loberman@redhat.com>
Link: https://patch.msgid.link/20261031222133489hI-BEIDpFMxrtlntnLiFb@zte.com.cn
Signed-off-by: Martin K. Petersen (Oracle) <mkp@kernel.org>
Diffstat (limited to 'scripts/Makefile.thinlto')
0 files changed, 0 insertions, 0 deletions
