cmd/extension: avoid NULL pointer dereference
authorHeinrich Schuchardt <heinrich.schuchardt@canonical.com>
Sun, 16 Nov 2025 11:41:53 +0000 (12:41 +0100)
committerTom Rini <trini@konsulko.com>
Thu, 27 Nov 2025 14:43:45 +0000 (08:43 -0600)
extension_get_list() will return NULL if there is no extension device.
Check for this situation.

Addresses-Coverity-ID: 638557 - Null pointer dereferences (NULL_RETURNS)
Fixes: 2d12958ee71b ("boot: Remove legacy extension board support")
Signed-off-by: Heinrich Schuchardt <heinrich.schuchardt@canonical.com>
Acked-by: Kory Maincent <kory.maincent@bootlin.com>
cmd/extension_board.c

index 86e4795..75358ab 100644 (file)
@@ -99,6 +99,10 @@ static int do_extension_list(struct cmd_tbl *cmdtp, int flag,
        int i = 0;
 
        extension_list = extension_get_list();
+       if (!extension_list) {
+               printf("No extension device\n");
+               return CMD_RET_FAILURE;
+       }
        if (!alist_get_ptr(extension_list, 0)) {
                printf("No extension registered - Please run \"extension scan\"\n");
                return CMD_RET_SUCCESS;