Skip to content

Commit 62d5e97

Browse files
tlendackybp3tk0v
authored andcommitted
x86/sev: Change npages to unsigned long in snp_accept_memory()
In snp_accept_memory(), the npages variables value is calculated from phys_addr_t variables but is an unsigned int. A very large range passed into snp_accept_memory() could lead to truncating npages to zero. This doesn't happen at the moment but let's be prepared. Fixes: 6c32117 ("x86/sev: Add SNP-specific unaccepted memory support") Signed-off-by: Tom Lendacky <[email protected]> Signed-off-by: Borislav Petkov (AMD) <[email protected]> Cc: <[email protected]> Link: https://lore.kernel.org/r/6d511c25576494f682063c9fb6c705b526a3757e.1687441505.git.thomas.lendacky@amd.com
1 parent 6bc6f7d commit 62d5e97

File tree

1 file changed

+1
-2
lines changed

1 file changed

+1
-2
lines changed

arch/x86/kernel/sev.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -868,8 +868,7 @@ void snp_set_memory_private(unsigned long vaddr, unsigned long npages)
868868

869869
void snp_accept_memory(phys_addr_t start, phys_addr_t end)
870870
{
871-
unsigned long vaddr;
872-
unsigned int npages;
871+
unsigned long vaddr, npages;
873872

874873
if (!cc_platform_has(CC_ATTR_GUEST_SEV_SNP))
875874
return;

0 commit comments

Comments
 (0)