[PATCH 1/1] dm: avoid NULL dereference in add_item()

Simon Glass sjg at chromium.org
Tue Jul 26 21:52:59 CEST 2022


On Sun, 10 Jul 2022 at 07:40, Heinrich Schuchardt
<heinrich.schuchardt at canonical.com> wrote:
>
> acpi_add_other_item() passes dev = NULL. Instead of dev->name write the
> string "other" to the debug log:
>
>     ACPI: Writing ACPI tables at 1fd3000
>     0base: writing table '<NULL>'
>     * other: Added type 3, 0000000011fd4000, size 240
>     1facs: writing table 'FACS'
>     * other: Added type 3, 0000000011fd4240, size 40
>     5csrt: writing table 'CSRT'
>     * other: Added type 3, 0000000011fd4280, size 30
>
> Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt at canonical.com>
> ---
>  drivers/core/acpi.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>

I'm not sure what the purpose of this is and the commit message does
not state it. But it seems reasonable:

Reviewed-by: Simon Glass <sjg at chromium.org>

Applied to u-boot-dm, thanks!


More information about the U-Boot mailing list