]> git.ipfire.org Git - thirdparty/kernel/stable.git/commit
powercap: fix sscanf() error return value handling
authorSumeet Pawnikar <sumeet4linux@gmail.com>
Sun, 7 Dec 2025 15:15:48 +0000 (20:45 +0530)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 17 Jan 2026 15:31:28 +0000 (16:31 +0100)
commitfeb28b6827ece47cce585599a00b02ee579532bc
treea3d9831dd2af6f296477f379ab3dc820bc155e31
parent3835d59f6999338220ceaf41cfa60967c90b84a8
powercap: fix sscanf() error return value handling

[ Upstream commit efc4c35b741af973de90f6826bf35d3b3ac36bf1 ]

Fix inconsistent error handling for sscanf() return value check.

Implicit boolean conversion is used instead of explicit return
value checks. The code checks if (!sscanf(...)) which is incorrect
because:
 1. sscanf returns the number of successfully parsed items
 2. On success, it returns 1 (one item passed)
 3. On failure, it returns 0 or EOF
 4. The check 'if (!sscanf(...))' is wrong because it treats
    success (1) as failure

All occurrences of sscanf() now uses explicit return value check.
With this behavior it returns '-EINVAL' when parsing fails (returns
0 or EOF), and continues when parsing succeeds (returns 1).

Signed-off-by: Sumeet Pawnikar <sumeet4linux@gmail.com>
[ rjw: Subject and changelog edits ]
Link: https://patch.msgid.link/20251207151549.202452-1-sumeet4linux@gmail.com
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
drivers/powercap/powercap_sys.c