On Sun, 2024-09-29 at 12:56 +0200, Thorsten Blum wrote: > Use vmalloc_array() instead of vmalloc() to calculate the number of > bytes to allocate. > > Signed-off-by: Thorsten Blum <[email protected]> > --- > arch/x86/kernel/cpu/sgx/main.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/x86/kernel/cpu/sgx/main.c > b/arch/x86/kernel/cpu/sgx/main.c > index 9ace84486499..1a59e5956f4b 100644 > --- a/arch/x86/kernel/cpu/sgx/main.c > +++ b/arch/x86/kernel/cpu/sgx/main.c > @@ -630,7 +630,7 @@ static bool __init sgx_setup_epc_section(u64 > phys_addr, u64 size, > if (!section->virt_addr) > return false; > > - section->pages = vmalloc(nr_pages * sizeof(struct > sgx_epc_page)); > + section->pages = vmalloc_array(nr_pages, sizeof(struct > sgx_epc_page)); > if (!section->pages) { > memunmap(section->virt_addr); > return false;
Reviewed-by: Jarkko Sakkinen <[email protected]> BR, Jarkko
