While commit 55f1ea15216 ("efi: Fix for_each_efi_memory_desc_in_map() for empty memmaps") made an attempt to deal with empty memory maps, it didn't address the case where the desc_size field never gets set, as is apparently the case when running under Xen. Reported-by: <lists@xxxxxxxxxxxx> Cc: Vitaly Kuznetsov <vkuznets@xxxxxxxxxx> Cc: Jiri Slaby <jslaby@xxxxxxx> Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx> Tested-by: <lists@xxxxxxxxxxxx> --- include/linux/efi.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) --- 4.8-rc1/include/linux/efi.h +++ 4.8-rc1-EFI-memdesc-iterator-Xen/include/linux/efi.h @@ -946,7 +946,7 @@ extern int efi_memattr_apply_permissions /* Iterate through an efi_memory_map */ #define for_each_efi_memory_desc_in_map(m, md) \ for ((md) = (m)->map; \ - ((void *)(md) + (m)->desc_size) <= (m)->map_end; \ + ((void *)(md) + (m)->desc_size - 1) < (m)->map_end; \ (md) = (void *)(md) + (m)->desc_size) /** -- To unsubscribe from this list: send the line "unsubscribe linux-efi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html