]> git.hungrycats.org Git - linux/commit
module: decompress: check return value of module_extend_max_pages()
authorAndrii Kuchmenko <capyenglishlite@gmail.com>
Mon, 18 May 2026 14:32:33 +0000 (17:32 +0300)
committerSami Tolvanen <samitolvanen@google.com>
Thu, 4 Jun 2026 16:37:32 +0000 (16:37 +0000)
commit786d2d84416a9a1c1a47b71a68d679d886284be2
tree06e4a0f75d5381e36489ace097ce386951aa23d2
parentac2f40107cf1296955c867dc31b1a1eceb8ec569
module: decompress: check return value of module_extend_max_pages()

module_extend_max_pages() calls kvrealloc() internally and returns
-ENOMEM on allocation failure. The return value is never checked.

If the initial allocation fails, info->pages remains NULL and
info->max_pages remains 0. Subsequent calls to module_get_next_page()
will attempt to dynamically grow the array by calling
module_extend_max_pages(info, 0) since info->used_pages is 0. This
results in kvrealloc(NULL, 0) returning ZERO_SIZE_PTR, which is treated
as a success, leading to a dereference of ZERO_SIZE_PTR and a kernel
oops.

Fix: add the missing error check after module_extend_max_pages() and
return immediately on failure. This matches the pattern used by every
other kvrealloc() caller in the module loading path.

Fixes: b1ae6dc41eaa ("module: add in-kernel support for decompressing")
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Luis Chamberlain <mcgrof@kernel.org>
Cc: stable@vger.kernel.org
Signed-off-by: Andrii Kuchmenko <capyenglishlite@gmail.com>
Reviewed-by: Christophe Leroy (CS GROUP) <chleroy@kernel.org>
[Sami: Corrected the analysis in the commit message.]
Signed-off-by: Sami Tolvanen <samitolvanen@google.com>
kernel/module/decompress.c