Re: [PATCH] mailbox: pcc: Fix an invalid-load caught by the address sanitizer

From: Sudeep Holla
Date: Tue May 10 2022 - 11:10:35 EST


On Mon, May 09, 2022 at 09:17:16AM -0500, Mario Limonciello wrote:
> `pcc_mailbox_probe` doesn't initialize all memory that has been allocated
> before the first time that one of it's members `txdone_irq` may be
> accessed.
>
> This leads to a an invalid load any time that this member is accessed:
> [ 2.429769] UBSAN: invalid-load in drivers/mailbox/pcc.c:684:22
> [ 2.430324] UBSAN: invalid-load in drivers/mailbox/mailbox.c:486:12
> [ 4.276782] UBSAN: invalid-load in drivers/acpi/cppc_acpi.c:314:45
>
> Link: https://bugzilla.kernel.org/show_bug.cgi?id=215587
> Fixes: ce028702ddbc ("mailbox: pcc: Move bulk of PCCT parsing into pcc_mbox_probe")

Thanks for catching and fixing this.

Reviewed-by: Sudeep Holla <sudeep.holla@xxxxxxx>

--
Regards,
Sudeep