mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2026-09-22 09:34:56 +02:00
Input: ims-pcu - fix firmware leak in async update
The firmware object was not being released if validation failed.
Use __free(firmware) to ensure the firmware is always released.
Fixes: 628329d524 ("Input: add IMS Passenger Control Unit driver")
Cc: stable@vger.kernel.org
Reported-by: Sashiko bot <sashiko-bot@kernel.org>
Assisted-by: Gemini:gemini-3.1-pro
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
This commit is contained in:
@@ -935,9 +935,10 @@ out:
|
||||
return retval;
|
||||
}
|
||||
|
||||
static void ims_pcu_process_async_firmware(const struct firmware *fw,
|
||||
static void ims_pcu_process_async_firmware(const struct firmware *_fw,
|
||||
void *context)
|
||||
{
|
||||
const struct firmware *fw __free(firmware) = _fw;
|
||||
struct ims_pcu *pcu = context;
|
||||
int error;
|
||||
|
||||
@@ -957,8 +958,6 @@ static void ims_pcu_process_async_firmware(const struct firmware *fw,
|
||||
scoped_guard(mutex, &pcu->cmd_mutex)
|
||||
ims_pcu_handle_firmware_update(pcu, fw);
|
||||
|
||||
release_firmware(fw);
|
||||
|
||||
out:
|
||||
complete(&pcu->async_firmware_done);
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user